summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorfsamuel@chromium.org <fsamuel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-07-28 18:19:21 +0000
committerfsamuel@chromium.org <fsamuel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-07-28 18:19:21 +0000
commit360f17f364b9a4b94ee1a49eca844fa2c8fd2e6e (patch)
tree1b332eec3c67945be78c6b6bb189591448335d2f
parent4b39ec1129680f363db28c87d0045b1e728c4b26 (diff)
downloadchromium_src-360f17f364b9a4b94ee1a49eca844fa2c8fd2e6e.zip
chromium_src-360f17f364b9a4b94ee1a49eca844fa2c8fd2e6e.tar.gz
chromium_src-360f17f364b9a4b94ee1a49eca844fa2c8fd2e6e.tar.bz2
Removed a hack as WebKit seems to handle combo box positioning correctly now.
BUG=none TEST=manually Review URL: http://codereview.chromium.org/7522021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94503 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--chrome/browser/renderer_host/render_widget_host_view_views.cc9
1 files changed, 1 insertions, 8 deletions
diff --git a/chrome/browser/renderer_host/render_widget_host_view_views.cc b/chrome/browser/renderer_host/render_widget_host_view_views.cc
index a9b036f..5873a0e 100644
--- a/chrome/browser/renderer_host/render_widget_host_view_views.cc
+++ b/chrome/browser/renderer_host/render_widget_host_view_views.cc
@@ -109,14 +109,7 @@ void RenderWidgetHostViewViews::InitAsPopup(
// to tell the parent it's showing a popup so that it doesn't respond to
// blurs.
parent->is_showing_context_menu_ = true;
- views::View* root_view = GetWidget()->GetRootView();
- // TODO(fsamuel): WebKit is computing the screen coordinates incorrectly.
- // Fixing this is a long and involved process, because WebKit needs to know
- // how to direct an IPC at a particular View. For now, we simply convert
- // the broken screen coordinates into relative coordinates.
- gfx::Point p(pos.x() - root_view->GetScreenBounds().x(),
- pos.y() - root_view->GetScreenBounds().y());
- views::View::SetBounds(p.x(), p.y(), pos.width(), pos.height());
+ views::View::SetBoundsRect(pos);
Show();
if (NeedsInputGrab()) {