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

extensions/common/api/web_view_internal.json

Index: extensions/common/api/web_view_internal.json
diff --git a/extensions/common/api/web_view_internal.json b/extensions/common/api/web_view_internal.json
index bb85dabf9a964b2058b8e2d117f0acf4e64f00bd..bae49bd6034a9f3523cc598ee6e396215eaf4667 100644
--- a/extensions/common/api/web_view_internal.json
+++ b/extensions/common/api/web_view_internal.json
@@ -120,6 +120,12 @@
             "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,