Regression: Bookmarks are seen missing in bookmark manager |
|||||||
Issue descriptionVersion: 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.
,
May 10 2016
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.
,
May 10 2016
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
,
May 10 2016
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!
,
May 10 2016
Issue 610605 has been merged into this issue.
,
May 10 2016
This is caused by my change indeed. Looking...
,
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
,
May 10 2016
,
May 11 2016
Issue 610929 has been merged into this issue. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by sc00335...@techmahindra.com
, May 10 2016