Remove code that was only used by navigator.connect.

Remove code that was only used by navigator.connect.

All the logic to send messages over MessagePort's as base::Value instances
was only used by navigator.connect. So with that API gone, also clean up all
the base::Value logic.

This more or less reverts https://codereview.chromium.org/944443003 and
https://codereview.chromium.org/924983002/.

BUG=461906
Committed: https://crrev.com/d27aef56ddecda2e7245808330c1d13bae5fb4ac
Cr-Commit-Position: refs/heads/master@{#393589}

[Marijn Kruisselbrink, 2016-05-11 23:05:07]

Patchset #2 (id:20001) has been deleted

[Marijn Kruisselbrink, 2016-05-11 23:12:35]

Description was changed from

==========
Remove code that was only used by navigator.connect.

All the logic to send messages over MessagePort's as base::Value instances
was only used by navigator.connect. So with that API gone, also clean up all
the base::Value logic.

BUG=461906
==========

to

==========
Remove code that was only used by navigator.connect.

All the logic to send messages over MessagePort's as base::Value instances
was only used by navigator.connect. So with that API gone, also clean up all
the base::Value logic.

This more or less reverts https://codereview.chromium.org/944443003 and
https://codereview.chromium.org/924983002/.

BUG=461906
==========

[Marijn Kruisselbrink, 2016-05-12 00:07:08]

mek@chromium.org changed reviewers:
+ avi@chromium.org, haraken@chromium.org, sgurun@chromium.org,
tsepez@chromium.org

[Marijn Kruisselbrink, 2016-05-12 00:07:09]

avi for content/ OWNERS
sgurun for android_webview/ OWNERS
tsepez for IPC OWNERS
haraken for WebKit OWNERS

[Tom Sepez, 2016-05-12 00:25:05]

IPC OWNER LGTM.

[haraken, 2016-05-12 00:38:44]

WebKit/Source/ LGTM

[Avi (use Gerrit), 2016-05-12 01:39:25]

content lgtm

[kinuko, 2016-05-12 02:04:46]

kinuko@chromium.org changed reviewers:
+ kinuko@chromium.org

[kinuko, 2016-05-12 02:04:47]

drive-by lgtm for the part I care about

https://codereview.chromium.org/1974613002/diff/40001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebMessagePortChannelClient.h (right):

https://codereview.chromium.org/1974613002/diff/40001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebMessagePortChannelClient.h:40: // access
to the V8 Context this message port lives in.
nit: The second sentence seems now obsolete

[sgurun-gerrit only, 2016-05-12 18:54:26]

On 2016/05/12 02:04:47, kinuko wrote:
> drive-by lgtm for the part I care about
> 
>
https://codereview.chromium.org/1974613002/diff/40001/third_party/WebKit/publ...
> File third_party/WebKit/public/platform/WebMessagePortChannelClient.h (right):
> 
>
https://codereview.chromium.org/1974613002/diff/40001/third_party/WebKit/publ...
> third_party/WebKit/public/platform/WebMessagePortChannelClient.h:40: // access
> to the V8 Context this message port lives in.
> nit: The second sentence seems now obsolete

aw lgtm

[Marijn Kruisselbrink, 2016-05-12 18:59:22]

The CQ bit was checked by mek@chromium.org

[commit-bot: I haz the power, 2016-05-12 18:59:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1974613002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1974613002/40001

[commit-bot: I haz the power, 2016-05-12 19:10:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-12 19:10:33]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Marijn Kruisselbrink, 2016-05-12 19:15:36]

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

[Marijn Kruisselbrink, 2016-05-12 19:15:37]

+alekseys for chrome/browser/printing OWNERS

[kinuko, 2016-05-13 04:43:38]

Could we fix the comment before landing?

On 2016/05/12 02:04:47, kinuko wrote:
>
https://codereview.chromium.org/1974613002/diff/40001/third_party/WebKit/publ...
> File third_party/WebKit/public/platform/WebMessagePortChannelClient.h (right):
> 
>
https://codereview.chromium.org/1974613002/diff/40001/third_party/WebKit/publ...
> third_party/WebKit/public/platform/WebMessagePortChannelClient.h:40: // access
> to the V8 Context this message port lives in.
> nit: The second sentence seems now obsolete

[Marijn Kruisselbrink, 2016-05-13 15:53:20]

On 2016/05/13 at 04:43:38, kinuko wrote:
> Could we fix the comment before landing?

Oops, I'm sorry about that. Fixed now.

[Aleksey Shlyapnikov, 2016-05-13 18:24:23]

lgtm

printing lgtm

[Marijn Kruisselbrink, 2016-05-13 18:25:15]

The CQ bit was checked by mek@chromium.org

[Marijn Kruisselbrink, 2016-05-13 18:25:15]

The patchset sent to the CQ was uploaded after l-g-t-m from sgurun@chromium.org,
avi@chromium.org, kinuko@chromium.org, tsepez@chromium.org, haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1974613002/#ps50001
(title: "fix comment")

[commit-bot: I haz the power, 2016-05-13 18:25:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1974613002/50001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1974613002/50001

[commit-bot: I haz the power, 2016-05-13 18:33:26]

Committed patchset #3 (id:50001)

[commit-bot: I haz the power, 2016-05-13 18:35:08]

Description was changed from

==========
Remove code that was only used by navigator.connect.

All the logic to send messages over MessagePort's as base::Value instances
was only used by navigator.connect. So with that API gone, also clean up all
the base::Value logic.

This more or less reverts https://codereview.chromium.org/944443003 and
https://codereview.chromium.org/924983002/.

BUG=461906
==========

to

==========
Remove code that was only used by navigator.connect.

All the logic to send messages over MessagePort's as base::Value instances
was only used by navigator.connect. So with that API gone, also clean up all
the base::Value logic.

This more or less reverts https://codereview.chromium.org/944443003 and
https://codereview.chromium.org/924983002/.

BUG=461906

Committed: https://crrev.com/d27aef56ddecda2e7245808330c1d13bae5fb4ac
Cr-Commit-Position: refs/heads/master@{#393589}
==========

[commit-bot: I haz the power, 2016-05-13 18:35:10]

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