[Refactor] Cache the views::Widget* that the FrameMaximizeButton acts on

ash/wm/caption_buttons/frame_caption_button_container_view.h

 // Copyright 2013 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 ASH_WM_CAPTION_BUTTONS_FRAME_CAPTION_BUTTON_CONTAINER_VIEW_H_
 #define ASH_WM_CAPTION_BUTTONS_FRAME_CAPTION_BUTTON_CONTAINER_VIEW_H_
 
 #include "ash/ash_export.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/view.h"
 
 namespace views {
 class CustomButton;
-class NonClientFrameView;
 class Widget;
 }
 
 namespace ash {
 
 // Container view for the frame caption buttons. It performs the appropriate
 // action when a caption button is clicked.
 class ASH_EXPORT FrameCaptionButtonContainerView
     : public views::View,
       public views::ButtonListener {
@@ skipping 12 matching lines @@
     HEADER_STYLE_SHORT,
 
     // Restored tabbed browser windows, popups for browser windows, restored
     // hosted app windows, popups for hosted app windows.
     HEADER_STYLE_TALL,
 
     // AppNonClientFrameViewAsh.
     HEADER_STYLE_MAXIMIZED_HOSTED_APP
   };
 
   // |frame_view| and |frame| are the NonClientFrameView and the views::Widget

[James Cook, 2013/09/16 16:08:07]

nit: update comment

   // that the caption buttons act on.
   // |minimize_allowed| indicates whether the frame can be minimized (either via
   // the maximize/restore button or via a dedicated button).
-  // TODO(pkotwicz): Remove the |frame_view| parameter once FrameMaximizeButton
-  // is refactored to take in a views::Widget instead.
-  FrameCaptionButtonContainerView(views::NonClientFrameView* frame_view,
-                                  views::Widget* frame,
+  FrameCaptionButtonContainerView(views::Widget* frame,
                                   MinimizeAllowed minimize_allowed);
   virtual ~FrameCaptionButtonContainerView();
 
   // For testing.
   class TestApi {
    public:
     explicit TestApi(FrameCaptionButtonContainerView* container_view)
         : container_view_(container_view) {
     }
 
@@ skipping 74 matching lines @@
   views::CustomButton* minimize_button_;
   views::CustomButton* size_button_;
   views::CustomButton* close_button_;
 
   DISALLOW_COPY_AND_ASSIGN(FrameCaptionButtonContainerView);
 };
 
 }  // namesapace ash
 
 #endif  // ASH_WM_CAPTION_BUTTONS_FRAME_CAPTION_BUTTON_CONTAINER_VIEW_H_