DO NOT REVIEW Bindings and plumbing for push messaging API.

Source/modules/pushmessaging/NavigatorPushManager.h

Index: Source/modules/pushmessaging/NavigatorPushManager.h
diff --git a/Source/modules/pushmessaging/NavigatorPushManager.h b/Source/modules/pushmessaging/NavigatorPushManager.h
new file mode 100644
index 0000000000000000000000000000000000000000..db228660a51ed61a2864d546c4022483367ed81c
--- /dev/null
+++ b/Source/modules/pushmessaging/NavigatorPushManager.h
@@ -0,0 +1,32 @@
+// 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 NavigatorPushManager_h
+#define NavigatorPushManager_h
+
+#include "platform/Supplementable.h"
+
+namespace WebCore {
+
+class Navigator;
+class PushManager;
+
+class NavigatorPushManager FINAL : public Supplement<Navigator> {
+public:
+    virtual ~NavigatorPushManager();
+    static NavigatorPushManager& from(Navigator&);
+
+    static PushManager* push(Navigator&);
+    PushManager* pushManager() const;

[Peter Beverloo, 2014/03/10 13:59:28]

I don't think it's good to mark this method as const :(. If it were just to ref
and deref the RefPtr it'd be fine, but it actually creates a PushManager
instance if it doesn't exist yet. Naming it ensurePushManager() and have it be
non-const would be a lot better, but this seems to be used more often :/.

[Michael van Ouwerkerk, 2014/03/10 15:12:04]

Maybe I'm doing the old thing because I looked at the Geolocation code, among
others.

+
+private:
+    NavigatorPushManager();
+    static const char* supplementName();
+
+    mutable RefPtr<PushManager> m_pushManager;
+};
+
+} // namespace WebCore
+
+#endif // NavigatorPushManager_h