Do all OOLing in the views code. linux_views now builds clean with the clang plugin.

chrome/browser/ui/views/autocomplete/autocomplete_result_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 "chrome/browser/ui/views/autocomplete/autocomplete_result_view.h"
 
 #include "base/i18n/bidi_line_iterator.h"
 #include "chrome/browser/ui/views/autocomplete/autocomplete_result_view_model.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "grit/generated_resources.h"
@@ skipping 16 matching lines @@
 // The minimum distance between the top and bottom of the {icon|text} and the
 // top or bottom of the row.
 const int kMinimumIconVerticalPadding = 2;
 const int kMinimumTextVerticalPadding = 3;
 
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // AutocompleteResultView, public:
 
+// Precalculated data used to draw the portion of a match classification that
+// fits entirely within one run.
+struct AutocompleteResultView::ClassificationData {
+  string16 text;
+  const gfx::Font* font;
+  SkColor color;
+  int pixel_width;
+};
+
+// Precalculated data used to draw a complete visual run within the match.
+// This will include all or part of at leasdt one, and possibly several,
+// classifications.
+struct AutocompleteResultView::RunData {
+  size_t run_start;  // Offset within the match text where this run begins.
+  int visual_order;  // Where this run occurs in visual order.  The earliest
+  // run drawn is run 0.
+  bool is_rtl;
+  int pixel_width;
+  Classifications classifications;  // Classification pieces within this run,
+  // in logical order.

[Nico, 2011/03/04 22:53:02]

align with // on prev line

+};
+
 // This class is a utility class for calculations affected by whether the result
 // view is horizontally mirrored.  The drawing functions can be written as if
 // all drawing occurs left-to-right, and then use this class to get the actual
 // coordinates to begin drawing onscreen.
 class AutocompleteResultView::MirroringContext {
  public:
   MirroringContext() : center_(0), right_(0) {}
 
   // Tells the mirroring context to use the provided range as the physical
   // bounds of the drawing region.  When coordinate mirroring is needed, the
@@ skipping 420 matching lines @@
       //   * It's normal, and will be able to draw successfully with the
       //     ellipsis we'll append to it, or
       //   * It is also bold, in which case we don't want to fall back
       //     to a normal ellipsis anyway (see comment above).
     }
   }
 
   // We couldn't draw anything.
   runs->clear();
 }