Update import order CheckStyle configuration.

Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) imports from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.

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

[Bernhard Bauer, 2016-06-30 09:21:20]

Description was changed from

==========
Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) import from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.
==========

to

==========
Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) imports from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.
==========

[Bernhard Bauer, 2016-07-12 09:38:50]

bauerb@chromium.org changed reviewers:
+ aberent@chromium.org

[Bernhard Bauer, 2016-07-12 09:38:51]

Please review. Thanks!

[aberent, 2016-07-12 10:11:24]

lgtm

[Bernhard Bauer, 2016-09-09 10:21:35]

The CQ bit was checked by bauerb@chromium.org

[commit-bot: I haz the power, 2016-09-09 10:21:50]

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-09 10:27:31]

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

[commit-bot: I haz the power, 2016-09-09 10:27:32]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Bernhard Bauer, 2016-09-09 10:46:42]

bauerb@chromium.org changed reviewers:
+ jbudorick@chromium.org

[Bernhard Bauer, 2016-09-09 10:46:42]

John, could I get an OWNERS review please?

[Bernhard Bauer, 2016-10-04 14:24:09]

On 2016/09/09 10:46:42, Bernhard Bauer wrote:
> John, could I get an OWNERS review please?

Ping? :)

[jbudorick, 2016-10-04 15:12:32]

:O sorry, this got lost in my inbox somewhere.

lgtm

[Bernhard Bauer, 2016-10-04 15:30:07]

The CQ bit was checked by bauerb@chromium.org

[commit-bot: I haz the power, 2016-10-04 15:30:25]

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-10-04 16:04:34]

Description was changed from

==========
Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) imports from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.
==========

to

==========
Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) imports from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.
==========

[commit-bot: I haz the power, 2016-10-04 16:04:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-04 16:06:48]

Description was changed from

==========
Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) imports from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.
==========

to

==========
Update import order CheckStyle configuration.

The import order now separates Chromium (and downstream) imports from other
imports with com or org prefixes.

Also remove some prefixes that aren't used in Chromium code.

Finally, add an importorder file for configuring the import order in Eclipse.

Committed: https://crrev.com/ba39aa11d397ca1768ae2a702bec32aab9283770
Cr-Commit-Position: refs/heads/master@{#422818}
==========

[commit-bot: I haz the power, 2016-10-04 16:06:49]

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

[dominickn, 2016-10-05 02:12:12]

On 2016/10/04 16:06:49, commit-bot: I haz the power wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/ba39aa11d397ca1768ae2a702bec32aab9283770
> Cr-Commit-Position: refs/heads/master@{#422818}

This can result in utterly mysterious presubmit failures in the CQ if your tree
doesn't have this commit and you submit Java files with the old import ordering
(e.g. crrev.com/2351113005).

It took me quite a while to track down that checkstyle had changed. Can this be
advertised in some way? And for those of us who don't use eclim or Eclipse, is
there some canonical way to get the order right other than guesswork?