Change SetPosition() to SetBounds() in FullscreenExitView::Layout().

The code currently erroneously assumes that SetPosition() will calculate the preferred size and use that for the bounds, which causes all the elements in the popup to be zero size.

Review URL: http://codereview.chromium.org/8310015

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105917 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 16 insertions, 8 deletions

diff --git a/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc b/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc
index 3fcb352..ceebc768 100644
--- a/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc
+++ b/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc
@@ -188,15 +188,23 @@ void FullscreenExitBubbleViews::FullscreenExitView::Layout() {
       link_size.width();
   int link_y = insets.top() + (inner_height - link_size.height()) / 2;
 
-  message_label_.SetPosition(gfx::Point(insets.left() + kPaddingPx,
-                             message_label_y));
-  link_.SetPosition(gfx::Point(link_x, link_y));
+  message_label_.SetBounds(insets.left() + kPaddingPx,
+                           message_label_y,
+                           message_label_size.width(),
+                           message_label_size.height());
+  link_.SetBounds(link_x,
+                  link_y,
+                  link_size.width(),
+                  link_size.height());
   if (show_buttons_) {
-    accept_button_->SetPosition(gfx::Point(button_box_x,
-        insets.top() + kPaddingPx));
-    deny_button_->SetPosition(gfx::Point(
-        button_box_x + accept_size.width() + kPaddingPx,
-        insets.top() + kPaddingPx));
+    accept_button_->SetBounds(button_box_x,
+                              insets.top() + kPaddingPx,
+                              accept_size.width(),
+                              accept_size.height());
+    deny_button_->SetBounds(button_box_x + accept_size.width() + kPaddingPx,
+                            insets.top() + kPaddingPx,
+                            deny_size.width(),
+                            deny_size.height());
   }
 }