Don't require DEPS OWNERS when moving lines around in a DEPS file.

Don't require DEPS OWNERS when moving lines around in a DEPS file.

As a bonus, less regex than before and also correctly handles the
'!' prefix in DEPS files now.

BUG=702851
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2768063004
Cr-Commit-Position: refs/heads/master@{#461266}
Committed: https://chromium.googlesource.com/chromium/src/+/da9479f3d2ff217723adc834581d41671c18d06c

[dcheng, 2017-03-23 05:40:25]

Please compare PS1 (original) and PS3 (with special parsing).

It feels kind of lame to just cargo cult and copy and paste this into the
PRESUBMIT, but I don't have a good idea on how to share this.

[dcheng, 2017-03-24 18:37:56]

ping =)

[dcheng, 2017-03-28 15:51:43]

ping =)

[Dirk Pranke, 2017-03-31 22:50:52]

lgtm

[dcheng, 2017-03-31 22:53:11]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2017-03-31 22:54:03]

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, 2017-03-31 23:09:43]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491000791155040,
"parent_rev": "fb9dbe97eb9b44c90b42f4c2754393e6896a3d04", "commit_rev":
"da9479f3d2ff217723adc834581d41671c18d06c"}

[commit-bot: I haz the power, 2017-03-31 23:10:31]

Description was changed from

==========
Don't require DEPS OWNERS when moving lines around in a DEPS file.

As a bonus, less regex than before and also correctly handles the
'!' prefix in DEPS files now.

BUG=702851
R=dpranke@chromium.org
==========

to

==========
Don't require DEPS OWNERS when moving lines around in a DEPS file.

As a bonus, less regex than before and also correctly handles the
'!' prefix in DEPS files now.

BUG=702851
R=dpranke@chromium.org

Review-Url: https://codereview.chromium.org/2768063004
Cr-Commit-Position: refs/heads/master@{#461266}
Committed:
https://chromium.googlesource.com/chromium/src/+/da9479f3d2ff217723adc834581d...
==========

[commit-bot: I haz the power, 2017-03-31 23:10:32]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/da9479f3d2ff217723adc834581d...

[Sam McNally, 2017-04-03 01:32:23]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2792853002/ by sammc@chromium.org.

The reason for reverting is: Breaks presubmit when adding DEPS files:
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.linux....