Move push client to frame.

Source/modules/push_messaging/PushController.h

 // 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.
 
 #ifndef PushController_h
 #define PushController_h
 
-#include "core/page/Page.h"
+#include "core/frame/LocalFrame.h"
 #include "platform/Supplementable.h"
 #include "wtf/Forward.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/PassOwnPtr.h"
 
 namespace blink {
 
 class WebPushClient;
 
-class PushController final : public NoBaseWillBeGarbageCollected<PushController>, public WillBeHeapSupplement<Page> {
+class PushController final : public NoBaseWillBeGarbageCollected<PushController>, public WillBeHeapSupplement<LocalFrame> {

[Peter Beverloo, 2014/10/29 11:25:07]

This will only be used for requesting permission, right? You should probably
call it PushPermissionController to clarify its responsibilities. (Which I
should do for Notifications as well.)

[Michael van Ouwerkerk, 2014/11/11 17:20:06]

This will also plumb through a sender id from the manifest.

     WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(PushController);
     WTF_MAKE_NONCOPYABLE(PushController);
 
 public:
     static PassOwnPtrWillBeRawPtr<PushController> create(WebPushClient*);
     static const char* supplementName();
-    static PushController* from(Page* page) { return static_cast<PushController*>(WillBeHeapSupplement<Page>::from(page, supplementName())); }
-    static WebPushClient* clientFrom(Page*);
+    static PushController* from(LocalFrame* frame) { return static_cast<PushController*>(WillBeHeapSupplement<LocalFrame>::from(frame, supplementName())); }
+    static WebPushClient* clientFrom(LocalFrame*);
 
     WebPushClient* client() const { return m_client; }

[Peter Beverloo, 2014/10/29 11:25:07]

nit: this should be private.

[Michael van Ouwerkerk, 2014/11/11 17:20:06]

Actually, I think this method is not needed at all.

 
-    virtual void trace(Visitor* visitor) override { WillBeHeapSupplement<Page>::trace(visitor); }
+    virtual void trace(Visitor* visitor) override { WillBeHeapSupplement<LocalFrame>::trace(visitor); }
 
 private:
     explicit PushController(WebPushClient*);
 
     WebPushClient* m_client;
 };
 
-void providePushControllerTo(Page&, WebPushClient*);
+void providePushControllerTo(LocalFrame&, WebPushClient*);
 
 } // namespace blink
 
 #endif // PushController_h