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

Unified Diff: chrome/browser/ui/views/extensions/bundle_installed_bubble.cc

Issue 10162017: Re-reland Propagate OnNativeWidgetMove to delegate/observers, etc. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync and merge. Created 8 years, 8 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/views/extensions/bundle_installed_bubble.cc
diff --git a/chrome/browser/ui/views/extensions/bundle_installed_bubble.cc b/chrome/browser/ui/views/extensions/bundle_installed_bubble.cc
index 0b2f59859826ce900cb3ca34ebcefc4bc143920a..49f8bf416ece36129714245f091583c51468f0ad 100644
--- a/chrome/browser/ui/views/extensions/bundle_installed_bubble.cc
+++ b/chrome/browser/ui/views/extensions/bundle_installed_bubble.cc
@@ -43,12 +43,9 @@ class BundleInstalledBubble : public views::BubbleDelegateView,
public views::ButtonListener {
public:
BundleInstalledBubble(const BundleInstaller* bundle,
- Browser* browser) {
- BrowserView* browser_view = BrowserView::GetBrowserViewForBrowser(browser);
-
- set_anchor_view(browser_view->GetToolbarView()->app_menu());
- set_arrow_location(views::BubbleBorder::TOP_RIGHT);
-
+ View* anchor_view,
+ views::BubbleBorder::ArrowLocation arrow_location)
+ : views::BubbleDelegateView(anchor_view, arrow_location) {
GridLayout* layout = GridLayout::CreatePanel(this);
SetLayoutManager(layout);
views::ColumnSet* column_set = layout->AddColumnSet(kColumnSetId);
@@ -166,5 +163,7 @@ class BundleInstalledBubble : public views::BubbleDelegateView,
void BundleInstaller::ShowInstalledBubble(
const BundleInstaller* bundle, Browser* browser) {
- new BundleInstalledBubble(bundle, browser);
+ BrowserView* browser_view = BrowserView::GetBrowserViewForBrowser(browser);
+ views::View* anchor = browser_view->GetToolbarView()->app_menu();
+ new BundleInstalledBubble(bundle, anchor, views::BubbleBorder::TOP_RIGHT);
}
« no previous file with comments | « chrome/browser/ui/views/critical_notification_bubble_view.cc ('k') | chrome/browser/ui/views/extensions/extension_popup.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698