My favorites | Sign in
Project Home Wiki Issues
New issue   Search
for
  Advanced search   Search tips
Issue 828: Anchors to lines in diff viewer don't work
2 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  chip...@gmail.com
Closed:  Feb 2009


Sign in to add a comment
 
Reported by mkoeb...@gmail.com, Jan 22, 2009
*NOTE: Do not post confidential information in this bug report.*

What's the URL of the page containing the problem?
Any URL of the format
http://host/reviewboard/r/1148/diff/1/?file=26188#file26188line161

What steps will reproduce the problem?
1. go to a review and view the comments to a certain line of code in the diff
2. Click on the line of code which is a link with a url similar as
described above
3. The browser shows you the diff but does not scroll down to the
previously clicked line of code.

What is the expected output? What do you see instead?
I would expect that the browser scrolls down to the line of code I just
clicked on.

What operating system are you using? What browser?
firefox 2 & 3, linux and windows

Please provide any additional information below.
Not sure whether that is related...(probably not)
Under windows only and firefox 2 I get the following js error when I click
on the link to the line of code a comment was left for:

Error: [Exception... "Component is not available"  nsresult: "0x80040111
(NS_ERROR_NOT_AVAILABLE)"  location: "JS frame ::
file:///C:/PROGRA~1/MOZILL~1/components/nsSessionStore.js :: sss_saveState
:: line 1772"  data: no]
Source File: file:///C:/PROGRA~1/MOZILL~1/components/nsSessionStore.js
Line: 1772

the code it refers to is:

  /**

   * save state to disk

   * @param aUpdateAll

   *        Bool update all windows 

   */

  saveState: function sss_saveState(aUpdateAll) {

    // if crash recovery is disabled, only save session resuming information

    if (!this._resume_from_crash && this._loadState == STATE_RUNNING)

      return;

    

    this._dirty = aUpdateAll;

    var oState = this._getCurrentState();

1772:    oState.session = { state: ((this._loadState == STATE_RUNNING) ?
STATE_RUNNING_STR : STATE_STOPPED_STR) };

    this._writeFile(this._sessionFile, oState.toSource());

    this._lastSaveTime = Date.now();

  },


Comment 1 by project member chip...@gmail.com, Jan 23, 2009
(No comment was entered for this change.)
Status: Confirmed
Labels: Milestone-Release1.0 Component-DiffViewer
Comment 2 by project member chip...@gmail.com, Jan 23, 2009
That error seems unrelated.

Found and fixed the problem. Should be fixed in r1702.
Status: Fixed
Owner: chipx86
Comment 3 by mkoeb...@gmail.com, Feb 3, 2009
the progressive load must have come in its way.

Initially, the anchor is not visited.
So you click on the link, the browser loads the diff but does not scroll down.
If you now click with the mouse into the URL field of your browser and hit enter (not
F5!), the browser will now scroll down to it.
Comment 4 by project member chip...@gmail.com, Feb 3, 2009
Ah, right, that definitely will cause issues. I'll look into possible fixes for this
for beta 1.
Status: Confirmed
Comment 5 by project member chip...@gmail.com, Feb 15, 2009
Fixed in r1783.
Status: Fixed
Sign in to add a comment

Powered by Google Project Hosting