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

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

Issue 795053003: [Password Manager] Close the bubble when fullscreen state gets change. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addresses Peter's review comments. Created 6 years 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
Index: chrome/browser/ui/views/managed_full_screen_bubble_delegate_view.cc
diff --git a/chrome/browser/ui/views/managed_full_screen_bubble_delegate_view.cc b/chrome/browser/ui/views/managed_full_screen_bubble_delegate_view.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4642cb38e9e21e9ca1452353f2bc1b49fe167ead
--- /dev/null
+++ b/chrome/browser/ui/views/managed_full_screen_bubble_delegate_view.cc
@@ -0,0 +1,58 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/views/managed_full_screen_bubble_delegate_view.h"
+
+#include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
+#include "content/public/browser/notification_source.h"
+#include "ui/gfx/geometry/rect.h"
+
+ManagedFullScreenBubbleDelegateView::ManagedFullScreenBubbleDelegateView(
+ views::View* anchor_view,
+ content::WebContents* web_contents)
+ : BubbleDelegateView(
+ anchor_view,
+ anchor_view ?
+ views::BubbleBorder::TOP_RIGHT : views::BubbleBorder::NONE) {
+ // Add observer to close the bubble if the fullscreen state changes.
+ if (web_contents) {
+ Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
+ registrar_.Add(this, chrome::NOTIFICATION_FULLSCREEN_CHANGED,
+ content::Source<FullscreenController>(
+ browser->fullscreen_controller()));
+ }
+}
+
+ManagedFullScreenBubbleDelegateView::~ManagedFullScreenBubbleDelegateView() {
+}
+
+void ManagedFullScreenBubbleDelegateView::Observe(
+ int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) {
+ DCHECK_EQ(chrome::NOTIFICATION_FULLSCREEN_CHANGED, type);
+ GetWidget()->SetVisibilityAnimationTransition(views::Widget::ANIMATE_NONE);
+ Close();
+}
+
+void ManagedFullScreenBubbleDelegateView::Close() {
+ GetWidget()->Close();
+}
+
+void ManagedFullScreenBubbleDelegateView::AdjustForFullscreen(
+ const gfx::Rect& screen_bounds) {
+ if (GetAnchorView())
+ return;
+
+ const int kBubblePaddingFromScreenEdge = 20;
+ int bubble_half_width = width() / 2;
+ const int x_pos = base::i18n::IsRTL() ?
+ (screen_bounds.x() + bubble_half_width + kBubblePaddingFromScreenEdge) :
+ (screen_bounds.right() - bubble_half_width -
+ kBubblePaddingFromScreenEdge);
+ SetAnchorRect(gfx::Rect(x_pos, screen_bounds.y(), 0, 0));
+}

Powered by Google App Engine
This is Rietveld 408576698