[linux_views][Win] spoof proof redesign infobar extension with tab.

chrome/browser/ui/views/frame/browser_view.cc

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index 894a4a58c4c8a8df184ec3ccb723b0121314f659..651d66f6eee3a5516fa829fa88f54047c4e863ed 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -1805,11 +1805,16 @@ void BrowserView::Layout() {
 }
 
 void BrowserView::PaintChildren(gfx::Canvas* canvas) {
-  views::ClientView::PaintChildren(canvas);
+  // Paint the |infobar_container_| last so that it may paint its
+  // overlapping tabs.
+  for (int i = 0; i < child_count(); ++i) {
+    View* child = GetChildViewAt(i);
+    DCHECK(child) << "Should not have a NULL child View for index in bounds";

[Peter Kasting, 2011/03/07 20:14:30]

Nit: I would remove this DCHECK().  There are hundreds of places in code we
iterate through View children, and AFAIK none DCHECK the pointers.

[Sheridan Rawlins, 2011/03/08 01:38:19]

Done.

+    if (child != infobar_container_)
+      child->Paint(canvas);
+  }
 
-  infobar_container_->PaintInfoBarArrows(canvas->AsCanvasSkia(),
-                                         this,
-                                         GetInfoBarArrowCenterX());
+  infobar_container_->Paint(canvas);
 }
 
 void BrowserView::ViewHierarchyChanged(bool is_add,