MenuButton: support Mac look & feel

ui/gfx/scoped_canvas.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 UI_GFX_SCOPED_CANVAS_H_
 #define UI_GFX_SCOPED_CANVAS_H_
 
 #include "base/macros.h"
 #include "ui/gfx/canvas.h"
 
 namespace gfx {
 
+class Rect;
+
 // Saves the drawing state, and restores the state when going out of scope.
 class ScopedCanvas {
  public:
   explicit ScopedCanvas(gfx::Canvas* canvas) : canvas_(canvas) {
     if (canvas_)
       canvas_->Save();
   }
   ~ScopedCanvas() {
     if (canvas_)
       canvas_->Restore();
   }
 
  private:
   gfx::Canvas* canvas_;
 
   DISALLOW_COPY_AND_ASSIGN(ScopedCanvas);
 };
 
+// Saves the drawing state, and applies an RTL transform for the supplied rect
+// if the UI is in RTL layout. The resulting transform is such that anything
+// drawn inside the supplied rect will be flipped horizontally.
+class ScopedRTLFlipCanvas {

[tapted, 2016/04/26 05:05:57]

ooh this needs to be GFX_EXPORT (+include gfx_export.h)

[Elly Fong-Jones, 2016/04/26 17:41:25]

I made this and ScopedCanvas both GFX_EXPORT. I'm surprised that this worked
before...

[tapted, 2016/04/27 03:25:11]

The inlined methods wouldn't have required it, but now there's a .cc component
builds will only get the symbol if its exported

+ public:
+  ScopedRTLFlipCanvas(gfx::Canvas* canvas, const gfx::Rect& rect);
+  ~ScopedRTLFlipCanvas() {}
+
+ private:
+  ScopedCanvas canvas_;
+
+  DISALLOW_COPY_AND_ASSIGN(ScopedRTLFlipCanvas);
+};
+
 }  // namespace gfx
 
 #endif  // UI_GFX_SCOPED_CANVAS_H_