New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 693583 link

Starred by 3 users
Status: Duplicate
Merged: issue 697786
Owner: ----
Closed: Mar 2017
Cc:
█ dsansome@chromium.org
█ emso@chromium.org
j...@chromium.org
flyboy@chromium.org
rauch+monorail-auto@google.com
█ orodley+autocc@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Code and XRefs not properly synced

Project Member Reported by npm@chromium.org, Feb 17 2017 
What steps will reproduce the problem?
(1) Go to https://cs.chromium.org/chromium/src/third_party/pdfium/core/fxge/cfx_renderdevice.h?l=80
(2) Click on m_bFontStyle, there is a reference in cfx_renderdevice.cpp line 929, click on it.

What is the expected result?
cfx_renderdevice.cpp line 929 contains bFontStyle

What happens instead?
It does not, instead it should be line 944.

Comment 1 by emso@chromium.org, Feb 20 2017

Status: Assigned (was: Untriaged)
Testing this know I don't see the same problem but I saw the problem around the time this bug was filed (got interrupted while triaging bugs). With the recent git integration, files get updated "instantly" while Xrefs still have a turn around time of around 4 hours. I think there is some adjustment for the kind of differences that can happen in this situation, but there is a limit to how well things can be fixed.

We should investigate this further to clarify what the expectation should be with the current setup.

Comment 2 by dsansome@chromium.org, Feb 22 2017

Cc: dsansome@chromium.org emso@chromium.org
 Issue 694511  has been merged into this issue.

Comment 3 by emso@chromium.org, Feb 22 2017 

Second issue reported has the same problem of Xrefs being out of sync and then some time later the problem has disappeared. The likely cause of this is the recent Git integration which makes updating of file content and the search index much faster than the updating of Xrefs.

Comment 4 by dsansome@chromium.org, Feb 28 2017 

 Issue 696879  has been merged into this issue.

Comment 5 by emso@chromium.org, Mar 2 2017

Mergedinto: 697786
Status: Duplicate (was: Assigned)
►

Sign in to add a comment