[md] Give each infobar its own layer.

chrome/browser/ui/views/infobars/infobar_container_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/infobars/infobar_container_view.h"
 
 #include "chrome/browser/ui/view_ids.h"
 #include "chrome/browser/ui/views/infobars/infobar_view.h"
 #include "chrome/grit/generated_resources.h"
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/view_targeter.h"
 
 // static
 const char InfoBarContainerView::kViewClassName[] = "InfoBarContainerView";
 
 InfoBarContainerView::InfoBarContainerView(Delegate* delegate)
     : infobars::InfoBarContainer(delegate) {
   set_id(VIEW_ID_INFO_BAR_CONTAINER);
   SetEventTargeter(make_scoped_ptr(new views::ViewTargeter(this)));
-  SetPaintToLayer(true);
-  layer()->SetFillsBoundsOpaquely(false);
 }
 
 InfoBarContainerView::~InfoBarContainerView() {
   RemoveAllInfoBarsForDestruction();
 }
 
 gfx::Size InfoBarContainerView::GetPreferredSize() const {
   int total_height;
   GetVerticalOverlap(&total_height);
   gfx::Size size(0, total_height);
@@ skipping 32 matching lines @@
 
 void InfoBarContainerView::PlatformSpecificRemoveInfoBar(
     infobars::InfoBar* infobar) {
   RemoveChildView(static_cast<InfoBarView*>(infobar));
 }
 
 bool InfoBarContainerView::DoesIntersectRect(const View* target,
                                              const gfx::Rect& rect) const {
   DCHECK_EQ(this, target);
   // Don't handle events on the vertical overlap portion of the view (the
   // vertical space occupied by the arrow).

[Peter Kasting, 2016/03/14 22:37:36]

It would be nice to include some of Terry's explanation in this (and/or on the
base class declaration of this function) and use the same comment here and in
InfoBarView.

[Evan Stade, 2016/03/14 23:12:46]

I've tried to improve the comment (and match between the two impls), but it
seems like any more in-depth explanations or clarifications belong in
ViewTargeterDelegate::DoesIntersectRect

[Peter Kasting, 2016/03/14 23:14:29]

Right, when I said "base class declaration" that's what I meant.  It would be
nice to make the comments on that more clear about what it's used for.

That's optional.

   return rect.bottom() >= GetVerticalOverlap(nullptr);
 }