Eliminate third_party/mojo/src from all targets' include paths

Eliminate third_party/mojo/src from all targets' include paths

Eliding presubmit here because the missing DEPS entries are going to be addressed here: https://codereview.chromium.org/1410053006

BUG=554556
R=jam@chromium.org
NOPRESUBMIT=true

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

[jam, 2015-11-11 20:13:28]

rubberstamp lgtm. didn't look at all the cc files, just some. i did look at
build.gn and gyp files.

my main question is: don't you also need to change the GN files to not have
third_party/mojo as an include dir?

[Ken Rockot(use gerrit already), 2015-11-11 20:25:15]

On 2015/11/11 at 20:13:28, jam wrote:
> rubberstamp lgtm. didn't look at all the cc files, just some. i did look at
build.gn and gyp files.
> 
> my main question is: don't you also need to change the GN files to not have
third_party/mojo as an include dir?

Ah you're right, thanks. I've wiped out the public config which was responsible
for setting that include path in GN targets.

[Ken Rockot(use gerrit already), 2015-11-11 20:49:30]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2015-11-11 20:49:31]

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/1435063002/#ps40001
(title: "minus gn config")

[commit-bot: I haz the power, 2015-11-11 20:52:23]

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

[Ken Rockot(use gerrit already), 2015-11-11 21:09:23]

Description was changed from

==========
Eliminate third_party/mojo/src from all targets' include paths

BUG=554556
R=jam@chromium.org
==========

to

==========
Eliminate third_party/mojo/src from all targets' include paths

Eliding presubmit here because the missing DEPS entries are going to be
addressed here: https://codereview.chromium.org/1410053006

BUG=554556
R=jam@chromium.org
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2015-11-11 21:14:36]

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

[commit-bot: I haz the power, 2015-11-11 21:14:38]

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-11-11 21:16:26]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2015-11-11 21:18:07]

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

[Ken Rockot(use gerrit already), 2015-11-11 21:26:42]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2015-11-11 21:26:44]

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/1435063002/#ps60001
(title: "fix DEPS")

[commit-bot: I haz the power, 2015-11-11 21:28:30]

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

[Ken Rockot(use gerrit already), 2015-11-11 22:26:46]

Committed patchset #5 (id:50457) manually as
f2bfe56ff56152e21033992105c31437a3b150fe (presubmit successful).

[commit-bot: I haz the power, 2015-11-12 20:01:16]

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