Use a new Supplement constructor for LocalFrame's supplements

third_party/WebKit/Source/modules/sensor/SensorProviderProxy.cpp

Index: third_party/WebKit/Source/modules/sensor/SensorProviderProxy.cpp
diff --git a/third_party/WebKit/Source/modules/sensor/SensorProviderProxy.cpp b/third_party/WebKit/Source/modules/sensor/SensorProviderProxy.cpp
index 603e3e2bbdd84456e0bba6394a2bfadaeaf11d45..911c21cf7704f5c1a87d93859aa4da91a8f19de3 100644
--- a/third_party/WebKit/Source/modules/sensor/SensorProviderProxy.cpp
+++ b/third_party/WebKit/Source/modules/sensor/SensorProviderProxy.cpp
@@ -13,12 +13,12 @@
 namespace blink {
 
 // SensorProviderProxy
-SensorProviderProxy::SensorProviderProxy(LocalFrame* frame) {
-  initialize(frame);

[haraken, 2017/01/06 12:46:55]

This is not needed because it's called at line 43.

-}
+SensorProviderProxy::SensorProviderProxy(LocalFrame& frame)
+    : Supplement<LocalFrame>(frame) {}
 
-void SensorProviderProxy::initialize(LocalFrame* frame) {
-  DCHECK(!isInitialized());
+void SensorProviderProxy::initializeIfNeeded(LocalFrame* frame) {
+  if (isInitialized())
+    return;
 
   frame->interfaceProvider()->getInterface(
       mojo::MakeRequest(&m_sensorProvider));
@@ -37,13 +37,10 @@ SensorProviderProxy* SensorProviderProxy::from(LocalFrame* frame) {
   SensorProviderProxy* providerProxy = static_cast<SensorProviderProxy*>(
       Supplement<LocalFrame>::from(*frame, supplementName()));
   if (!providerProxy) {
-    providerProxy = new SensorProviderProxy(frame);
+    providerProxy = new SensorProviderProxy(*frame);
     Supplement<LocalFrame>::provideTo(*frame, supplementName(), providerProxy);
   }
-
-  if (!providerProxy->isInitialized())
-    providerProxy->initialize(frame);
-
+  providerProxy->initializeIfNeeded(frame);
   return providerProxy;
 }