Bug 1165458. Raise vsync timeout limit for gtests. r=kats
authorMason Chang <mchang@mozilla.com>
Wed, 20 Jan 2016 10:29:39 -0800
changeset 280819 b8dfc7fb1398fbc20994ba73fee89a1a9fe9935c
parent 280818 b67a5343e79fef3d6e949e1c437c9a0e5435e3e7
child 280820 417708054b36fcc7fc50156303b2221775dfb6f9
push id29922
push usercbook@mozilla.com
push dateThu, 21 Jan 2016 10:51:00 +0000
treeherdermozilla-central@977d78a8dd78 [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")
   {
   }