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

Issue 672429 link

Starred by 1 user
Status: Fixed
Owner:
ew...@chromium.org
Closed: Dec 2016
Cc:
nyerramilli@chromium.org
█ brajkumar@chromium.org
anthonyvd@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Sync error messege doesn't appear on wrench menu list.

Reported by dchau...@etouch.net, Dec 8 2016 
Chrome Version: 56.0.2924.21 (Official Build)4d0fd85f092f947ef72046f62e953ecf44d5e64a-refs/branch-heads/2924@{#401} 32/64-bit.
OS: Mac(10.11.6, 10.12.1, 10.12), Windows(7,8,8.1,10), Linux (14.04 LTS).

What steps will reproduce the problem?
1. Launch chrome and sign in to chrome with those credentials which having passphrase enable.
2. Now, go wrench menu list and observe.

"Sync error: Update Sync passphrase.." message is not seen on wrench menu list
"Sync error: Update Sync passphrase.." message should seen on wrench menu list.

This is a regression issue, broken in M-56 series below is manual regression range.

Good build: 55.0.2883.0
Bad build: 56.0.2884.0

Kindly review the attached screen-cast for reference.
Actual behavior.mp4
1.4 MB View Download
Expected behavior.mp4
800 KB View Download

Comment 1 by brajkumar@chromium.org, Dec 8 2016

Cc: brajkumar@chromium.org
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: anthonyvd@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,

You are probably looking for a change made after 423786 (known good), but no later than 423787 (first known bad).

CHANGE-LOG URL:
---------------------------------------
https://chromium.googlesource.com/chromium/src/+log/520abebc83f34968d12bb892bd91b0ae28ed413b..e3c600aa49b1e2bed95bfe9298a733bd72a0cb4e

From the CL above, assigning the issue to the concern owner
Review-Url: https://codereview.chromium.org/2396893002

anthonyvd@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Note: Adding RB-Stable since this issue is broken in M-56, please feel free to edit if this is not the case.

Thanks!

Comment 2 by anthonyvd@chromium.org, Dec 8 2016

Cc: anthonyvd@chromium.org
Owner: ew...@chromium.org
+ewald. This is WAI since the new user menu, right?

Comment 3 by ew...@chromium.org, Dec 8 2016

Status: Fixed (was: Assigned)
Correct, closing out as WAI. These errors are now surfaced in a more visible and friendly way in the user menu chip (upper right hand corner).
Project Member

Comment 4 by blocker...@chrome-infra-auth.iam.gserviceaccount.com, Dec 8 2016

Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-56; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-56 label, otherwise remove Merge-TBD label. Thanks.

Comment 5 by ligim...@chromium.org, Dec 13 2016 

Is there any merge required here?If not please remove Merge-TBD label.

Comment 6 by ew...@chromium.org, Dec 14 2016

Labels: -Merge-TBD
Project Member

Comment 7 by sheriffbot@chromium.org, Dec 16 2016

Labels: Merge-Review-56 Hotlist-Merge-Review
This bug requires manual review: No test file found in commits.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 8 by sheriffbot@chromium.org, Dec 16 2016 

This bug requires manual review: No test file found in commits.
Please contact the milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 9 by dimu@google.com, Dec 16 2016

Labels: -Merge-Review-56 -Hotlist-Merge-Review
[Automated comment] removing mislabelled Merge-Review-56, Hotlist-Merge-Review
►

Sign in to add a comment