Improve handling of DEPS OWNERS PRESUBMIT check

Improve handling of DEPS OWNERS PRESUBMIT check

BUG=593533
Committed: https://chromium.googlesource.com/chromium/src/+/ec17f88453662349b4ab73a51cf74bd416eca2da

[Bill Hesse, 2016-07-04 14:13:16]

whesse@google.com changed reviewers:
+ whesse@google.com

[Bill Hesse, 2016-07-04 14:13:17]

lgtm

https://codereview.chromium.org/2120283002/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/2120283002/diff/1/PRESUBMIT.py#newcode1106
PRESUBMIT.py:1106: '\n    '.join(suggested_owners or [])))
I wouldn't change the indents after % unless it is really done that way in all
of the other code in this project.

[Paweł Hajdan Jr., 2016-07-04 14:13:35]

phajdan.jr@chromium.org changed reviewers:
+ emso@chromium.org, sergiyb@chromium.org, tandrii@chromium.org
- whesse@google.com

[commit-bot: I haz the power, 2016-07-04 14:20:27]

Description was changed from

==========
Improve handling of DEPS OWNERS PRESUBMIT check

BUG=593533
==========

to

==========
Improve handling of DEPS OWNERS PRESUBMIT check

BUG=593533

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

[commit-bot: I haz the power, 2016-07-04 14:20:28]

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

[Paweł Hajdan Jr., 2016-07-04 14:21:24]

Description was changed from

==========
Improve handling of DEPS OWNERS PRESUBMIT check

BUG=593533

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

to

==========
Improve handling of DEPS OWNERS PRESUBMIT check

BUG=593533

Committed:
https://chromium.googlesource.com/chromium/src/+/ec17f88453662349b4ab73a51cf7...
==========

[Paweł Hajdan Jr., 2016-07-04 14:21:26]

Committed patchset #1 (id:1) manually as
ec17f88453662349b4ab73a51cf74bd416eca2da (presubmit successful).

[tandrii(chromium), 2016-07-04 17:21:39]

lgtm

https://codereview.chromium.org/2120283002/diff/1/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/2120283002/diff/1/PRESUBMIT.py#newcode1106
PRESUBMIT.py:1106: '\n    '.join(suggested_owners or [])))
On 2016/07/04 14:13:16, Bill Hesse wrote:
> I wouldn't change the indents after % unless it is really done that way in all
> of the other code in this project.

indents in depot_tools are grossly inconsistent :(