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

Issue 11576051: Revert 173259, broke win7 tests: (Closed)

Created:
8 years ago by Nico
Modified:
8 years ago
Reviewers:
zmo
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium
Visibility:
Public.

Description

Revert 173259, broke win7 tests: http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/13892/steps/content_unittests/logs/GpuSideBlacklisting GpuDataManagerImplTest.GpuSideBlacklisting: browser\gpu\gpu_data_manager_impl_unittest.cc(147): error: Value of: manager->GpuAccessAllowed() Actual: true Expected: false browser\gpu\gpu_data_manager_impl_unittest.cc(150): error: Value of: manager->GetBlacklistedFeatures() Actual: 4 Expected: GPU_FEATURE_TYPE_WEBGL | GPU_FEATURE_TYPE_ACCELERATED_2D_CANVAS Which is: 5 > Fix a bug in r173248 > > Otherwise it will fail GPU blacklisting. > > BUG=162928, 165374 > TEST=manual about:gpu on machines with blacklisted features > TBR=kbr > Review URL: https://codereview.chromium.org/11577033 TBR=zmo@google.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=173266

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -15 lines) Patch
M content/browser/gpu/gpu_data_manager_impl.cc View 3 chunks +9 lines, -15 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Nico
8 years ago (2012-12-15 03:20:36 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698