Implements part of Device Motion in the Renderer

content/browser/device_orientation/device_orientation_browsertest.cc

 // 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.
 
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
+#include "content/browser/device_orientation/motion.h"
 #include "content/browser/device_orientation/orientation.h"
 #include "content/browser/device_orientation/provider.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_switches.h"
 
 namespace device_orientation {
 
 class MockProvider : public Provider {
  public:
-  explicit MockProvider(const Orientation& orientation)
-      : orientation_(orientation),
-        added_observer_(false),
-        removed_observer_(false) {}
+  explicit MockProvider(const Motion& motion, const Orientation& orientation)

[bulach, 2012/07/02 12:47:56]

nit: "explicit" is no longer needed..

[aousterh, 2012/07/04 13:31:55]

Done.

+      : motion_(motion),
+        orientation_(orientation),
+        added_motion_observer_(false),
+        added_orientation_observer_(false),
+        removed_motion_observer_(false),
+        removed_orientation_observer_(false) {}
 
-  virtual void AddObserver(Observer* observer) {
-    added_observer_ = true;
+  virtual void AddMotionObserver(MotionObserver* observer) {
+    added_motion_observer_ = true;
+    observer->OnMotionUpdate(motion_);
+  }
+  virtual void AddOrientationObserver(OrientationObserver* observer) {
+    added_orientation_observer_ = true;
     observer->OnOrientationUpdate(orientation_);
   }
-  virtual void RemoveObserver(Observer* observer) {
-    removed_observer_ = true;
+  virtual void RemoveMotionObserver(MotionObserver* observer) {
+    removed_motion_observer_ = true;
+  }
+  virtual void RemoveOrientationObserver(OrientationObserver* observer) {
+    removed_orientation_observer_ = true;
   }
 
+  Motion motion_;
   Orientation orientation_;
-  bool added_observer_;
-  bool removed_observer_;
+  bool added_motion_observer_;
+  bool added_orientation_observer_;
+  bool removed_motion_observer_;
+  bool removed_orientation_observer_;
 
  private:
   virtual ~MockProvider() {}
 };
 
 class DeviceOrientationBrowserTest : public InProcessBrowserTest {
  public:
   // From InProcessBrowserTest.
   virtual void SetUpCommandLine(CommandLine* command_line) {
     EXPECT_TRUE(!command_line->HasSwitch(switches::kDisableDeviceOrientation));
   }
 
   GURL testUrl(const FilePath::CharType* filename) {
     const FilePath kTestDir(FILE_PATH_LITERAL("device_orientation"));
     return ui_test_utils::GetTestUrl(kTestDir, FilePath(filename));
   }
 };
 
-// crbug.com/113952
-IN_PROC_BROWSER_TEST_F(DeviceOrientationBrowserTest, BasicTest) {
+IN_PROC_BROWSER_TEST_F(DeviceOrientationBrowserTest, BasicMotionTest) {
+  Motion kTestMotion;

[bulach, 2012/07/02 12:47:56]

nit: "k" is used for constants, in this case just kTestOrientation..
perhaps it's clearer to:

const Orientation kTestOrientation;
Motion test_motion;
test_motion.SetFooBar..();

[aousterh, 2012/07/04 13:31:55]

Done.

+  Orientation kTestOrientation;
+  kTestMotion.setAccelerationX(1);
+  kTestMotion.setAccelerationY(2);
+  kTestMotion.setAccelerationZ(3);
+  kTestMotion.setAccelerationIncludingGravityX(4);
+  kTestMotion.setAccelerationIncludingGravityY(5);
+  kTestMotion.setAccelerationIncludingGravityZ(6);
+  kTestMotion.setRotationRateAlpha(7);
+  kTestMotion.setRotationRateBeta(8);
+  kTestMotion.setRotationRateGamma(9);
+  kTestMotion.setInterval(10);
+  scoped_refptr<MockProvider> provider(
+    new MockProvider(kTestMotion, kTestOrientation));
+  Provider::SetInstanceForTests(provider.get());
+
+  // The test page will register an event handler for motion events,
+  // expects to get an event with kTestMotion motion,
+  // then removes the event handler and navigates to #pass.
+  GURL test_url = testUrl(FILE_PATH_LITERAL("device_motion_test.html"));
+  ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(browser(),
+                                                            test_url,
+                                                            2);
+
+  // Check that the page got the event it expected and that the provider
+  // saw requests for adding and removing an observer.
+  EXPECT_EQ("pass", browser()->GetActiveWebContents()->GetURL().ref());
+  EXPECT_TRUE(provider->added_motion_observer_);
+  EXPECT_TRUE(provider->removed_motion_observer_);
+}
+
+IN_PROC_BROWSER_TEST_F(DeviceOrientationBrowserTest, BasicOrientationTest) {
+  const Motion kMotion;
   const Orientation kTestOrientation(true, 1, true, 2, true, 3, true, true);
-  scoped_refptr<MockProvider> provider(new MockProvider(kTestOrientation));
+  scoped_refptr<MockProvider> provider(
+    new MockProvider(kMotion, kTestOrientation));
   Provider::SetInstanceForTests(provider.get());
 
   // The test page will register an event handler for orientation events,
   // expects to get an event with kTestOrientation orientation,
   // then removes the event handler and navigates to #pass.
   GURL test_url = testUrl(FILE_PATH_LITERAL("device_orientation_test.html"));
   ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(browser(),
                                                             test_url,
                                                             2);
 
   // Check that the page got the event it expected and that the provider
   // saw requests for adding and removing an observer.
   EXPECT_EQ("pass", browser()->GetActiveWebContents()->GetURL().ref());
-  EXPECT_TRUE(provider->added_observer_);
-  EXPECT_TRUE(provider->removed_observer_);
+  EXPECT_TRUE(provider->added_orientation_observer_);
+  EXPECT_TRUE(provider->removed_orientation_observer_);
 }
 
 } //  namespace device_orientation