build: use class path bases, not strings

build: use class path bases, not strings

Depends on https://codereview.chromium.org/1925653002

TBR=martiniss

BUG=chromium:605919
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300394

[Paweł Hajdan Jr., 2016-04-27 12:40:24]

phajdan.jr@chromium.org changed reviewers:
+ iannucci@chromium.org, martiniss@chromium.org

[martiniss, 2016-04-27 21:41:45]

The CQ bit was checked by martiniss@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-27 21:41:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1923883003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1923883003/1

[commit-bot: I haz the power, 2016-04-27 21:49:58]

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

[commit-bot: I haz the power, 2016-04-27 21:49:59]

Dry run: Try jobs failed on following builders:
  Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[martiniss, 2016-04-27 22:10:06]

idea looks fine to me, but there are several breakages both in the build
presubmit, and the build_internal tryjob that probably need to be fixed.

[Paweł Hajdan Jr., 2016-05-02 08:59:32]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-05-02 08:59:41]

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

[commit-bot: I haz the power, 2016-05-02 08:59:41]

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

[commit-bot: I haz the power, 2016-05-02 08:59:42]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[Paweł Hajdan Jr., 2016-05-02 09:04:07]

Description was changed from

==========
build: use class path bases, not strings

Depends on https://codereview.chromium.org/1925653002

BUG=chromium:605919
==========

to

==========
build: use class path bases, not strings

Depends on https://codereview.chromium.org/1925653002

TBR=martiniss

BUG=chromium:605919
==========

[Paweł Hajdan Jr., 2016-05-02 09:04:10]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-05-02 09:04:12]

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

[commit-bot: I haz the power, 2016-05-02 09:08:11]

Description was changed from

==========
build: use class path bases, not strings

Depends on https://codereview.chromium.org/1925653002

TBR=martiniss

BUG=chromium:605919
==========

to

==========
build: use class path bases, not strings

Depends on https://codereview.chromium.org/1925653002

TBR=martiniss

BUG=chromium:605919

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300394
==========

[commit-bot: I haz the power, 2016-05-02 09:08:12]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300394