2654: Set, jump between bookmarks in DiffViewer

ryan.b******@gmai***** (Google Code) (Is this you? Claim this profile.)
Feb. 4, 2014
774
3082
> What version are you running?
1.6.1

> What's the URL of the page this enhancement relates to, if any?
???

https://reviewboard.example.com/r/12345/diff/#index_header

> Describe the enhancement and the motivation for it.

Developer posts a sizable review with a fair number of functions spread across multiple files.  During review, I want to sanity check calls into new function whose return value, memory allocations, etc. I don't know.  So, I need to jump to the called function, see what it does, what its side effects are, etc., and then return to the call to see if things check out.  (For example, did the called function return a pointer to something allocated from the heap, and does the current function need to free it in case of failure?)

With the current diff viewer, I have to manually search for the definition of the new function, then manually go back to its call site.  It'd be nice if I could 
   1.  record my current location in the diff viewer,
   2.  seek somewhere else in the diff, and then
   3.  press a key/keys or click something to immediately jump back to the saved location.

> What operating system are you using? What browser?
Google Chrome 18–19 on Linux and Windows 7.
david
#1 david
  • +Component-DiffViewer
david
#2 david
david
#3 david
Very similar to 774.