Improve the diagnostics for include order presubmit checks

Improve the diagnostics for include order presubmit checks

The information given by the presubmit checks when an include error
violation is found is not always the clearest, especially to a new
Chrome developer. The rules for determining which group a header file
goes into are not immediately obvious and the warnings don't say whether
the problem is a sort order or a group order. This change adds that
information. It also puts the URL on a separate line for easier
selection, especially important on Windows.

The old output looks like this:

** Presubmit Messages **
Your #include order seems to be broken. Remember to use the right collation (LC_COLLATE=C) and check https://google-styleguide.googlecode.com

m/svn/trunk/cppguide.html#Names_and_Order_of_Includes
      chrome\browser\task_manager\task_manager.cc:10 \
      chrome\browser\task_manager\task_manager.cc:11

The new output looks like this:

** Presubmit Messages **
Your #include order seems to be broken. Remember to use the right collation (LC_COLLATE=C) and check
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Names_and_Order_of_Includes
      chrome\browser\task_manager\task_manager.cc:10: - c++ system include file in wrong block \
      chrome\browser\task_manager\task_manager.cc:11: - line belongs before previous line

Notice the explanations for why a line is highlighted, and the
non-wordwrapped URL.

R=maruel@chromium.org

Committed: https://crrev.com/70fadb041abae3242d2c00066ff806f47a98aecc
Cr-Commit-Position: refs/heads/master@{#336813}

[brucedawson, 2015-06-29 23:17:30]

How does this change look? It would have saved me a lot of time and
investigation.

Note that it currently fails presubmit checks (ironic!) due to a bug in
__main__.UserMetricsActionTest in PRESUBMIT.py. That failure also shows if you
run PRESUBMIT.py with no changes present. It comes from
https://codereview.chromium.org/966703003/ and is being separately investigated.

[Marc-Antoine Ruel (Google), 2015-06-29 23:35:42]

maruel@google.com changed reviewers:
+ jochen@chromium.org, phajdan.jr@chromium.org
- maruel@chromium.org

[Marc-Antoine Ruel (Google), 2015-06-29 23:35:43]

Redirecting as I'm not an owner anymore.

[jochen (gone - plz use gerrit), 2015-06-30 07:58:47]

lgtm

[Paweł Hajdan Jr., 2015-06-30 07:58:56]

LGTM

[brucedawson, 2015-06-30 17:32:04]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2015-06-30 17:33:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1213113004/1

[commit-bot: I haz the power, 2015-06-30 17:48:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-30 17:48:54]

Patchset 1 (id:??) landed as
https://crrev.com/70fadb041abae3242d2c00066ff806f47a98aecc
Cr-Commit-Position: refs/heads/master@{#336813}