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

Unified Diff: chrome/browser/ui/browser_dialogs.h

Issue 1570153003: Use MacViews for ContentSettingBubbles. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes for tapted Created 4 years, 11 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
Index: chrome/browser/ui/browser_dialogs.h
diff --git a/chrome/browser/ui/browser_dialogs.h b/chrome/browser/ui/browser_dialogs.h
index 8288ff7805d9c640491b571ea5f7ad331800d70c..85c3982dd47489594769bbcf8aa4ac99720afc6d 100644
--- a/chrome/browser/ui/browser_dialogs.h
+++ b/chrome/browser/ui/browser_dialogs.h
@@ -13,6 +13,7 @@
#include "ui/gfx/native_widget_types.h"
class Browser;
+class ContentSettingBubbleModel;
class GURL;
class LoginHandler;
class Profile;
@@ -116,6 +117,18 @@ void ShowBookmarkEditorViews(gfx::NativeWindow parent_window,
const BookmarkEditor::EditDetails& details,
BookmarkEditor::Configuration configuration);
+// This is a class so that it can be friended from ContentSettingBubbleContents,
+// which allows it to call SetAnchorRect().
+class ContentSettingBubbleViewsBridge {
tapted 2016/01/11 23:21:38 I think this should be up in the #if defined(OS_MA
Elly Fong-Jones 2016/01/12 14:09:59 I think it also needs to be inside the TOOLKIT_VIE
+ public:
+ static void Show(gfx::NativeView parent_view,
+ ContentSettingBubbleModel* model,
+ content::WebContents* web_contents,
+ const gfx::Point& anchor);
+ private:
+ ContentSettingBubbleViewsBridge();
tapted 2016/01/11 23:21:38 DISALLOW_IMPLICIT_CONSTRUCTORS(..)
Elly Fong-Jones 2016/01/12 14:09:59 Done.
+};
+
#endif // TOOLKIT_VIEWS
} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698