Clarify runtime.connect and runtime.sendMessage docs.

Clarify runtime.connect and runtime.sendMessage docs.

BUG=326250
R=mkearney@chromium.org
TBR=yoz@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245292

[not at google - send to devlin, 2014-01-10 20:08:17]

I will certainly need some help to polish this.

[mkearney1, 2014-01-13 22:08:43]

lgtm

https://codereview.chromium.org/134533004/diff/1/chrome/common/extensions/api...
File chrome/common/extensions/api/runtime.json (right):

https://codereview.chromium.org/134533004/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/runtime.json:215: "description": "Sends a single
message to event listeners within your extension/app or a different
extension/app. Similar to $ref:runtime.connect but only sends a single message,
with an optional response. If sending to your extension the
$ref:runtime.onMessage event will be fired in each page, or
$ref:runtime.onMessageExternal if a different extension. Note that extensions
cannot send messages to content scripts using this method. To send messages to
content scripts, use $ref:tabs.sendMessage.",
Small nit: comma after 'If sending to your extension' and also after
onMessageExternal (before 'if a different extension).

https://codereview.chromium.org/134533004/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/runtime.json:217: {"type": "string", "name":
"extensionId", "optional": true, "description": "The ID of the extension/app to
send the message to. If omitted the message will be sent to your own
extension/app. Required if sending messages from a web page for <a
href=\"manifest/externally_connectable.html\">web messaging</a>."},
Small nit: comma after 'If omitted'.

[not at google - send to devlin, 2014-01-16 00:25:41]

https://codereview.chromium.org/134533004/diff/1/chrome/common/extensions/api...
File chrome/common/extensions/api/runtime.json (right):

https://codereview.chromium.org/134533004/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/runtime.json:215: "description": "Sends a single
message to event listeners within your extension/app or a different
extension/app. Similar to $ref:runtime.connect but only sends a single message,
with an optional response. If sending to your extension the
$ref:runtime.onMessage event will be fired in each page, or
$ref:runtime.onMessageExternal if a different extension. Note that extensions
cannot send messages to content scripts using this method. To send messages to
content scripts, use $ref:tabs.sendMessage.",
On 2014/01/13 22:08:43, mkearney1 wrote:
> Small nit: comma after 'If sending to your extension' and also after
> onMessageExternal (before 'if a different extension).

Done.

https://codereview.chromium.org/134533004/diff/1/chrome/common/extensions/api...
chrome/common/extensions/api/runtime.json:217: {"type": "string", "name":
"extensionId", "optional": true, "description": "The ID of the extension/app to
send the message to. If omitted the message will be sent to your own
extension/app. Required if sending messages from a web page for <a
href=\"manifest/externally_connectable.html\">web messaging</a>."},
On 2014/01/13 22:08:43, mkearney1 wrote:
> Small nit: comma after 'If omitted'.

Done.

[commit-bot: I haz the power, 2014-01-16 00:38:23]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[commit-bot: I haz the power, 2014-01-16 01:28:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kalman@chromium.org/134533004/50001

[commit-bot: I haz the power, 2014-01-16 06:54:27]

Retried try job too often on win_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-16 17:19:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kalman@chromium.org/134533004/50001

[commit-bot: I haz the power, 2014-01-16 19:50:05]

Change committed as 245292