Unit test for BrowserViewLayout that does not depend on BrowserView

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

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 
 #include <map>
 #include <string>
 #include <vector>
@@ skipping 442 matching lines @@
   views::SingleSplitView* GetContentsSplitForTest() { return contents_split_; }
   ContentsContainer* GetContentsContainerForTest() {
     return contents_container_;
   }
   OverlayContainer* GetOverlayContainerForTest() {
     return overlay_container_;
   }
   views::WebView* GetContentsWebViewForTest() { return contents_web_view_; }
 
  private:
-  friend class BrowserViewLayout;
+  // Do not friend BrowserViewLayout. Use the BrowserViewLayoutDelegate
+  // interface to keep these two classes decoupled and testable.
+  friend class BrowserViewLayoutDelegateImpl;
   FRIEND_TEST_ALL_PREFIXES(BrowserViewTest, BrowserView);
   FRIEND_TEST_ALL_PREFIXES(BrowserViewsAccessibilityTest,
                            TestAboutChromeViewAccObj);
 
   enum FullscreenType {
     FOR_DESKTOP,
     FOR_METRO
   };
 
   // We store this on linux because we must call ProcessFullscreen()
@@ skipping 301 matching lines @@
   gfx::ScopedSysColorChangeListener color_change_listener_;
 
   scoped_ptr<InstantOverlayControllerViews> overlay_controller_;
 
   mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_