Revert of Skia Filter Quality and Scaling Metrics

Revert of Skia Filter Quality and Scaling Metrics (patchset #3 id:60001 of https://codereview.chromium.org/1686203002/ )

Reason for revert:
reverting this to address Ilya's comments.

Original issue's description:
> Skia Filter Quality and Scaling Metrics
> 
> Adds histogram metrics to log the filter quality and scale factor of
> each image draw. To make the data easier to consume, this is broken
> down into a number of individual histograms:
> - Filter quality across all draw calls
> - Scale amount across all draw calls
> - Scale amount per filter quality (4 histograms total)
> 
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1686203002
> 
> Committed: https://skia.googlesource.com/skia/+/f57b3a6e4a002caf01378832cbd756c6c163a783

TBR=bsalomon@google.com,isherman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Committed: https://skia.googlesource.com/skia/+/82e26bf23923f20828c850d1a0e01d8bde69cfff

[ericrk, 2016-02-25 22:53:12]

Created Revert of Skia Filter Quality and Scaling Metrics

[commit-bot: I haz the power, 2016-02-25 22:53:13]

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

[commit-bot: I haz the power, 2016-02-25 22:53:16]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-02-26 04:53 UTC

[commit-bot: I haz the power, 2016-02-25 23:02:53]

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

[Ilya Sherman, 2016-02-25 23:25:29]

On 2016/02/25 22:53:16, commit-bot: I haz the power wrote:
> Note for Reviewers:
> The CQ is waiting for an approval.
> If you believe that the CL is not ready yet, or if you would like to L-G-T-M
> with comments then please uncheck the CQ checkbox.
> 
> Waiting for LGTM from valid reviewer(s) till 2016-02-26 04:53 UTC

FWIW, I don't mind if you address my comments in a follow-up CL, rather than by
reverting.

[ericrk, 2016-02-25 23:25:54]

The CQ bit was unchecked by ericrk@chromium.org

[ericrk, 2016-02-25 23:27:42]

The CQ bit was checked by ericrk@chromium.org

[commit-bot: I haz the power, 2016-02-25 23:27:45]

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

[ericrk, 2016-02-25 23:32:04]

On 2016/02/25 23:25:29, Ilya Sherman wrote:
> On 2016/02/25 22:53:16, commit-bot: I haz the power wrote:
> > Note for Reviewers:
> > The CQ is waiting for an approval.
> > If you believe that the CL is not ready yet, or if you would like to L-G-T-M
> > with comments then please uncheck the CQ checkbox.
> > 
> > Waiting for LGTM from valid reviewer(s) till 2016-02-26 04:53 UTC
> 
> FWIW, I don't mind if you address my comments in a follow-up CL, rather than
by
> reverting.

Realized that my SkAssert with the "none" part is a bit questionable - no idea
where that came from :), and looking back, it doesn't make sense... going to
just go ahead and revert to avoid any issues and will clean up w.r.t. your other
comments and re-land.

Thanks though!

[commit-bot: I haz the power, 2016-02-26 04:53:04]

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

[commit-bot: I haz the power, 2016-02-26 04:53:05]

No LGTM from a valid reviewer yet. Please ask for an LGTM from a
full Skia committer

[fmalita_google_do_not_use, 2016-02-26 06:14:32]

The CQ bit was checked by fmalita@google.com

[commit-bot: I haz the power, 2016-02-26 06:14:44]

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

[commit-bot: I haz the power, 2016-02-26 06:14:45]

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

[commit-bot: I haz the power, 2016-02-26 06:14:46]

No LGTM from a valid reviewer yet. Please ask for an LGTM from a
full Skia committer

[fmalita_google_do_not_use, 2016-02-26 06:15:13]

lgtm

[fmalita_google_do_not_use, 2016-02-26 06:15:19]

The CQ bit was checked by fmalita@google.com

[commit-bot: I haz the power, 2016-02-26 06:15:23]

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

[commit-bot: I haz the power, 2016-02-26 06:15:33]

Description was changed from

==========
Revert of Skia Filter Quality and Scaling Metrics (patchset #3 id:60001 of
https://codereview.chromium.org/1686203002/ )

Reason for revert:
reverting this to address Ilya's comments.

Original issue's description:
> Skia Filter Quality and Scaling Metrics
> 
> Adds histogram metrics to log the filter quality and scale factor of
> each image draw. To make the data easier to consume, this is broken
> down into a number of individual histograms:
> - Filter quality across all draw calls
> - Scale amount across all draw calls
> - Scale amount per filter quality (4 histograms total)
> 
> BUG=skia:
> GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
> 
> Committed:
https://skia.googlesource.com/skia/+/f57b3a6e4a002caf01378832cbd756c6c163a783

TBR=bsalomon@google.com,isherman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
==========

to

==========
Revert of Skia Filter Quality and Scaling Metrics (patchset #3 id:60001 of
https://codereview.chromium.org/1686203002/ )

Reason for revert:
reverting this to address Ilya's comments.

Original issue's description:
> Skia Filter Quality and Scaling Metrics
> 
> Adds histogram metrics to log the filter quality and scale factor of
> each image draw. To make the data easier to consume, this is broken
> down into a number of individual histograms:
> - Filter quality across all draw calls
> - Scale amount across all draw calls
> - Scale amount per filter quality (4 histograms total)
> 
> BUG=skia:
> GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
> 
> Committed:
https://skia.googlesource.com/skia/+/f57b3a6e4a002caf01378832cbd756c6c163a783

TBR=bsalomon@google.com,isherman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Committed:
https://skia.googlesource.com/skia/+/82e26bf23923f20828c850d1a0e01d8bde69cfff
==========

[commit-bot: I haz the power, 2016-02-26 06:15:34]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/82e26bf23923f20828c850d1a0e01d8bde69cfff

[fmalita_google_do_not_use, 2016-02-26 06:17:10]

On 2016/02/26 06:14:46, commit-bot: I haz the power wrote:
> No LGTM from a valid reviewer yet. Please ask for an LGTM from a
> full Skia committer

Hrmph.  Revert not landing without a lgtm!?

FTR, the assert was triggering and blocking the roll - e.g.
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...