[Cronet] Add presubmit check to verify Cronet package division.

[Cronet] Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.

BUG=629299
Committed: https://crrev.com/17ebec651e9b55a0dfe8d64864a0bc3592b3b4ee
Cr-Commit-Position: refs/heads/master@{#418623}

[pauljensen, 2016-09-09 11:27:59]

Description was changed from

==========
Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.
==========

to

==========
[Cronet] Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.

BUG=629299
==========

[pauljensen, 2016-09-09 11:27:59]

pauljensen@chromium.org changed reviewers:
+ mef@chromium.org

[pauljensen, 2016-09-09 11:28:55]

pauljensen@chromium.org changed reviewers:
+ kapishnikov@chromium.org
- mef@chromium.org

[pauljensen, 2016-09-09 11:28:55]

Andrei, PTAL.

[kapishnikov, 2016-09-09 14:42:02]

On 2016/09/09 11:28:55, pauljensen wrote:
> Andrei, PTAL.

Looks good. One thing, while refactoring I have changed the LoadState class
package to "impl" and kept it in "java/src". I think the LoadState class should
not be a part of the API.

[pauljensen, 2016-09-09 18:58:58]

On 2016/09/09 14:42:02, kapishnikov wrote:
> On 2016/09/09 11:28:55, pauljensen wrote:
> > Andrei, PTAL.
> 
> Looks good. One thing, while refactoring I have changed the LoadState class
> package to "impl" and kept it in "java/src". I think the LoadState class
should
> not be a part of the API.

Ok, I put LoadState back, PTAL.

[pauljensen, 2016-09-14 11:58:38]

Ping Andrei.

[kapishnikov, 2016-09-14 17:18:13]

Paul, sorry for the late review. LGTM

[pauljensen, 2016-09-14 18:01:36]

The CQ bit was checked by pauljensen@chromium.org

[commit-bot: I haz the power, 2016-09-14 18:02:11]

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-09-14 18:53:44]

Description was changed from

==========
[Cronet] Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.

BUG=629299
==========

to

==========
[Cronet] Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.

BUG=629299
==========

[commit-bot: I haz the power, 2016-09-14 18:53:45]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-09-14 18:56:02]

Description was changed from

==========
[Cronet] Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.

BUG=629299
==========

to

==========
[Cronet] Add presubmit check to verify Cronet package division.

Cronet API classes must be in org.chromium.net package and
Cronet implementation classes must not be in org.chromium.net package.

BUG=629299

Committed: https://crrev.com/17ebec651e9b55a0dfe8d64864a0bc3592b3b4ee
Cr-Commit-Position: refs/heads/master@{#418623}
==========

[commit-bot: I haz the power, 2016-09-14 18:56:03]

Patchset 6 (id:??) landed as
https://crrev.com/17ebec651e9b55a0dfe8d64864a0bc3592b3b4ee
Cr-Commit-Position: refs/heads/master@{#418623}