Animated Rotation

chrome/browser/ui/touch/frame/touch_browser_frame_view.h

Index: chrome/browser/ui/touch/frame/touch_browser_frame_view.h
diff --git a/chrome/browser/ui/touch/frame/touch_browser_frame_view.h b/chrome/browser/ui/touch/frame/touch_browser_frame_view.h
index 4593d70b1800d3abf54fe784380ad18c7ccd0c50..22a85398ee36c15af15aedf651f7425a95220f53 100644
--- a/chrome/browser/ui/touch/frame/touch_browser_frame_view.h
+++ b/chrome/browser/ui/touch/frame/touch_browser_frame_view.h
@@ -8,8 +8,10 @@
 
 #include "chrome/browser/tabs/tab_strip_model_observer.h"
 #include "chrome/browser/ui/views/frame/opaque_browser_frame_view.h"
+#include "content/browser/sensors/sensors_provider.h"
 #include "content/common/notification_observer.h"
 #include "content/common/notification_registrar.h"
+#include "content/common/sensors.h"
 #include "googleurl/src/gurl.h"
 #include "ui/base/animation/animation_delegate.h"
 #include "views/focus/focus_manager.h"
@@ -28,6 +30,10 @@ namespace ui {
 class SlideAnimation;
 }
 
+namespace views {
+class LayerPropertySetter;
+}
+
 class TouchBrowserFrameView
     : public OpaqueBrowserFrameView,
       public views::FocusChangeListener,
@@ -36,7 +42,8 @@ class TouchBrowserFrameView
       public
       chromeos::input_method::InputMethodManager::VirtualKeyboardObserver,
 #endif
-      public ui::AnimationDelegate {
+      public ui::AnimationDelegate,
+      public sensors::Listener {
  public:
   enum VirtualKeyboardType {
     NONE,
@@ -70,6 +77,10 @@ class TouchBrowserFrameView
       const std::string& virtual_keyboard_layout);
 #endif
 
+  // sensors::Listener implementation
+  virtual void OnScreenOrientationChanged(
+      const sensors::ScreenOrientation& change) OVERRIDE;
+
  protected:
   // Overridden from OpaqueBrowserFrameView
   virtual int GetReservedHeight() const;
@@ -105,6 +116,7 @@ class TouchBrowserFrameView
   KeyboardContainerView* keyboard_;
   NotificationRegistrar registrar_;
   GURL url_;
+  views::LayerPropertySetter* setter_;

[sky, 2011/08/25 21:10:12]

Initialize this. In fact maybe this should just be a boolean indicating whether
you installed the setter and not the actual setter. Maybe you could install it
in the constructor though and not have to explicitly set it at rotate time. Even
better that way.

 
   scoped_ptr<ui::SlideAnimation> animation_;