Add DriverBugWorkaroundsInGpuProcessPage to gpu_process_test.py

content/test/gpu/page_sets/gpu_process_tests.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 import sys
 from telemetry.story import story_set as story_set_module
 from telemetry.page import page_test
 
 from gpu_tests import gpu_test_base
 
 class GpuProcessSharedPageState(gpu_test_base.GpuSharedPageState):
@@ skipping 151 matching lines @@
       page_set=story_set,
       shared_page_state_class=SkipGpuProcessSharedPageState,
       expectations=expectations)
 
   def Validate(self, tab, results):
     has_gpu_process_js = 'chrome.gpuBenchmarking.hasGpuProcess()'
     has_gpu_process = tab.EvaluateJavaScript(has_gpu_process_js)
     if has_gpu_process:
       raise page_test.Failure('GPU process detected')
 
+
+class DriverBugWorkaroundsSharedPageState(GpuProcessSharedPageState):
+  def __init__(self, test, finder_options, story_set):
+    super(DriverBugWorkaroundsSharedPageState, self).__init__(
+      test, finder_options, story_set)
+    options = finder_options.browser_options
+    options.AppendExtraBrowserArgs('--use_testing_gpu_driver_workaround')

[Ken Russell (switch to Gerrit), 2016/03/30 22:17:42]

Update per naming nit below.

[Julien Isorce Samsung, 2016/03/31 15:20:09]

Done.

+
+
+class DriverBugWorkaroundsGpuProcessPage(gpu_test_base.PageBase):

[Ken Russell (switch to Gerrit), 2016/03/30 22:17:41]

nit: DriverBugWorkaroundsInGpuProcessPage

[Julien Isorce Samsung, 2016/03/31 15:20:09]

Done.

+  def __init__(self, story_set, expectations):
+    super(DriverBugWorkaroundsGpuProcessPage, self).__init__(
+      url='chrome:gpu',
+      name='GpuProcess.driver_bug_workarounds_gpu_process',

[Ken Russell (switch to Gerrit), 2016/03/30 22:17:41]

nit: driver_bug_workarounds_in_gpu_process

[Julien Isorce Samsung, 2016/03/31 15:20:09]

Done.

+      page_set=story_set,
+      shared_page_state_class=DriverBugWorkaroundsSharedPageState,
+      expectations=expectations)
+
+  def Validate(self, tab, results):
+    if not tab.EvaluateJavaScript('VerifyDriverBugWorkarounds("%s")'
+        % 'use_testing_gpu_driver_workaround'):

[Ken Russell (switch to Gerrit), 2016/03/30 22:17:41]

Update name.

[Julien Isorce Samsung, 2016/03/31 15:20:09]

Done.

+      print 'Test failed. Printing page contents:'
+      print tab.EvaluateJavaScript('document.body.innerHTML')
+      raise page_test.Failure('browser and gpu process list of driver bug \

[Ken Russell (switch to Gerrit), 2016/03/30 22:17:42]

Grammar nit:
"Browser\'s and GPU process\'s lists of driver bug workarounds are not equal"

[Julien Isorce Samsung, 2016/03/31 15:20:09]

Done.

+        workarounds are not equals')

[Ken Russell (switch to Gerrit), 2016/03/30 22:17:41]

Is this really what this test verifies -- that the browser process's and GPU
process's notions of the driver bug workarounds are equal? It seems to be
testing the GPU process's notions, via the new IPC, but it doesn't look to me
like it verifies that the browser's notion is the same.

[Julien Isorce Samsung, 2016/03/31 15:20:09]

Let me explain why I think it does, but maybe I am mistaken:
There is only one GpuDataManager and it leaves in the Browser Process. When
going to chrome://gpu page,
gpu_internal_ui.cc::GpuMessageHandler::OnGpuInfoUpdate() is triggered and calls
GpuDataManagerImpl::GetInstance()->GetDriverBugWorkarounds() and passes it to
web_ui()->CallJavascriptFunction(). So the list returned by above
GetDriverBugWorkarounds() is exactly what we see in the page chrome://gpu.
Then looking at GpuDataManagerImplPrivate::GetDriverBugWorkarounds() it actually
returns an equivalent to gpu_driver_bugs_ (which was created with
gpu_driver_bug_list_->MakeDecision(gpu_info_)).
So for me the comment really says what this test verifies. Ah also
VerifyDriverBugWorkaroundIsPresent does 2 things, browser/gpu list comparisons
and testing_workaround presence.

+
+
 class GpuProcessTestsStorySet(story_set_module.StorySet):
 
   """ Tests that accelerated content triggers the creation of a GPU process """
 
   def __init__(self, expectations):
     super(GpuProcessTestsStorySet, self).__init__(
       serving_dirs=set(['../../../../content/test/data']))
 
     urls_and_names_list = [
       ('file://../../data/gpu/functional_canvas_demo.html',
        'GpuProcess.canvas2d'),
       ('file://../../data/gpu/functional_3d_css.html',
        'GpuProcess.css3d'),
       ('file://../../data/gpu/functional_webgl.html',
        'GpuProcess.webgl')
     ]
 
     for url, name in urls_and_names_list:
       self.AddStory(GpuProcessTestsPage(url, name, self, expectations))
 
     self.AddStory(FunctionalVideoPage(self, expectations))
     self.AddStory(GpuInfoCompletePage(self, expectations))
     self.AddStory(NoGpuProcessPage(self, expectations))
     self.AddStory(SoftwareGpuProcessPage(self, expectations))
     self.AddStory(SkipGpuProcessPage(self, expectations))
+    self.AddStory(DriverBugWorkaroundsGpuProcessPage(self, expectations))
 
   @property
   def allow_mixed_story_states(self):
     # Return True here in order to be able to run pages with different browser
     # command line arguments.
     return True