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

Unified Diff: content/test/gpu/gpu_tests/gpu_process.py

Issue 1463823002: Add DriverBugWorkaroundsInGpuProcessPage to gpu_process_test.py (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add a testing GpuChannelMsg_GetDriverBugWorkArounds IPC message and use it in a new browser test Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/test/gpu/gpu_tests/gpu_process.py
diff --git a/content/test/gpu/gpu_tests/gpu_process.py b/content/test/gpu/gpu_tests/gpu_process.py
index 5dfabdefa6d204cb1e05363e13cb36c28d60cb89..ee104972c3895608159ddc156c2a7de6a64edfe4 100644
--- a/content/test/gpu/gpu_tests/gpu_process.py
+++ b/content/test/gpu/gpu_tests/gpu_process.py
@@ -16,6 +16,23 @@ test_harness_script = r"""
}
window.domAutomationController = domAutomationController;
+
+ function VerifyDriverBugWorkarounds(workaround) {
Ken Russell (switch to Gerrit) 2016/03/30 22:17:41 nit: VerifyDriverBugWorkaroundIsPresent
Julien Isorce Samsung 2016/03/31 15:20:09 Done.
+ var browser_list = document.querySelector('.workarounds-list');
+ var gpu_list = chrome.gpuBenchmarking.getGpuDriverBugWorkarounds();
+ var found = false;
+
+ for (var i=0; i < browser_list.childElementCount; i++) {
+ var workaround_str = browser_list.children[i].textContent;
+ if (gpu_list.indexOf(workaround_str) < 0)
+ return false;
+
+ if (!found && workaround_str == workaround)
+ found = true;
+ }
+
+ return found;
+ };
"""
class GpuProcessValidator(gpu_test_base.ValidatorBase):

Powered by Google App Engine
This is Rietveld 408576698