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

Unified Diff: chrome/browser/ui/gtk/bookmarks/bookmark_bar_gtk.cc

Issue 10692164: Remove browser::FindTabbedBrowser call in bookmark_utils.cc. All callers, with the exception of met… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 5 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/gtk/bookmarks/bookmark_bar_gtk.cc
===================================================================
--- chrome/browser/ui/gtk/bookmarks/bookmark_bar_gtk.cc (revision 146039)
+++ chrome/browser/ui/gtk/bookmarks/bookmark_bar_gtk.cc (working copy)
@@ -1150,10 +1150,10 @@
DCHECK(node->is_url());
DCHECK(page_navigator_);
- Profile* profile = browser_->profile();
- RecordAppLaunch(profile, node->url());
- bookmark_utils::OpenAll(window_->GetNativeWindow(), profile, page_navigator_,
- node, event_utils::DispositionForCurrentButtonPressEvent());
+ RecordAppLaunch(browser_->profile(), node->url());
+ bookmark_utils::OpenAll(
+ window_->GetNativeWindow(), page_navigator_, node,
+ event_utils::DispositionForCurrentButtonPressEvent());
content::RecordAction(UserMetricsAction("ClickedBookmarkBarURLButton"));
}
@@ -1237,8 +1237,8 @@
PopupForButton(sender);
} else if (event->button.button == 2) {
const BookmarkNode* node = GetNodeForToolButton(sender);
- bookmark_utils::OpenAll(window_->GetNativeWindow(), browser_->profile(),
- page_navigator_, node, NEW_BACKGROUND_TAB);
+ bookmark_utils::OpenAll(window_->GetNativeWindow(), page_navigator_, node,
+ NEW_BACKGROUND_TAB);
}
}

Powered by Google App Engine
This is Rietveld 408576698