AutocompletePopupContentsView must override View::PaintChildren to prevent views

chrome/browser/ui/views/autocomplete/touch_autocomplete_popup_contents_view.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 "touch_autocomplete_popup_contents_view.h"
 
 #include "chrome/browser/autocomplete/autocomplete_edit_view.h"
 #include "chrome/browser/profiles/profile.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/canvas_skia.h"
@@ skipping 66 matching lines @@
 }
 
 TouchAutocompletePopupContentsView::~TouchAutocompletePopupContentsView() {
 }
 
 void TouchAutocompletePopupContentsView::UpdatePopupAppearance() {
   AutocompletePopupContentsView::UpdatePopupAppearance();
   Layout();
 }
 
-void TouchAutocompletePopupContentsView::PaintChildren(
+void TouchAutocompletePopupContentsView::LayoutChildren() {
+  std::vector<View*> visible_children(GetVisibleChildren());
+  gfx::Rect bounds(GetContentsBounds());
+  double child_width =
+      static_cast<double>(bounds.width()) / visible_children.size();
+  int x = bounds.x();
+  for (size_t i = 0; i < visible_children.size(); ++i) {
+    int next_x = bounds.x() + static_cast<int>(((i + 1) * child_width) + 0.5);
+    visible_children[i]->SetBounds(x, bounds.y(), next_x - x, bounds.height());
+    x = next_x;
+  }
+}
+
+void TouchAutocompletePopupContentsView::PaintResultViews(
     gfx::CanvasSkia* canvas) {
-  AutocompletePopupContentsView::PaintChildren(canvas);
+  AutocompletePopupContentsView::PaintResultViews(canvas);
 
   // Draw divider lines.
   std::vector<View*> visible_children(GetVisibleChildren());
   if (visible_children.size() < 2)
     return;
   SkColor color = AutocompleteResultView::GetColor(
       AutocompleteResultView::NORMAL, AutocompleteResultView::DIMMED_TEXT);
   gfx::Rect bounds(GetContentsBounds());
   for (std::vector<View*>::const_iterator i(visible_children.begin() + 1);
        i != visible_children.end(); ++i) {
     canvas->DrawLineInt(color, (*i)->x(), bounds.y(), (*i)->x(),
                         bounds.bottom());
   }
 }
 
-void TouchAutocompletePopupContentsView::LayoutChildren() {
-  std::vector<View*> visible_children(GetVisibleChildren());
-  gfx::Rect bounds(GetContentsBounds());
-  double child_width =
-      static_cast<double>(bounds.width()) / visible_children.size();
-  int x = bounds.x();
-  for (size_t i = 0; i < visible_children.size(); ++i) {
-    int next_x = bounds.x() + static_cast<int>(((i + 1) * child_width) + 0.5);
-    visible_children[i]->SetBounds(x, bounds.y(), next_x - x, bounds.height());
-    x = next_x;
-  }
-}
-
 int TouchAutocompletePopupContentsView::CalculatePopupHeight() {
   DCHECK_GE(static_cast<size_t>(child_count()), model_->result().size());
   int popup_height = 0;
   for (size_t i = 0; i < model_->result().size(); ++i) {
     popup_height = std::max(popup_height,
                             GetChildViewAt(i)->GetPreferredSize().height());
   }
   popup_height = std::max(popup_height, opt_in_view_ ?
       opt_in_view_->GetPreferredSize().height() : 0);
   return popup_height;
@@ skipping 10 matching lines @@
 std::vector<views::View*>
     TouchAutocompletePopupContentsView::GetVisibleChildren() {
   std::vector<View*> visible_children;
   for (int i = 0; i < child_count(); ++i) {
     View* v = GetChildViewAt(i);
     if (GetChildViewAt(i)->IsVisible())
       visible_children.push_back(v);
   }
   return visible_children;
 }