In about:gpu page, make sure feature status and problem description match

In about:gpu page, make sure feature status and problem description match

BUG=405354
TEST=about:gpu page for Win/NVIDIA with driver version forced to "", gpu_unittests
R=kbr@chromium.org

Committed: https://crrev.com/3f0a6c825a02f72c9bde6d2f074010d090590b7a
Cr-Commit-Position: refs/heads/master@{#292103}

[Zhenyao Mo, 2014-08-26 01:54:55]

Please take a look.

[Ken Russell (switch to Gerrit), 2014-08-26 02:31:18]

Could you please add a test for this case?

[Zhenyao Mo, 2014-08-26 02:45:15]

please take another look.

https://codereview.chromium.org/503243002/diff/20001/gpu/config/gpu_control_l...
File gpu/config/gpu_control_list_unittest.cc (right):

https://codereview.chromium.org/503243002/diff/20001/gpu/config/gpu_control_l...
gpu/config/gpu_control_list_unittest.cc:327: EXPECT_EQ(0u,
decision_entries.size());
Before the change, this test fails (returns 1 instead of 0);
After the change, this test passes.

[Ken Russell (switch to Gerrit), 2014-08-26 20:49:21]

LGTM

https://codereview.chromium.org/503243002/diff/20001/gpu/config/gpu_control_l...
File gpu/config/gpu_control_list_unittest.cc (right):

https://codereview.chromium.org/503243002/diff/20001/gpu/config/gpu_control_l...
gpu/config/gpu_control_list_unittest.cc:327: EXPECT_EQ(0u,
decision_entries.size());
On 2014/08/26 02:45:15, Zhenyao Mo wrote:
> Before the change, this test fails (returns 1 instead of 0);
> After the change, this test passes.

OK, sounds good. But should the test also verify the transition from
needs_more_info() returning true to it returning false?

[Zhenyao Mo, 2014-08-27 03:44:06]

https://codereview.chromium.org/503243002/diff/20001/gpu/config/gpu_control_l...
File gpu/config/gpu_control_list_unittest.cc (right):

https://codereview.chromium.org/503243002/diff/20001/gpu/config/gpu_control_l...
gpu/config/gpu_control_list_unittest.cc:327: EXPECT_EQ(0u,
decision_entries.size());
On 2014/08/26 20:49:21, Ken Russell wrote:
> On 2014/08/26 02:45:15, Zhenyao Mo wrote:
> > Before the change, this test fails (returns 1 instead of 0);
> > After the change, this test passes.
> 
> OK, sounds good. But should the test also verify the transition from
> needs_more_info() returning true to it returning false?

Done.

[Zhenyao Mo, 2014-08-27 03:44:09]

The CQ bit was checked by zmo@chromium.org

[Zhenyao Mo, 2014-08-27 03:44:14]

The CQ bit was checked by zmo@chromium.org

[commit-bot: I haz the power, 2014-08-27 03:45:06]

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

[commit-bot: I haz the power, 2014-08-27 04:56:13]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-27 06:41:27]

Committed patchset #3 (40001) as 395912708fc8ecc2882b3441bbcb5cb2618b62b1

[Zhenyao Mo, 2014-08-27 19:43:11]

Patchset #4 (id:60001) has been deleted

[commit-bot: I haz the power, 2014-09-10 02:49:55]

Patchset 4 (id:??) landed as
https://crrev.com/3f0a6c825a02f72c9bde6d2f074010d090590b7a
Cr-Commit-Position: refs/heads/master@{#292103}