Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(511)

Issue 8510030: Revert 109328 - Add tests to make sure gpu blacklist actually block gpu features. (Closed)

Created:
9 years, 1 month ago by Zhenyao Mo
Modified:
9 years, 1 month ago
Reviewers:
zmo
CC:
chromium-reviews, jam, apatrick_chromium, dpranke-watch+content_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Revert 109328 - Add tests to make sure gpu blacklist actually block gpu features. The general idea is to use jbates's newly added tracing: if a feature is blacklisted, we check the tracing data to make sure gpu process hasn't launched. BUG=101222 TEST=browser_tests Review URL: http://codereview.chromium.org/8497034 TBR=zmo@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109334

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -263 lines) Patch
M chrome/chrome_tests.gypi View 1 chunk +0 lines, -1 line 0 comments Download
D chrome/test/data/gpu/feature_canvas2d.html View 1 chunk +0 lines, -35 lines 0 comments Download
D chrome/test/data/gpu/feature_compositing.html View 1 chunk +0 lines, -21 lines 0 comments Download
D chrome/test/data/gpu/feature_webgl.html View 1 chunk +0 lines, -33 lines 0 comments Download
D chrome/test/gpu/gpu_feature_browsertest.cc View 1 chunk +0 lines, -171 lines 0 comments Download
M content/browser/gpu/gpu_process_host.cc View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Zhenyao Mo
9 years, 1 month ago (2011-11-09 23:40:49 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698