Some cleanup in view menu delegate

chrome/views/menu_button.cc

 // Copyright (c) 2006-2008 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 <atlbase.h>
 #include <atlapp.h>
 
 #include "chrome/views/menu_button.h"
 
 #include "chrome/app/theme/theme_resources.h"
@@ skipping 139 matching lines @@
     // We're about to show the menu from a mouse press. By showing from the
     // mouse press event we block RootView in mouse dispatching. This also
     // appears to cause RootView to get a mouse pressed BEFORE the mouse
     // release is seen, which means RootView sends us another mouse press no
     // matter where the user pressed. To force RootView to recalculate the
     // mouse target during the mouse press we explicitly set the mouse handler
     // to NULL.
     GetRootView()->SetMouseHandler(NULL);
 
     menu_visible_ = true;
-    menu_delegate_->RunMenu(this, menu_position.ToPOINT(),
-                            GetWidget()->GetHWND());
+    menu_delegate_->RunMenu(this, menu_position, GetWidget()->GetHWND());
     menu_visible_ = false;
     menu_closed_time_ = Time::Now();
 
     // Now that the menu has closed, we need to manually reset state to
     // "normal" since the menu modal loop will have prevented normal
     // mouse move messages from getting to this View. We set "normal"
     // and not "hot" because the likelihood is that the mouse is now
     // somewhere else (user clicked elsewhere on screen to close the menu
     // or selected an item) and we will inevitably refresh the hot state
     // in the event the mouse _is_ over the view.
@@ skipping 77 matching lines @@
 }
 
 bool MenuButton::GetAccessibleState(VARIANT* state) {
   DCHECK(state);
 
   state->lVal |= STATE_SYSTEM_HASPOPUP;
   return true;
 }
 
 }  // namespace views