Linux Aura Task Manager Frame Buttons Misaligned

ui/views/window/custom_frame_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 UI_VIEWS_WINDOW_CUSTOM_FRAME_VIEW_H_
 #define UI_VIEWS_WINDOW_CUSTOM_FRAME_VIEW_H_
 

[flackr, 2014/04/22 18:36:45]

#include <vector>

[jonross, 2014/04/23 18:53:53]

Done.

 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "ui/views/controls/button/button.h"
+#include "ui/views/linux_ui/window_button_order_observer.h"

[flackr, 2014/04/22 18:36:45]

Including ui/views/linux_ui/ from all views platforms feels wrong, should this
be guarded by a macro? i.e.
#if defined(OS_LINUX) && !defined(OS_CHROMEOS)

[jonross, 2014/04/23 18:53:53]

Done.

+#include "ui/views/window/frame_buttons.h"
 #include "ui/views/window/non_client_view.h"
 
 namespace gfx {
 class ImageSkia;
 }
 
 namespace views {
 
 class FrameBackground;
 class ImageButton;
 class Widget;
 
 ///////////////////////////////////////////////////////////////////////////////
 //
 // CustomFrameView
 //
 //  A view that provides the non client frame for Windows. This means
 //  rendering the non-standard window caption, border, and controls.
 //
 ////////////////////////////////////////////////////////////////////////////////
-class CustomFrameView : public NonClientFrameView,
-                        public ButtonListener {
+class VIEWS_EXPORT CustomFrameView : public NonClientFrameView,
+                        public ButtonListener,
+                        public WindowButtonOrderObserver {

[flackr, 2014/04/22 18:36:45]

nit: align public, which probably means wrapping the ':'

[jonross, 2014/04/23 18:53:53]

Done.

  public:
   CustomFrameView();
   virtual ~CustomFrameView();
 
   void Init(Widget* frame);
 
-  // Overridden from NonClientFrameView:
+  // NonClientFrameView:
   virtual gfx::Rect GetBoundsForClientView() const OVERRIDE;
   virtual gfx::Rect GetWindowBoundsForClientBounds(
       const gfx::Rect& client_bounds) const OVERRIDE;
   virtual int NonClientHitTest(const gfx::Point& point) OVERRIDE;
   virtual void GetWindowMask(const gfx::Size& size,
                              gfx::Path* window_mask) OVERRIDE;
   virtual void ResetWindowControls() OVERRIDE;
   virtual void UpdateWindowIcon() OVERRIDE;
   virtual void UpdateWindowTitle() OVERRIDE;
 
-  // Overridden from View:
+  // View:
   virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
   virtual void Layout() OVERRIDE;
   virtual gfx::Size GetPreferredSize() OVERRIDE;
   virtual gfx::Size GetMinimumSize() OVERRIDE;
   virtual gfx::Size GetMaximumSize() OVERRIDE;
 
-  // Overridden from ButtonListener:
+  // ButtonListener:
   virtual void ButtonPressed(Button* sender, const ui::Event& event) OVERRIDE;
 
+  // WindowButtonOrderObserver:
+  virtual void OnWindowButtonOrderingChange(
+      const std::vector<views::FrameButton>& leading_buttons,
+      const std::vector<views::FrameButton>& trailing_buttons) OVERRIDE;
+
  private:
+  friend class CustomFrameViewTest;
+
   // Returns the thickness of the border that makes up the window frame edges.
   // This does not include any client edge.
   int FrameBorderThickness() const;
 
   // Returns the thickness of the entire nonclient left, right, and bottom
   // borders, including both the window frame and any client edge.
   int NonClientBorderThickness() const;
 
   // Returns the height of the entire nonclient top border, including the window
   // frame, any title area, and any connected client edge.
@@ skipping 26 matching lines @@
   void PaintRestoredFrameBorder(gfx::Canvas* canvas);
   void PaintMaximizedFrameBorder(gfx::Canvas* canvas);
   void PaintTitleBar(gfx::Canvas* canvas);
   void PaintRestoredClientEdge(gfx::Canvas* canvas);
 
   // Compute aspects of the frame needed to paint the frame background.
   SkColor GetFrameColor() const;
   const gfx::ImageSkia* GetFrameImage() const;
 
   // Layout various sub-components of this view.
+  void LayoutButton(ImageButton* button, int x, int y, bool leading);

[flackr, 2014/04/22 18:36:45]

I think it would be worth documenting this functionn as it's not obvious what
all of these args should be. I.e. are x and y used directly or suggestions, what
does leading mean?

[jonross, 2014/04/23 18:53:53]

Removed the need for leading. Added comments.

   void LayoutWindowControls();
   void LayoutTitleBar();
   void LayoutClientView();
 
   // Creates, adds and returns a new window caption button (e.g, minimize,
   // maximize, restore).
   ImageButton* InitWindowCaptionButton(int accessibility_string_id,
                                        int normal_image_id,
                                        int hot_image_id,
                                        int pushed_image_id);
@@ skipping 15 matching lines @@
   ImageButton* maximize_button_;
   ImageButton* restore_button_;
   ImageButton* close_button_;
 
   // Should maximize button be shown?
   bool should_show_maximize_button_;
 
   // Background painter for the window frame.
   scoped_ptr<FrameBackground> frame_background_;
 
+  // Layout arrangement of the window caption buttons. On linux these will be
+  // set via OnWindowButtonOrderingChange. On other platforms a default
+  // arrangement of a trailing minimize, maximize, close, will be set.
+  std::vector<views::FrameButton> leading_buttons_;
+  std::vector<views::FrameButton> trailing_buttons_;
+
+  // The horizontal boundaries for the title bar to layout within. Restricted
+  // by the space used by the leading and trailing buttons.
+  int minimum_title_bar_x_;
+  int maximum_title_bar_x_;
+
   DISALLOW_COPY_AND_ASSIGN(CustomFrameView);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_WINDOW_CUSTOM_FRAME_VIEW_H_