Add some webapk source directories to eclise .classpath

Add some webapk source directories to eclise .classpath

Committed: https://crrev.com/06abe9e0f846d32f5f5bcc6a670c494f9ca81eb6
Cr-Commit-Position: refs/heads/master@{#403241}

[Yaron, 2016-06-30 16:32:38]

yfriedman@chromium.org changed reviewers:
+ hanxi@chromium.org

[Xi Han, 2016-06-30 16:39:17]

Thanks for updating this file!

https://codereview.chromium.org/2115493005/diff/1/tools/android/eclipse/.clas...
File tools/android/eclipse/.classpath (right):

https://codereview.chromium.org/2115493005/diff/1/tools/android/eclipse/.clas...
tools/android/eclipse/.classpath:33: <classpathentry kind="src"
path="chrome/android/webapk/libs/common/src"/>
Move line 33 "libs/common" to after line 35 "libs/client/src".

https://codereview.chromium.org/2115493005/diff/1/tools/android/eclipse/.clas...
tools/android/eclipse/.classpath:38: <classpathentry kind="src"
path="chrome/android/webapk/shell_apk/javatests/dex_optimizer/src"/>
Add "shell_apk/javatests/canary_lib/src" too?

[Yaron, 2016-06-30 16:41:05]

https://codereview.chromium.org/2115493005/diff/1/tools/android/eclipse/.clas...
File tools/android/eclipse/.classpath (right):

https://codereview.chromium.org/2115493005/diff/1/tools/android/eclipse/.clas...
tools/android/eclipse/.classpath:33: <classpathentry kind="src"
path="chrome/android/webapk/libs/common/src"/>
On 2016/06/30 16:39:16, Xi Han wrote:
> Move line 33 "libs/common" to after line 35 "libs/client/src".

Done.

https://codereview.chromium.org/2115493005/diff/1/tools/android/eclipse/.clas...
tools/android/eclipse/.classpath:38: <classpathentry kind="src"
path="chrome/android/webapk/shell_apk/javatests/dex_optimizer/src"/>
On 2016/06/30 16:39:16, Xi Han wrote:
> Add "shell_apk/javatests/canary_lib/src" too?

It's really not interesting (just a placeholder) so I'm intentionally omittiing

[Yaron, 2016-06-30 16:41:41]

The CQ bit was checked by yfriedman@chromium.org

[Yaron, 2016-06-30 16:41:50]

The CQ bit was unchecked by yfriedman@chromium.org

[commit-bot: I haz the power, 2016-06-30 16:41:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 16:41:54]

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

[commit-bot: I haz the power, 2016-06-30 16:41:54]

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.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Xi Han, 2016-06-30 17:17:02]

lgtm

[Yaron, 2016-06-30 18:07:02]

The CQ bit was checked by yfriedman@chromium.org

[commit-bot: I haz the power, 2016-06-30 18:07:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 19:09:41]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-30 19:12:46]

Description was changed from

==========
Add some webapk source directories to eclise .classpath
==========

to

==========
Add some webapk source directories to eclise .classpath

Committed: https://crrev.com/06abe9e0f846d32f5f5bcc6a670c494f9ca81eb6
Cr-Commit-Position: refs/heads/master@{#403241}
==========

[commit-bot: I haz the power, 2016-06-30 19:12:47]

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