Remove obsolete references to threaded-compositing and impl-side-painting flags.

content/test/gpu/gpu_tests/gpu_rasterization.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.
 
 from gpu_tests import cloud_storage_test_base
 from gpu_tests import gpu_rasterization_expectations
 import page_sets
 
 from telemetry.page import page_test
 from telemetry.util import image_util
@@ skipping 16 matching lines @@
   window.domAutomationController = domAutomationController;
 """
 
 def _DidTestSucceed(tab):
   return tab.EvaluateJavaScript('domAutomationController._succeeded')
 
 class GpuRasterizationValidator(cloud_storage_test_base.ValidatorBase):
   def CustomizeBrowserOptions(self, options):
     # --test-type=gpu is used only to suppress the "Google API Keys are missing"
     # infobar, which causes flakiness in tests.
     options.AppendExtraBrowserArgs(['--enable-threaded-compositing',

[danakj, 2016/09/26 21:39:29]

I think this should go away too

[wkorman, 2016/09/26 21:44:29]

Are you sure? It looks like this still ties to some actual logic, as
!kEnableThreadedCompositing -> kDisableThreadedAnimation ->
is_threaded_animation_enabled:

https://cs.chromium.org/chromium/src/content/renderer/render_thread_impl.cc?c...

[danakj, 2016/09/26 21:46:06]

In layout tests we disable threaded compositing unless the flag is present. It
has no meaning outside of layout tests.

[Ken Russell (switch to Gerrit), 2016/09/26 21:48:57]

+1 for removing it then. Thanks danakj@ for pointing this out. These command
line flags were chosen a long time ago, before the compositor's current
behavior.

[wkorman, 2016/09/26 22:18:54]

Done.

-                                    '--enable-impl-side-painting',
                                     '--force-gpu-rasterization',
                                     '--enable-gpu-benchmarking',
                                     '--test-type=gpu'])
 
   def ValidateAndMeasurePage(self, page, tab, results):
     if not _DidTestSucceed(tab):
       raise page_test.Failure('Page indicated a failure')
 
     if not hasattr(page, 'expectations') or not page.expectations:
       raise page_test.Failure('Expectations not specified')
@@ skipping 31 matching lines @@
     return 'gpu_rasterization'
 
   def CreateStorySet(self, options):
     story_set = page_sets.GpuRasterizationTestsStorySet(self.GetExpectations())
     for page in story_set:
       page.script_to_evaluate_on_commit = test_harness_script
     return story_set
 
   def _CreateExpectations(self):
     return gpu_rasterization_expectations.GpuRasterizationExpectations()