Fixed combobox popup positioning after a change in GetViewBounds.

chrome/browser/renderer_host/render_widget_host_view_views.cc

 // Copyright (c) 2011 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 "chrome/browser/renderer_host/render_widget_host_view_views.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 104 matching lines @@
 void RenderWidgetHostViewViews::InitAsPopup(
     RenderWidgetHostView* parent_host_view,
     const gfx::Rect& pos) {
   RenderWidgetHostViewViews* parent =
       static_cast<RenderWidgetHostViewViews*>(parent_host_view);
   parent->AddChildView(this);
   // If the parent loses focus then the popup will close. So we need
   // 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::SetBoundsRect(pos);
+
+  // pos is in screen coordinates but a view is positioned relative
+  // to its parent. Here we convert screen coordinates to relative
+  // coordinates.
+  gfx::Point p(pos.x() - parent_host_view->GetViewBounds().x(),
+               pos.y() - parent_host_view->GetViewBounds().y());
+  views::View::SetBounds(p.x(), p.y(), pos.width(), pos.height());
   Show();
 
   if (NeedsInputGrab()) {
     set_focusable(true);
     RequestFocus();
   }
 }
 
 void RenderWidgetHostViewViews::InitAsFullscreen(
     RenderWidgetHostView* /*reference_host_view*/) {
@@ skipping 667 matching lines @@
 
 void RenderWidgetHostViewViews::FinishImeCompositionSession() {
   if (!has_composition_text_)
     return;
   if (host_)
     host_->ImeConfirmComposition();
   DCHECK(GetInputMethod());
   GetInputMethod()->CancelComposition(this);
   has_composition_text_ = false;
 }