Update mojo sdk to rev 1dc8a9a5db73d3718d99917fadf31f5fb2ebad4f

Update mojo sdk to rev 1dc8a9a5db73d3718d99917fadf31f5fb2ebad4f

BUG=None
R=jam@chromium.org

Committed: https://crrev.com/8ddc364fd10bcfe1a4332f0f66be4e5db84e39b6
Cr-Commit-Position: refs/heads/master@{#331044}

[jam, 2015-05-20 14:58:19]

https://codereview.chromium.org/1142043005/diff/1/build/module_args/mojo.gni
File build/module_args/mojo.gni (right):

https://codereview.chromium.org/1142043005/diff/1/build/module_args/mojo.gni#...
build/module_args/mojo.gni:20: mojo_disable_application_in_sdk = true
Thanks, I forgot to send out the local change for that :)

https://codereview.chromium.org/1142043005/diff/1/third_party/mojo/src/mojo/p...
File
third_party/mojo/src/mojo/public/java/application/src/org/chromium/mojo/application/ApplicationConnection.java
(right):

https://codereview.chromium.org/1142043005/diff/1/third_party/mojo/src/mojo/p...
third_party/mojo/src/mojo/public/java/application/src/org/chromium/mojo/application/ApplicationConnection.java:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
we don't want the files in third_party/mojo/src/mojo/public/java/application
since we have our own copy in mojo/application/public/java.

looks like I put in the wrong value of
"third_party/mojo/src/mojo/public/java/application" in rev_sdk's
sdk_dirs_to_not_clone. It should be "mojo/public/java/application".

Can you update the script and run it again so that these java files aren't
brought in?

[Ken Rockot(use gerrit already), 2015-05-20 16:28:37]

fixed; looking into windows mojo_system_unittests failures now

https://codereview.chromium.org/1142043005/diff/1/third_party/mojo/src/mojo/p...
File
third_party/mojo/src/mojo/public/java/application/src/org/chromium/mojo/application/ApplicationConnection.java
(right):

https://codereview.chromium.org/1142043005/diff/1/third_party/mojo/src/mojo/p...
third_party/mojo/src/mojo/public/java/application/src/org/chromium/mojo/application/ApplicationConnection.java:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
On 2015/05/20 14:58:18, jam wrote:
> we don't want the files in third_party/mojo/src/mojo/public/java/application
> since we have our own copy in mojo/application/public/java.
> 
> looks like I put in the wrong value of
> "third_party/mojo/src/mojo/public/java/application" in rev_sdk's
> sdk_dirs_to_not_clone. It should be "mojo/public/java/application".
> 
> Can you update the script and run it again so that these java files aren't
> brought in?

Done.

[jam, 2015-05-20 18:11:32]

lgtm

[Ken Rockot(use gerrit already), 2015-05-21 06:39:57]

Patchset #3 (id:40001) has been deleted

[Ken Rockot(use gerrit already), 2015-05-21 06:40:11]

Patchset #3 (id:60001) has been deleted

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

The CQ bit was checked by rockot@chromium.org

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

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1142043005/#ps180001
(title: "disable some tests")

[commit-bot: I haz the power, 2015-05-22 00:12:49]

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

[commit-bot: I haz the power, 2015-05-22 01:42:25]

Committed patchset #8 (id:180001)

[commit-bot: I haz the power, 2015-05-22 01:43:03]

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