Implement events for extensions management API.

BUG=51178
TEST=The events described in the experimental.management API should fire for
extensions and apps being installed, uninstalled, enabled, and disabled.


Review URL: http://codereview.chromium.org/3278003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57831 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 22 insertions, 19 deletions

diff --git a/chrome/common/extensions/api/extension_api.json b/chrome/common/extensions/api/extension_api.json
index d8b3c02..1014714 100644
--- a/chrome/common/extensions/api/extension_api.json
+++ b/chrome/common/extensions/api/extension_api.json
@@ -3998,6 +3998,7 @@
           {
             "name": "callback",
             "type": "function",
+            "optional": "true",
             "parameters": []
           }
         ]
@@ -4046,25 +4047,25 @@
     "events": [
       {
         "name": "onInstalled",
-        "description": "(NOT YET IMPLEMENTED)",
+        "description": "Fired when an app or extension has been installed.",
         "type": "function",
         "parameters": [{"name": "info", "$ref":"ExtensionInfo"}]
       },
       {
         "name": "onUninstalled",
-        "description": "(NOT YET IMPLEMENTED)",
+        "description": "Fired when an app or extension has been uninstalled.",
         "type": "function",
-        "parameters": [{"name": "id", "type": "string", "description": "The id of the extension that was uninstalled."}]
+        "parameters": [{"name": "info", "$ref":"ExtensionInfo"}]
       },
       {
         "name": "onEnabled",
-        "description": "(NOT YET IMPLEMENTED)",
+        "description": "Fired when an app or extension has been enabled.",
         "type": "function",
         "parameters": [{"name": "info", "$ref":"ExtensionInfo"}]
       },
       {
         "name": "onDisabled",
-        "description": "(NOT YET IMPLEMENTED)",
+        "description": "Fired when an app or extension has been disabled",
         "type": "function",
         "parameters": [{"name": "info", "$ref":"ExtensionInfo"}]
       }
diff --git a/chrome/common/extensions/docs/experimental.management.html b/chrome/common/extensions/docs/experimental.management.html
index 4f3ac8d..479cd35 100644
--- a/chrome/common/extensions/docs/experimental.management.html
+++ b/chrome/common/extensions/docs/experimental.management.html
@@ -597,7 +597,7 @@
                   <!-- Note: intentionally longer 80 columns -->
                   <span>chrome.experimental.management.setEnabled</span>(<span class="null"><span style="display: none; ">, </span><span>string</span>
                       <var><span>id</span></var></span><span class="null"><span>, </span><span>boolean</span>
-                      <var><span>enabled</span></var></span><span class="null"><span>, </span><span>function</span>
+                      <var><span>enabled</span></var></span><span class="optional"><span>, </span><span>function</span>
                       <var><span>callback</span></var></span>)</div>
 
               <div class="description">
@@ -732,7 +732,7 @@
                 <!-- TYPE -->
                 <div style="display:inline">
                   (
-                    <span class="optional" style="display: none; ">optional</span>
+                    <span class="optional">optional</span>
                     <span class="enum" style="display: none; ">enumerated</span>
                     <span id="typeTemplate">
                       <span style="display: none; ">
@@ -799,11 +799,11 @@
                 <div>
                   <div>
                   <h4>Callback function</h4>
-                  <p>
+                  <p style="display: none; ">
                     The callback <em>parameter</em> should specify a function
                     that looks like this:
                   </p>
-                  <p style="display: none; ">
+                  <p>
                     If you specify the <em>callback</em> parameter, it should
                     specify a function that looks like this:
                   </p>
@@ -1267,7 +1267,7 @@
 
               <div class="summary">
                 <!-- Note: intentionally longer 80 columns -->
-                <span class="subdued">chrome.experimental.management.</span><span>onUninstalled</span><span class="subdued">.addListener</span>(function(<span>string id</span>) <span class="subdued">{...}</span>);
+                <span class="subdued">chrome.experimental.management.</span><span>onUninstalled</span><span class="subdued">.addListener</span>(function(<span>ExtensionInfo info</span>) <span class="subdued">{...}</span>);
               </div>
 
               <div class="description">
@@ -1280,7 +1280,7 @@
                   <div>
                     <div>
           <dt>
-            <var>id</var>
+            <var>info</var>
               <em>
 
                 <!-- TYPE -->
@@ -1289,15 +1289,15 @@
                     <span class="optional" style="display: none; ">optional</span>
                     <span class="enum" style="display: none; ">enumerated</span>
                     <span id="typeTemplate">
-                      <span style="display: none; ">
-                        <a> Type</a>
-                      </span>
                       <span>
-                        <span style="display: none; ">
+                        <a href="experimental.management.html#type-ExtensionInfo">ExtensionInfo</a>
+                      </span>
+                      <span style="display: none; ">
+                        <span>
                           array of <span><span></span></span>
                         </span>
-                        <span>string</span>
-                        <span style="display: none; "></span>
+                        <span>paramType</span>
+                        <span></span>
                       </span>
                     </span>
                   )
@@ -1305,10 +1305,12 @@
 
               </em>
           </dt>
-          <dd class="todo" style="display: none; ">
+          <dd class="todo">
             Undocumented.
           </dd>
-          <dd>The id of the extension that was uninstalled.</dd>
+          <dd style="display: none; ">
+            Description of this parameter from the json schema.
+          </dd>
           <dd style="display: none; ">
             This parameter was added in version
             <b><span></span></b>.