Implement <webview>.addContentScript/removeContentScript API [3]

extensions/common/api/web_view_internal.json

 // 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.
 
 [
   {
     "namespace": "webViewInternal",
     "description": "none",
     "compiler_options": {
       "implemented_in": "extensions/browser/api/guest_view/web_view/web_view_internal_api.h"
@@ skipping 102 matching lines @@
             "items": { "type": "string"},
             "optional": true,
             "description": "The list of CSS files to be injected into matching pages. These are injected in the order they appear in this array, before any DOM is constructed or displayed for the page."
           },
           "js": {
             "type": "array",
             "items": { "type": "string"},
             "optional": true,
             "description": "The list of JavaScript files to be injected into matching pages. These are injected in the order they appear in this array."
           },
+          "code": {
+            "type": "array",

[xiyuan, 2015/04/07 20:35:02]

Shouldn't a string be sufficient?

[Xi Han, 2015/04/07 20:43:10]

I just make the "code" be consistent with "js" or "css". So user can inject
multiple code at the same time, just as injecting multiple .js files:)

[xiyuan, 2015/04/07 21:13:14]

"js"/"css" refers to files and it makes sense to have an array to combine
multiple files. But "code" essentially are strings and developer could just
concat them if needed. In this regard, array seems a bit unnecessary to me. But
I don't feel strongly about it. So okay to leave as-is.

[Xi Han, 2015/04/08 14:47:40]

Your point makes sense to me. Updated to a string:)

+            "items": { "type": "string"},
+            "optional": true,
+            "description": "The list of JavaScript code to be injected into matching pages. These are injected in the order they appear in this array."
+          },
           "run_at": {
             "type": "string",
             "optional": true,
             "enum": ["document_start", "document_end", "document_idle"],
             "description": "The soonest that the JavaScript or CSS will be injected into the tab. Defaults to \"document_idle\"."
           },
           "all_frames": {
             "type": "boolean",
             "optional": true,
             "description": "If allFrames is <code>true</code>, implies that the JavaScript or CSS should be injected into all frames of current page. By default, it's <code>false</code> and is only injected into the top frame."
@@ skipping 551 matching lines @@
             "type": "function",
             "description": "Called when deletion has completed.",
             "optional": true,
             "parameters": []
           }
         ]
       }
     ]
   }
 ]