[AppBanner] Blink implementation of an event when an install banner is shown.

Source/modules/app_banner/AppBannerController.h

Index: Source/modules/app_banner/AppBannerController.h
diff --git a/Source/modules/app_banner/AppBannerController.h b/Source/modules/app_banner/AppBannerController.h
new file mode 100644
index 0000000000000000000000000000000000000000..d97ab5dd176ebca907744dee8e24fe0d77ef3f81
--- /dev/null
+++ b/Source/modules/app_banner/AppBannerController.h
@@ -0,0 +1,24 @@
+// Copyright 2015 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 AppBannerController_h
+#define AppBannerController_h
+
+namespace blink {
+
+class LocalFrame;
+class WebString;
+
+class AppBannerController final {

[Mike West, 2015/03/06 14:30:29]

Wrapping this in a class doesn't really win you much, does it? We have a number
of other static functions in Blink (mostly in //platform). It wouldn't be
terrible to add another if you'd rather go that route.

[mlamouri (slow - plz ping), 2015/03/06 18:50:47]

It's in a class because AppBannerController will be used for userChoice. It will
carry the client in order to ask the embedder what the userchoice was (I started
implementing that then cleaned up things, thus the remainings). I would prefer
to keep it as is for now instead or removing the class then add it back. Will
add a FIXME though to make sure the issue is visible in case of userChoice never
happens.

+public:
+    static void willShowInstallBannerPrompt(LocalFrame*, const WebString& platform, bool* cancel);

[Mike West, 2015/03/06 14:30:29]

I have a vague suspicion that replacing that bool with an enum will be clearer
when you start using this.

[mlamouri (slow - plz ping), 2015/03/06 18:50:47]

Done.

+
+private:
+    AppBannerController() = delete;
+    ~AppBannerController() = delete;
+};
+
+} // namespace blink
+
+#endif // AppBannerController_h