Update mojo sdk to rev f84766d3b6420b7cf6a113d9d65d73cb5fe18d90

Update mojo sdk to rev f84766d3b6420b7cf6a113d9d65d73cb5fe18d90

Noteworthy in this roll is that InterfacePtrs may no longer be
bound-to or passed-as simple pipe handles, but instead as
InterfacePtrInfos. This requires updating several call sites
of InterfacePtr's Bind and PassMessagePipe.

BUG=
TBR=ben@chromium.org

Committed: https://crrev.com/d24477996ef10584ced040a13f82e7c7dc0216b1
Cr-Commit-Position: refs/heads/master@{#329783}

[Ken Rockot(use gerrit already), 2015-05-07 01:08:01]

Patchset #3 (id:40001) has been deleted

[Ken Rockot(use gerrit already), 2015-05-08 01:54:15]

Patchset #4 (id:80001) has been deleted

[Ken Rockot(use gerrit already), 2015-05-08 17:51:34]

Patchset #4 (id:100001) has been deleted

[Ken Rockot(use gerrit already), 2015-05-08 17:53:36]

rockot@chromium.org changed reviewers:
+ sky@chromium.org

[Ken Rockot(use gerrit already), 2015-05-08 17:53:37]

Scott, could you please take a look at the non-third_party, non-ipc bits of this
CL? Had to update some code per the CL description.

Landing this is still blocking on IPC fixes from
https://codereview.chromium.org/1130413002/, but we should be good to go after
that.

[sky, 2015-05-08 19:08:03]

I'm not familiar with the ipc code to do it justice. Can you find another owner?
The rest of the changes LGTM

https://codereview.chromium.org/1127293003/diff/120001/mojo/shell/application...
File mojo/shell/application_manager.cc (right):

https://codereview.chromium.org/1127293003/diff/120001/mojo/shell/application...
mojo/shell/application_manager.cc:44:
content_handler_.Bind(InterfacePtrInfo<ContentHandler>(pipe.handle0.Pass(),
0u));
nit: > 80.

[Ken Rockot(use gerrit already), 2015-05-08 19:12:48]

Thanks! I've asked morrita to review the IPC CL. The IPC changes here are just
merged directly from that CL for trybot testing. They aren't really a part of
this change.

https://codereview.chromium.org/1127293003/diff/120001/mojo/shell/application...
File mojo/shell/application_manager.cc (right):

https://codereview.chromium.org/1127293003/diff/120001/mojo/shell/application...
mojo/shell/application_manager.cc:44:
content_handler_.Bind(InterfacePtrInfo<ContentHandler>(pipe.handle0.Pass(),
0u));
On 2015/05/08 19:08:02, sky wrote:
> nit: > 80.

Fixed

[Ken Rockot(use gerrit already), 2015-05-12 17:39:04]

rockot@chromium.org changed reviewers:
+ jam@chromium.org

[Ken Rockot(use gerrit already), 2015-05-12 17:39:33]

+jam: Still blocking on the GYP dependency issue if you have any ideas

[Ken Rockot(use gerrit already), 2015-05-12 22:17:09]

Patchset #10 (id:240001) has been deleted

[Ken Rockot(use gerrit already), 2015-05-13 02:09:54]

Well, only Android is broken now and it repros locally, so at least we've got
that going for us.

There are a bunch of dex errors regarding duplicate definitions of generated
interface control messages (ICM) code.

The GN (third_party/mojo/src/mojo/public/java:bindings) and GYP
(third_party/mojo/mojo_public.gyp:mojo_bindings_java) rules for java bindings
should effectively produce the same jar files and result in the same impact on
their dependents.

Maybe I'm missing something, or maybe there's a functional difference between
linking against a srcjar (GN dependents do this) vs linking against individual
java files (GYP dependents do this).

[Ken Rockot(use gerrit already), 2015-05-14 00:08:46]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2015-05-14 00:08:47]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1127293003/#ps340001
(title: "1. fix gyp; 2. try not to cry; 3. cry a lot.")

[commit-bot: I haz the power, 2015-05-14 00:11:20]

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

[commit-bot: I haz the power, 2015-05-14 00:24:02]

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

[commit-bot: I haz the power, 2015-05-14 00:24:04]

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...)

[Ken Rockot(use gerrit already), 2015-05-14 00:53:40]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2015-05-14 00:53:42]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1127293003/#ps360001
(title: "formatting")

[commit-bot: I haz the power, 2015-05-14 00:54:36]

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

[commit-bot: I haz the power, 2015-05-14 02:07:25]

Committed patchset #15 (id:360001)

[commit-bot: I haz the power, 2015-05-14 02:08:10]

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