[subresource_filter] Remove unused png files.

[subresource_filter] Remove unused png files.

BUG=689992
Review-Url: https://codereview.chromium.org/2954673003
Cr-Commit-Position: refs/heads/master@{#481956}
Committed: https://chromium.googlesource.com/chromium/src/+/ca5afdce673a1578ad3970e9fc5c181a2646d81e

[shivanisha, 2017-06-23 17:17:48]

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

[commit-bot: I haz the power, 2017-06-23 17:18:09]

Dry run: CQ is trying da patch.

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

[shivanisha, 2017-06-23 17:19:33]

Description was changed from

==========
Removed unused png files.

BUG=689992
==========

to

==========
[subresource_filter] Remove unused png files.

BUG=689992
==========

[shivanisha, 2017-06-23 17:19:33]

shivanisha@chromium.org changed reviewers:
+ lgarron@chromium.org, thestig@chromium.org

[shivanisha, 2017-06-23 17:23:18]

lgarron@, PTAL , Thanks!
thestig@, PTAL for owner's approval, Thanks!

I would have expected git mv to have renamed the
allowed/blocked{subresource_filter}.pngs in the CL
https://codereview.chromium.org/2945113004, but not sure what's the issue but
tot shows both new and old files and the unused *subresource_filter.png files
also lead to a presubmit error for other CLs, which could be blocking others, so
it would be great if this change can land asap. Thanks.

[commit-bot: I haz the power, 2017-06-23 17:28:21]

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

[commit-bot: I haz the power, 2017-06-23 17:28:22]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-06-23 17:59:21]

lgtm

[shivanisha, 2017-06-23 18:00:26]

On 2017/06/23 at 17:23:18, shivanisha wrote:
> lgarron@, PTAL , Thanks!
> thestig@, PTAL for owner's approval, Thanks!
> 
> I would have expected git mv to have renamed the
allowed/blocked{subresource_filter}.pngs in the CL
https://codereview.chromium.org/2945113004, but not sure what's the issue but
tot shows both new and old files and the unused *subresource_filter.png files
also lead to a presubmit error for other CLs, which could be blocking others, so
it would be great if this change can land asap. Thanks.

Ah, may be there is something wrong with my branch because a fresh branch did
not give the presubmit error, which is good that it's not blocking others. This
Cl is still valid to remove the unused files, though.

[shivanisha, 2017-06-23 18:04:52]

On 2017/06/23 at 18:00:26, shivanisha wrote:
> On 2017/06/23 at 17:23:18, shivanisha wrote:
> > lgarron@, PTAL , Thanks!
> > thestig@, PTAL for owner's approval, Thanks!
> > 
> > I would have expected git mv to have renamed the
allowed/blocked{subresource_filter}.pngs in the CL
https://codereview.chromium.org/2945113004, but not sure what's the issue but
tot shows both new and old files and the unused *subresource_filter.png files
also lead to a presubmit error for other CLs, which could be blocking others, so
it would be great if this change can land asap. Thanks.
> 
> Ah, may be there is something wrong with my branch because a fresh branch did
not give the presubmit error, which is good that it's not blocking others. This
Cl is still valid to remove the unused files, though.

thestig@, Thanks!

[shivanisha, 2017-06-23 18:04:58]

The CQ bit was checked by shivanisha@chromium.org

[commit-bot: I haz the power, 2017-06-23 18:05:15]

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-06-23 18:10:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498241098163050, "parent_rev":
"98e76eb9375e2db29291163aeec9f0bdf9ff85cb", "commit_rev":
"ca5afdce673a1578ad3970e9fc5c181a2646d81e"}

[commit-bot: I haz the power, 2017-06-23 18:10:48]

Description was changed from

==========
[subresource_filter] Remove unused png files.

BUG=689992
==========

to

==========
[subresource_filter] Remove unused png files.

BUG=689992

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

[commit-bot: I haz the power, 2017-06-23 18:10:49]

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