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

Issue 610602 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Bookmarks are seen missing in bookmark manager

Project Member Reported by sc00335...@techmahindra.com, May 10 2016

Issue description

Version: 52.0.2730.0 dev
OS: Ubuntu 12.04,14.04

What steps will reproduce the problem?
(1) Launch chrome and sign to chrome with valid credentials
(2) After syncing go to bookmark manager or hit ctrl+shift+o and observe for bookmarks

Expected: Bookmarks should be seen in bookmark manager and bookmark bar
Actual: Instead bookmarks are seen missing in bookmark manager but seen in bookmark bar.

This is a regression issue broken in M52. Will provide bisect info soon.


 
Actual_bookmarks.png
99.0 KB View Download
Labels: OS-Windows
Issue is also seen in windows.

Comment 2 by ajha@chromium.org, May 10 2016

Labels: ReleaseBlock-Dev OS-Mac
Status: Untriaged (was: Unconfirmed)
Issue is reproducible on the latest canary(52.0.2730.0) on Mac OS 10.11.4 as well.

Note: Issue is not seen on 52.0.2729.0.


Summary: Regression: Bookmarks are seen missing in bookmark manager (was: Regression: Synced bookmarks are not seen in bookmark manager)
Many more issue are seen in bookmarks

1.Steps: Go to Bookmark manager >> Try adding page/folder from organize/folders sections and observe for bookmarked data in bookmark manager

2.Steps: Star any page on top right corner and observe for bookmarked page in bookmark manager

Comment 4 by ajha@chromium.org, May 10 2016

Labels: -Needs-Bisect hasbisect
Owner: dpa...@chromium.org
Status: Assigned (was: Untriaged)
Last good build: 52.0.2729.0
First bad build: 52.0.2730.0

Changelog:
==========
https://chromium.googlesource.com/chromium/src/+log/eab436d1bc2621beb79603ad478d9e21baff739d..ab21e7d69da6e7fa059dba9fc363db630ea2de02

Suspecting: https://codereview.chromium.org/1919183005

dpapad@: Could you please take a look and confirm if the change could be related.

Appreciate your help!
Cc: dpa...@chromium.org nyerramilli@chromium.org ashej...@chromium.org
 Issue 610605  has been merged into this issue.

Comment 6 by dpa...@chromium.org, May 10 2016

Status: Started (was: Assigned)
This is caused by my change indeed. Looking...
Project Member

Comment 7 by bugdroid1@chromium.org, May 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/87d0ec171e127ec7477c5d323f5e0d8715af12e5

commit 87d0ec171e127ec7477c5d323f5e0d8715af12e5
Author: dpapad <dpapad@chromium.org>
Date: Tue May 10 18:24:26 2016

WebUI: Fix bookmark manager regression.

BUG= 610602 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/1965863002
Cr-Commit-Position: refs/heads/master@{#392656}

[modify] https://crrev.com/87d0ec171e127ec7477c5d323f5e0d8715af12e5/chrome/browser/resources/bookmark_manager/main.html

Comment 8 by dpa...@chromium.org, May 10 2016

Status: Fixed (was: Started)
 Issue 610929  has been merged into this issue.

Sign in to add a comment