Implement the Blink part of the Device Motion API.

Source/WebKit/chromium/src/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 20 matching lines @@
 #include "config.h"
 #include "WebViewImpl.h"
 
 #include "AutofillPopupMenuClient.h"
 #include "BatteryClientImpl.h"
 #include "BatteryController.h"
 #include "CSSValueKeywords.h"
 #include "CompositionUnderlineVectorBuilder.h"
 #include "ContextFeaturesClientImpl.h"
 #include "DeviceOrientationClientProxy.h"
+#include "modules/device_orientation/DeviceMotionController.h"

[abarth-chromium, 2013/04/29 20:32:20]

Please put this include in alphabetical order.

[timvolodine, 2013/04/30 12:40:18]

Done.

 #include "GeolocationClientProxy.h"
 #include "GeolocationController.h"
 #include "GraphicsLayerFactoryChromium.h"
 #include "HTMLInputElement.h"
 #include "HTMLMediaElement.h"
 #include "HTMLNames.h"
 #include "HTMLTextAreaElement.h"
 #include "InspectorController.h"
 #include "InspectorInstrumentation.h"
 #include "LinkHighlight.h"
@@ skipping 423 matching lines @@
 #endif
     provideSpeechRecognitionTo(m_page.get(), m_speechRecognitionClient.get());
 #if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
     provideNotification(m_page.get(), notificationPresenterImpl());
 #endif
 #if ENABLE(NAVIGATOR_CONTENT_UTILS)
     provideNavigatorContentUtilsTo(m_page.get(), m_navigatorContentUtilsClient.get());
 #endif
 
     provideContextFeaturesTo(m_page.get(), m_featureSwitchClient.get());
+    provideDeviceMotionTo(m_page.get());

[abarth-chromium, 2013/04/29 20:32:20]

This shouldn't be needed.  This sort of thing is only needed when you need to
wire up a client, but there's not client here.

(Really most of these features that use clients should be refactored to not use
clients also.)

[timvolodine, 2013/04/30 12:40:18]

DeviceMotionController is still a supplement of a page because we need to know
the page it is active at for caching purposes (PageCache.cpp). Otherwise we will
likely need to change the interface of DOMWindow.

     provideDeviceOrientationTo(m_page.get(), m_deviceOrientationClientProxy.get());
     provideGeolocationTo(m_page.get(), m_geolocationClientProxy.get());
     m_geolocationClientProxy->setController(GeolocationController::from(m_page.get()));
 
 #if ENABLE(BATTERY_STATUS)
     provideBatteryTo(m_page.get(), m_batteryClient.get());
     m_batteryClient->setController(BatteryController::from(m_page.get()));
 #endif
 
     m_page->setGroupType(Page::SharedPageGroup);
@@ skipping 3719 matching lines @@
 }
 
 bool WebViewImpl::shouldDisableDesktopWorkarounds()
 {
     ViewportArguments arguments = mainFrameImpl()->frame()->document()->viewportArguments();
     return arguments.width == ViewportArguments::ValueDeviceWidth || !arguments.userZoom
         || (arguments.minZoom == arguments.maxZoom && arguments.minZoom != ViewportArguments::ValueAuto);
 }
 
 } // namespace WebKit