Cleanup: Remove unused arguments, member variable

ash/common/wm/overview/scoped_transform_overview_window.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_COMMON_WM_OVERVIEW_SCOPED_TRANSFORM_OVERVIEW_WINDOW_H_
 #define ASH_COMMON_WM_OVERVIEW_SCOPED_TRANSFORM_OVERVIEW_WINDOW_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 43 matching lines @@
                                                         int top_view_inset,
                                                         int title_height);
 
   // Returns the transform turning |src_rect| into |dst_rect|.
   static gfx::Transform GetTransformForRect(const gfx::Rect& src_rect,
                                             const gfx::Rect& dst_rect);
 
   explicit ScopedTransformOverviewWindow(WmWindow* window);
   ~ScopedTransformOverviewWindow();
 
-  gfx::Transform get_overview_transform() const { return overview_transform_; }
-
-  void set_overview_transform(const gfx::Transform& transform) {
-    overview_transform_ = transform;
-  }
-
   // Starts an animation sequence which will use animation settings specified by
   // |animation_type|. The |animation_settings| container is populated with
   // scoped entities and the container should be destroyed at the end of the
   // animation sequence.
   //
   // Example:
   //  ScopedTransformOverviewWindow overview_window(window);
   //  ScopedTransformOverviewWindow::ScopedAnimationSettings scoped_settings;
   //  overview_window.BeginScopedAnimation(
   //      OverviewAnimationType::OVERVIEW_ANIMATION_SELECTOR_ITEM_SCROLL_CANCEL,
@@ skipping 40 matching lines @@
   // Prepares for overview mode by doing any necessary actions before entering.
   void PrepareForOverview();
 
   // Applies the |transform| to the overview window and all of its transient
   // children.
   void SetTransform(WmWindow* root_window, const gfx::Transform& transform);
 
   // Set's the opacity of the managed windows.
   void SetOpacity(float opacity);
 
-  // Creates a mask layer with the bottom edge using rounded corners of
-  // |radius|. Uses SetAlphaShape to mask the header.
-  void HideHeaderAndSetShape(int radius);
+  // Hides the window header whose size is given in

[varkha, 2016/10/21 14:17:15]

nit: Incomplete sentence?

[oshima, 2016/10/24 20:45:14]

Done.

+  void HideHeader();
 
-  // Restores original window shape and removes the mask if installed. Safe to
-  // call even if HideHeaderAndSetShape() has not been called.
-  void ShowHeaderAndResetShape();
+  // Shows the window header that is hidden by HideHeader();

[varkha, 2016/10/21 14:17:15]

s/;/.

[oshima, 2016/10/24 20:45:14]

Done.

+  void ShowHeader();
 
   WmWindow* window() const { return window_; }
 
   // Closes the transient root of the window managed by |this|.
   void Close();
 
  private:
   friend class WindowSelectorTest;
 
   enum OriginalVisibility {
@@ skipping 29 matching lines @@
   bool ignored_by_shelf_;
 
   // True if the window has been transformed for overview mode.
   bool overview_started_;
 
   // The original transform of the window before entering overview mode.
   gfx::Transform original_transform_;
 
   // Keeps track of the original transform used when |this| has been positioned
   // during SelectorItem layout.
   gfx::Transform overview_transform_;

[varkha, 2016/10/21 14:19:55]

nit: Did you mean to remove this?

[oshima, 2016/10/24 20:45:14]

Yes. Done.

 
   // The original opacity of the window before entering overview mode.
   float original_opacity_;
 
   base::WeakPtrFactory<ScopedTransformOverviewWindow> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ScopedTransformOverviewWindow);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_WM_OVERVIEW_SCOPED_TRANSFORM_OVERVIEW_WINDOW_H_