Cleanup logic that generates about:gpu feature status.

Cleanup logic that generates about:gpu feature status.

Code cleanup for the logic that generates the labels used by the about:gpu page
to report on the feature status. Also separates (and fixes) reporting for impl-side-painting
and gpu rasterization.

BUG=386420
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278541

[vangelis, 2014-06-19 01:27:11]

Mo, can you please take a look?

[Zhenyao Mo, 2014-06-19 01:42:45]

LGTM

https://codereview.chromium.org/344653004/diff/1/content/browser/resources/gp...
File content/browser/resources/gpu/info_view.js (right):

https://codereview.chromium.org/344653004/diff/1/content/browser/resources/gp...
content/browser/resources/gpu/info_view.js:154: // GPU Switching
Actually gpu switching is moved to workaround list, so they will no longer
appear here.  We should just remove them.  (Sorry it slipped out of my radar
before).

[vangelis, 2014-06-19 02:10:31]

On 2014/06/19 01:42:45, Zhenyao Mo wrote:
> LGTM
> 
>
https://codereview.chromium.org/344653004/diff/1/content/browser/resources/gp...
> File content/browser/resources/gpu/info_view.js (right):
> 
>
https://codereview.chromium.org/344653004/diff/1/content/browser/resources/gp...
> content/browser/resources/gpu/info_view.js:154: // GPU Switching
> Actually gpu switching is moved to workaround list, so they will no longer
> appear here.  We should just remove them.  (Sorry it slipped out of my radar
> before).

Cool. Took care of that!

[Zhenyao Mo, 2014-06-19 16:52:01]

https://codereview.chromium.org/344653004/diff/20001/content/browser/resource...
File content/browser/resources/gpu/info_view.js (right):

https://codereview.chromium.org/344653004/diff/20001/content/browser/resource...
content/browser/resources/gpu/info_view.js:152: // GPU Switching
What I mean is, you can get rid of the below three status.

[vangelis, 2014-06-19 16:59:11]

On 2014/06/19 16:52:01, Zhenyao Mo wrote:
>
https://codereview.chromium.org/344653004/diff/20001/content/browser/resource...
> File content/browser/resources/gpu/info_view.js (right):
> 
>
https://codereview.chromium.org/344653004/diff/20001/content/browser/resource...
> content/browser/resources/gpu/info_view.js:152: // GPU Switching
> What I mean is, you can get rid of the below three status.

Done!

[Zhenyao Mo, 2014-06-19 17:54:31]

LGTM again.  Thank you.

[vangelis, 2014-06-19 18:21:06]

The CQ bit was checked by vangelis@google.com

[commit-bot: I haz the power, 2014-06-19 18:22:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vangelis@chromium.org/344653004/40001

[commit-bot: I haz the power, 2014-06-19 23:02:08]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/8...)
  linux_chromium_chromeos_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)

[commit-bot: I haz the power, 2014-06-19 23:05:50]

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

[commit-bot: I haz the power, 2014-06-19 23:05:51]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/8...)

[vangelis, 2014-06-19 23:08:02]

The CQ bit was checked by vangelis@google.com

[commit-bot: I haz the power, 2014-06-19 23:10:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/vangelis@chromium.org/344653004/40001

[commit-bot: I haz the power, 2014-06-20 00:09:02]

Change committed as 278541