Add chrome.webstore API methods to allow sites to see progress of installation

chrome/common/extensions/extension_messages.h

 // Copyright (c) 2012 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.
 
 // IPC messages for extensions.
 // Multiply-included message file, hence no include guard.
 
 #include <string>
 #include <vector>
 
@@ skipping 583 matching lines @@
 IPC_MESSAGE_ROUTED3(ExtensionHostMsg_ContentScriptsExecuting,
                     ExecutingScriptsMap,
                     int32 /* page_id of the _topmost_ frame */,
                     GURL /* url of the _topmost_ frame */)
 
 IPC_MESSAGE_ROUTED2(ExtensionHostMsg_DidGetApplicationInfo,
                     int32 /* page_id */,
                     WebApplicationInfo)
 
 // Sent by the renderer to implement chrome.webstore.install().
-IPC_MESSAGE_ROUTED4(ExtensionHostMsg_InlineWebstoreInstall,
+IPC_MESSAGE_ROUTED5(ExtensionHostMsg_InlineWebstoreInstall,
                     int32 /* install id */,
                     int32 /* return route id */,
                     std::string /* Web Store item ID */,
-                    GURL /* requestor URL */)
+                    GURL /* requestor URL */,
+                    int /* listeners_mask */)
+
+// Sent to the renderer if install stage updates were requested for an inline
+// install.
+IPC_MESSAGE_ROUTED1(ExtensionMsg_InlineInstallStageChanged,
+                    std::string /* stage */)

[jln (very slow on Chromium), 2014/03/10 23:35:41]

Could you make this an enum instad of a string?

We strongly discourage sending strings over IPCs:
http://www.chromium.org/Home/chromium-security/education/security-tips-for-ip....

[Devlin, 2014/03/11 17:22:06]

Done, and thanks for the tip! :)

[jln (very slow on Chromium), 2014/03/20 00:05:40]

Sorry, I should have been more clear. You need to serialize the enum properly,
please use IPC_ENUM_TRAITS_MAX_VALUE or IPC_ENUM_TRAITS_VALIDATE.

[Devlin, 2014/03/20 16:54:37]

Ah, I see.  Like this?

+
+// Sent to the renderer if download progress updates were requested for an
+// inline install.
+IPC_MESSAGE_ROUTED1(ExtensionMsg_InlineInstallDownloadProgress,
+                    int /* percent_downloaded */)
 
 // Sent by the renderer when a web page is checking if its app is installed.
 IPC_MESSAGE_ROUTED3(ExtensionHostMsg_GetAppInstallState,
                     GURL /* requestor_url */,
                     int32 /* return_route_id */,
                     int32 /* callback_id */)
 
 // Optional Ack message sent to the browser to notify that the response to a
 // function has been processed.
 IPC_MESSAGE_ROUTED1(ExtensionHostMsg_ResponseAck,
@@ skipping 47 matching lines @@
 // certain conditions.  This message is sent in response to several events:
 //
 // * ExtensionMsg_WatchPages was received, updating the set of conditions.
 // * A new page is loaded.  This will be sent after
 //   FrameHostMsg_DidCommitProvisionalLoad. Currently this only fires for the
 //   main frame.
 // * Something changed on an existing frame causing the set of matching searches
 //   to change.
 IPC_MESSAGE_ROUTED1(ExtensionHostMsg_OnWatchedPageChange,
                     std::vector<std::string> /* Matching CSS selectors */)