Fix remove_webcore_debug_symbols again to avoid constant building

Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty when
remove_webcore_debug_symbols is true. This change finishes the fix to
make remove_webcore_debug_symbols work properly.

R=tkent@chromium.org
BUG=647525
Review-Url: https://codereview.chromium.org/2973643003
Cr-Commit-Position: refs/heads/master@{#484672}
Committed: https://chromium.googlesource.com/chromium/src/+/961bda534caf2a184153b1458aae16be169bc951

[brucedawson, 2017-07-05 23:22:27]

The CQ bit was checked by brucedawson@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-05 23:22:43]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[brucedawson, 2017-07-05 23:23:19]

Description was changed from

==========
Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty. This change
finishes the fix to remove_webcore_debug_symbols.

R=nverne@chromium.org
BUG=647525
==========

to

==========
Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty. This change
finishes the fix to remove_webcore_debug_symbols.

R=tkent@chromium.org
BUG=647525
==========

[brucedawson, 2017-07-05 23:23:19]

brucedawson@chromium.org changed reviewers:
+ tkent@chromium.org
- nverne@chromium.org

[brucedawson, 2017-07-05 23:24:00]

Description was changed from

==========
Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty. This change
finishes the fix to remove_webcore_debug_symbols.

R=tkent@chromium.org
BUG=647525
==========

to

==========
Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty when
remove_webcore_debug_symbols is true. This change finishes the fix to
make remove_webcore_debug_symbols work properly.

R=tkent@chromium.org
BUG=647525
==========

[brucedawson, 2017-07-06 00:44:31]

Minor fix - PTAL.

[brucedawson, 2017-07-06 00:45:06]

Minor fix - PTAL.

[tkent, 2017-07-06 00:52:46]

lgtm

[commit-bot: I haz the power, 2017-07-06 03:35:16]

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

[commit-bot: I haz the power, 2017-07-06 03:35:17]

Dry run: This issue passed the CQ dry run.

[brucedawson, 2017-07-06 18:08:28]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2017-07-06 18:08: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, 2017-07-06 18:13:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499364508958770, "parent_rev":
"470a66824ca1607bac211e1bc7e31178a89d0dae", "commit_rev":
"961bda534caf2a184153b1458aae16be169bc951"}

[commit-bot: I haz the power, 2017-07-06 18:13:55]

Description was changed from

==========
Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty when
remove_webcore_debug_symbols is true. This change finishes the fix to
make remove_webcore_debug_symbols work properly.

R=tkent@chromium.org
BUG=647525
==========

to

==========
Fix remove_webcore_debug_symbols again to avoid constant building

The initial fix to remove_webcore_debug_symbols to make builds clean
when using it (crrev.com/2948363004) happened in parallel with a
change to add blink_controller (crrev.com/c/547379), so blink_controller
didn't use the new pattern, and is therefore always dirty when
remove_webcore_debug_symbols is true. This change finishes the fix to
make remove_webcore_debug_symbols work properly.

R=tkent@chromium.org
BUG=647525

Review-Url: https://codereview.chromium.org/2973643003
Cr-Commit-Position: refs/heads/master@{#484672}
Committed:
https://chromium.googlesource.com/chromium/src/+/961bda534caf2a184153b1458aae...
==========

[commit-bot: I haz the power, 2017-07-06 18:13:56]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/961bda534caf2a184153b1458aae...