Make it possible for a navigator.connect service to receive messages as base::Value.

Make it possible for a navigator.connect service to receive messages as base::Value.

Also adds a simple service to allow layout tests to test all this.

This is part of a series of changes:
[1/5] Blink side changes in https://codereview.chromium.org/924983002/ that expose a needed v8 context
[2/5] https://codereview.chromium.org/921013002/ allow MessagePort to send messages as values
[3/5] https://codereview.chromium.org/944443003/ which causes the send_messages_as_values flag to be propagated
[4/5] This CL
[5/5] https://codereview.chromium.org/940423004/ adds layout tests to blink to test these new features

BUG=426458
Committed: https://crrev.com/c9182205440405d83beda62b7a75210a26082b11
Cr-Commit-Position: refs/heads/master@{#318903}

[Marijn Kruisselbrink, 2015-02-20 23:19:37]

mek@chromium.org changed reviewers:
+ scheib@chromium.org

[Marijn Kruisselbrink, 2015-02-20 23:19:37]

Maybe hold of with reviewing until jochen is back from vacation and has had a
chance to agree (or disagree) with the required blink change.

[scheib, 2015-02-26 22:20:45]

lgtm

https://codereview.chromium.org/938403005/diff/60001/content/browser/navigato...
File
content/browser/navigator_connect/navigator_connect_service_worker_service_factory.cc
(right):

https://codereview.chromium.org/938403005/diff/60001/content/browser/navigato...
content/browser/navigator_connect/navigator_connect_service_worker_service_factory.cc:193:
// clean them up when a service worker registration is deleted.
Tack on an issue number for this, add it to the issue dependency tree.

[Marijn Kruisselbrink, 2015-02-27 22:23:12]

https://codereview.chromium.org/938403005/diff/60001/content/browser/navigato...
File
content/browser/navigator_connect/navigator_connect_service_worker_service_factory.cc
(right):

https://codereview.chromium.org/938403005/diff/60001/content/browser/navigato...
content/browser/navigator_connect/navigator_connect_service_worker_service_factory.cc:193:
// clean them up when a service worker registration is deleted.
On 2015/02/26 22:20:45, scheib wrote:
> Tack on an issue number for this, add it to the issue dependency tree.

Done.

[Marijn Kruisselbrink, 2015-02-27 22:23:40]

mek@chromium.org changed reviewers:
+ pfeldman@chromium.org

[Marijn Kruisselbrink, 2015-02-27 22:23:41]

+pfeldman for content/shell OWNERS

[pfeldman, 2015-03-02 16:54:19]

shell owner lgtm

[Marijn Kruisselbrink, 2015-03-02 16:58:49]

mek@chromium.org changed reviewers:
+ avi@chromium.org

[Marijn Kruisselbrink, 2015-03-02 16:58:50]

+avi for content/public OWNERS

[Avi (use Gerrit), 2015-03-03 17:02:50]

On 2015/03/02 16:58:50, Marijn Kruisselbrink wrote:
> +avi for content/public OWNERS

content/public lgtm

[Marijn Kruisselbrink, 2015-03-03 17:03:13]

The CQ bit was checked by mek@chromium.org

[Marijn Kruisselbrink, 2015-03-03 17:03:14]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org
Link to the patchset: https://codereview.chromium.org/938403005/#ps100001
(title: "mention bug")

[commit-bot: I haz the power, 2015-03-03 17:03:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/938403005/100001

[commit-bot: I haz the power, 2015-03-03 17:56:19]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-03-03 17:57:10]

Patchset 6 (id:??) landed as
https://crrev.com/c9182205440405d83beda62b7a75210a26082b11
Cr-Commit-Position: refs/heads/master@{#318903}