Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(322)

Unified Diff: chrome/browser/ui/views/exclusive_access_bubble_views.cc

Issue 1254543002: Change exclusive access popup behaviour with simplified-fullscreen-ui. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@exclusiveaccess-remove-confirmation
Patch Set: Respond to nits. Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/exclusive_access_bubble_views.cc
diff --git a/chrome/browser/ui/views/exclusive_access_bubble_views.cc b/chrome/browser/ui/views/exclusive_access_bubble_views.cc
index 466355d0ae325ae475b08fd75437040a6794b350..de3eb3ba78bf600ac1013bb4f6029f0016e42ae9 100644
--- a/chrome/browser/ui/views/exclusive_access_bubble_views.cc
+++ b/chrome/browser/ui/views/exclusive_access_bubble_views.cc
@@ -266,7 +266,11 @@ ExclusiveAccessBubbleViews::ExclusiveAccessBubbleViews(
popup_(nullptr),
animation_(new gfx::SlideAnimation(this)),
animated_attribute_(ANIMATED_ATTRIBUTE_BOUNDS) {
- animation_->Reset(1);
+ // With the simplified fullscreen UI flag, initially hide the bubble;
+ // otherwise, initially show it.
+ double initial_value =
+ ExclusiveAccessManager::IsSimplifiedFullscreenUIEnabled() ? 0 : 1;
+ animation_->Reset(initial_value);
// Create the contents view.
ui::Accelerator accelerator(ui::VKEY_UNKNOWN, ui::EF_NONE);
@@ -297,7 +301,8 @@ ExclusiveAccessBubbleViews::ExclusiveAccessBubbleViews(
// that it is sliding off the top of the screen.
popup_->GetRootView()->SetLayoutManager(nullptr);
view_->SetBounds(0, 0, size.width(), size.height());
- popup_->ShowInactive(); // This does not activate the popup.
+ if (!ExclusiveAccessManager::IsSimplifiedFullscreenUIEnabled())
+ popup_->ShowInactive(); // This does not activate the popup.
popup_->AddObserver(this);
« no previous file with comments | « chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698