Do not lose secondary gpus and make sure to have an active gpu on multiple gpu configurations

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):
 
   gpu_switches = ['--gpu-no-complete-info-collection',
                   '--gpu-testing-os-version',
                   '--gpu-testing-vendor-id',
                   '--gpu-testing-device-id',
+                  '--gpu-testing-secondary-vendor-id',
+                  '--gpu-testing-secondary-device-id',
                   '--gpu-testing-gl-vendor',
                   '--gpu-testing-gl-renderer',
                   '--gpu-testing-gl-version']
 
   def __init__(self, test, finder_options, story_set):
     super(GpuProcessSharedPageState, self).__init__(
       test, finder_options, story_set)
     options = finder_options.browser_options.extra_browser_args
 
     # Clear all existing gpu testing switches.
@@ skipping 172 matching lines @@
         % (workaround, browser_list))
 
     gpu_list = tab.EvaluateJavaScript( \
       'chrome.gpuBenchmarking.getGpuDriverBugWorkarounds()')
     if not workaround in gpu_list:
       print 'Test failed. Printing page contents:'
       print tab.EvaluateJavaScript('document.body.innerHTML')
       raise page_test.Failure('%s missing in GPU process workarounds: %s' \
         % (workaround, gpu_list))
 
+class IdentifyActiveGpuSharedPageState(GpuProcessSharedPageState):
+  def __init__(self, test, finder_options, story_set):
+    super(IdentifyActiveGpuSharedPageState, self).__init__(
+      test, finder_options, story_set)
+    options = finder_options.browser_options
+
+    if sys.platform in ('cygwin', 'win32') or sys.platform.startswith('linux'):
+      options.AppendExtraBrowserArgs('--gpu-testing-vendor-id=0x1002')
+      options.AppendExtraBrowserArgs('--gpu-testing-device-id=0x6779')
+      options.AppendExtraBrowserArgs('--gpu-testing-secondary-vendor-id=0x10de')
+      options.AppendExtraBrowserArgs('--gpu-testing-secondary-device-id=0x0de1')

[Ken Russell (switch to Gerrit), 2016/04/14 01:54:59]

A machine with both AMD and NVIDIA GPUs isn't a realistic scenario. Could you
change the fake data to define an Intel GPU and a discrete NVIDIA GPU? Can you
test more scenarios, like multiple cases for the GL vendor causing the
integrated and discrete GPU to be active?

[Julien Isorce Samsung, 2016/04/14 18:02:47]

Make sense I do not remember why I put AMD instead of Intel since on my laptop I
have what you said: Intel GPU and NVIDIA GPU as discrete.
Sure I can add tests for more combinations.

+      options.AppendExtraBrowserArgs('--gpu-testing-gl-vendor=nouveau')
+
+class IdentifyActiveGpuPage(gpu_test_base.PageBase):
+
+  def __init__(self, story_set, expectations):
+    super(IdentifyActiveGpuPage, self).__init__(
+      url='chrome:gpu',
+      name='GpuProcess.identify_active_gpu',
+      page_set=story_set,
+      shared_page_state_class=IdentifyActiveGpuSharedPageState,
+      expectations=expectations)
+
+  def Validate(self, tab, results):
+    basic_infos = tab.EvaluateJavaScript('browserBridge.gpuInfo.basic_info')
+    active_gpu = []
+    inactive_gpus = []
+    index = 0
+    for info in basic_infos:
+      description = info['description']
+      value = info['value']
+      if description.startswith('GPU%d' % index) and value.startswith('VENDOR'):
+        if value.endswith('*ACTIVE*'):
+          active_gpu.append(value)
+        else:
+          inactive_gpus.append(value)
+        index += 1
+
+    print active_gpu, inactive_gpus
+
+    if len(active_gpu) != 1 or \
+      active_gpu[0] != 'VENDOR = 0x10de, DEVICE= 0x0de1 *ACTIVE*':
+      raise page_test.Failure('Active GPU field is wrong %s' % active_gpu)
+
+    if len(inactive_gpus) != 1 or \
+      inactive_gpus[0] != 'VENDOR = 0x1002, DEVICE= 0x6779':
+      raise page_test.Failure('Inactive GPU field is wrong %s' % inactive_gpus)
+
 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(DriverBugWorkaroundsInGpuProcessPage(self, expectations))
+    self.AddStory(IdentifyActiveGpuPage(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