Fix measurement of DeviceMotion and DeviceOrientation on secure origins.

Source/modules/device_orientation/DeviceMotionController.cpp

 // Copyright 2014 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 "config.h"
 #include "modules/device_orientation/DeviceMotionController.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/Settings.h"
 #include "core/frame/UseCounter.h"
@@ skipping 27 matching lines @@
     DeviceMotionController* controller = static_cast<DeviceMotionController*>(DocumentSupplement::from(document, supplementName()));
     if (!controller) {
         controller = new DeviceMotionController(document);
         DocumentSupplement::provideTo(document, supplementName(), adoptPtrWillBeNoop(controller));
     }
     return *controller;
 }
 
 void DeviceMotionController::didAddEventListener(LocalDOMWindow* window, const AtomicString& eventType)
 {
+    if (eventType != eventTypeName())
+        return;
+
     if (document().frame()) {
         String errorMessage;
         if (document().securityOrigin()->canAccessFeatureRequiringSecureOrigin(errorMessage)) {
             UseCounter::count(document().frame(), UseCounter::DeviceMotionSecureOrigin);
         } else {
             UseCounter::count(document().frame(), UseCounter::DeviceMotionInsecureOrigin);
             if (document().frame()->settings()->strictPowerfulFeatureRestrictions())
                 return;
         }
     }
 
     DeviceSingleWindowEventController::didAddEventListener(window, eventType);

[Mike West, 2015/03/12 21:25:12]

Returning early changes the behavior of this delegation as well. Is that
intentional?

[timvolodine, 2015/03/13 11:40:55]

that's no problem, actually this method has an early return as well, it's just
the logging that is impacted.

 }
 
 bool DeviceMotionController::hasLastData()
 {
     return DeviceMotionDispatcher::instance().latestDeviceMotionData();
 }
 
 void DeviceMotionController::registerWithDispatcher()
 {
     DeviceMotionDispatcher::instance().addController(this);
@@ skipping 20 matching lines @@
     return EventTypeNames::devicemotion;
 }
 
 DEFINE_TRACE(DeviceMotionController)
 {
     DeviceSingleWindowEventController::trace(visitor);
     DocumentSupplement::trace(visitor);
 }
 
 } // namespace blink