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

Unified Diff: chrome/browser/views/toolbar_view.cc

Issue 270067: Converts some uses of native_view to native_window. This is necessitated... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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/views/toolbar_view.cc
===================================================================
--- chrome/browser/views/toolbar_view.cc (revision 28706)
+++ chrome/browser/views/toolbar_view.cc (working copy)
@@ -252,14 +252,13 @@
////////////////////////////////////////////////////////////////////////////////
// ToolbarView, views::MenuDelegate implementation:
-void ToolbarView::RunMenu(views::View* source, const gfx::Point& pt,
- gfx::NativeView parent) {
+void ToolbarView::RunMenu(views::View* source, const gfx::Point& pt) {
switch (source->GetID()) {
case VIEW_ID_PAGE_MENU:
- RunPageMenu(pt, parent);
+ RunPageMenu(pt);
break;
case VIEW_ID_APP_MENU:
- RunAppMenu(pt, parent);
+ RunAppMenu(pt);
break;
default:
NOTREACHED() << "Invalid source menu.";
@@ -997,12 +996,12 @@
bookmark_menu_->SetIcon(*tp->GetBitmapNamed(IDR_MENU_BOOKMARK));
}
-void ToolbarView::RunPageMenu(const gfx::Point& pt, gfx::NativeView parent) {
+void ToolbarView::RunPageMenu(const gfx::Point& pt) {
CreatePageMenu();
page_menu_menu_->RunMenuAt(pt, views::Menu2::ALIGN_TOPRIGHT);
}
-void ToolbarView::RunAppMenu(const gfx::Point& pt, gfx::NativeView parent) {
+void ToolbarView::RunAppMenu(const gfx::Point& pt) {
CreateAppMenu();
app_menu_menu_->RunMenuAt(pt, views::Menu2::ALIGN_TOPRIGHT);
}

Powered by Google App Engine
This is Rietveld 408576698