Bug 1165458. Raise vsync timeout limit for gtests. r=kats
authorMason Chang <mchang@mozilla.com>
Wed, 20 Jan 2016 10:29:39 -0800
changeset 280753 b8dfc7fb1398fbc20994ba73fee89a1a9fe9935c
parent 280752 b67a5343e79fef3d6e949e1c437c9a0e5435e3e7
child 280754 417708054b36fcc7fc50156303b2221775dfb6f9
push id70568
push usermchang@mozilla.com
push dateWed, 20 Jan 2016 18:29:44 +0000
treeherdermozilla-inbound@b8dfc7fb1398 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskats
bugs1165458
milestone46.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1165458. Raise vsync timeout limit for gtests. r=kats
gfx/tests/gtest/TestVsync.cpp
--- a/gfx/tests/gtest/TestVsync.cpp
+++ b/gfx/tests/gtest/TestVsync.cpp
@@ -17,17 +17,18 @@
 #include "mozilla/VsyncDispatcher.h"
 
 using namespace mozilla;
 using namespace mozilla::gfx;
 using namespace mozilla::layers;
 using ::testing::_;
 
 // Timeout for vsync events to occur in milliseconds
-const int kVsyncTimeoutMS = 50;
+// Windows 8.1 has intermittents at 50 ms. Raise limit to 5 vsync intervals.
+const int kVsyncTimeoutMS = 80;
 
 class TestVsyncObserver : public VsyncObserver {
 public:
   TestVsyncObserver()
     : mDidGetVsyncNotification(false)
     , mVsyncMonitor("VsyncMonitor")
   {
   }