commit | dc4f172dfe2167666aff966b119181c57639e46d | [log] [tgz] |
---|---|---|
author | sangwoo.ko <sangwoo108@gmail.com> | Fri Feb 16 22:49:46 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Feb 16 22:49:46 2018 |
tree | 09787bf752b1bfd802810d92f714d5e3d25733ee | |
parent | 0a29b20ea9f2b1e02b76412f233321348889b7c0 [diff] |
Try saving pinned tabs on OnBrowserRemoved Bug:71939 PinnedTabService is observing two events: NOTIFICATION_CLOSE_ALL_BROWSERS_REQUEST and BrowserList::OnBrowserClosing. The first is when users want to shut down entire app and the latter is when users close a browser at once even if it has multiple tabs. But the bug is edge case which doesn't belong to any case above for now. When tab strip is empty, BrowserListObserver::OnBrowserClosing is not called even this will end up with closing the browser. So try saving pinned tabs again on BrowserListObserver::OnBrowserRemoved. Change-Id: Id73891b9509918bca3b1dcaf0b34cc32a7a687a7 Test: PinnedTabServiceBrowserTest.* Reviewed-on: https://chromium-review.googlesource.com/892539 Reviewed-by: Scott Violet <sky@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#537442}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .