Elide text in the new Autofill UI

chrome/browser/ui/views/autofill/autofill_popup_view_views.cc

 // Copyright (c) 2012 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/ui/views/autofill/autofill_popup_view_views.h"
 
 #include "chrome/browser/ui/autofill/autofill_popup_controller.h"
 #include "grit/ui_resources.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebAutofillClient.h"
 #include "ui/base/keycodes/keyboard_codes.h"
@@ skipping 219 matching lines @@
   int top_of_popup = 0;
   if (GetScreenSize().height() < bottom_of_field + popup_height) {
     // The popup must appear above the field.
     top_of_popup = controller_->element_bounds().y() - popup_height;
   } else {
     // The popup can appear below the field.
     top_of_popup = bottom_of_field;
   }
 
   controller_->SetPopupBounds(gfx::Rect(
-      controller_->element_bounds().x(),
+      controller_->popup_bounds().x(),
       top_of_popup,
       controller_->GetPopupRequiredWidth(),
       popup_height));
 }

[Ilya Sherman, 2013/01/11 23:05:35]

WDYT -- should all of this logic be moved into the controller as well?

[csharp, 2013/01/14 20:32:33]

Done. This was just created because I didn't know a platform independent way to
get the screen bounds before.

 
 gfx::Size AutofillPopupViewViews::GetScreenSize() {
   gfx::Screen* screen =
       gfx::Screen::GetScreenFor(controller_->container_view());
   gfx::Display display =
       screen->GetDisplayNearestPoint(controller_->element_bounds().origin());
 
   return display.GetSizeInPixel();
 }
 
 AutofillPopupView* AutofillPopupView::Create(
     AutofillPopupController* controller) {
   return new AutofillPopupViewViews(controller);
 }