diff options
| author | rsimha@chromium.org <rsimha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-28 23:25:11 +0000 |
|---|---|---|
| committer | rsimha@chromium.org <rsimha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-28 23:25:11 +0000 |
| commit | fe71ee35c16c81e2bc3fdd7e82298c2122fa196b (patch) | |
| tree | 3c43bbd530cdeaba64d8b643eb1c5a995601a576 /native_client_sdk | |
| parent | 18d2667665811a055d55aeededc9a6e4cb91cbc0 (diff) | |
| download | chromium_src-fe71ee35c16c81e2bc3fdd7e82298c2122fa196b.zip chromium_src-fe71ee35c16c81e2bc3fdd7e82298c2122fa196b.tar.gz chromium_src-fe71ee35c16c81e2bc3fdd7e82298c2122fa196b.tar.bz2 | |
[sync] Prevent SyncSetupOverlay.closeOverlay() from closing non-sync overlays
The chrome webui settings page contains code to hide the sync settings
overlay if sync is disabled or if the user is signed out in the
background while the overlay is visible. It does so by calling the
SyncSetupOverlay.CloseOverlay() function.
It turns out that if SyncSetupOverlay.closeOverlay() is called when
the sync settings overlay is not visible, it will close whatever overlay
is visible (in case of bug 244141, the clear-browsing-data overlay).
This patch modifies SyncSetupOverlay.CloseOverlay() to only close the
sync setup overlay if it is visible, and not any other overlay.
BUG=244141
TEST=All non-sync settings UI overlays must stay visible when the user is disconnected or when sync is disabled.
R=zea@chromium.org
Review URL: https://codereview.chromium.org/15740030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202687 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'native_client_sdk')
0 files changed, 0 insertions, 0 deletions