Title: [226590] branches/safari-604.5.100-branch
- Revision
- 226590
- Author
- [email protected]
- Date
- 2018-01-08 16:07:10 -0800 (Mon, 08 Jan 2018)
Log Message
Apply patch. rdar://problem/36257610
Reduce the precision of "high" resolution time to 1ms
https://bugs.webkit.org/show_bug.cgi?id=180910
<rdar://problem/36085943>
Reviewed by Saam Barati.
Add a flaky test expectation to imported/w3c/web-platform-tests/workers/worker-performance.worker.html
due to a test bug. The test fix is tracked in https://github.com/w3c/web-platform-tests/pull/8711
* TestExpectations:
Modified Paths
Diff
Modified: branches/safari-604.5.100-branch/LayoutTests/ChangeLog (226589 => 226590)
--- branches/safari-604.5.100-branch/LayoutTests/ChangeLog 2018-01-09 00:07:06 UTC (rev 226589)
+++ branches/safari-604.5.100-branch/LayoutTests/ChangeLog 2018-01-09 00:07:10 UTC (rev 226590)
@@ -1,5 +1,33 @@
2017-12-18 Jason Marcell <[email protected]>
+ Apply patch. rdar://problem/36257610
+
+ Reduce the precision of "high" resolution time to 1ms
+ https://bugs.webkit.org/show_bug.cgi?id=180910
+ <rdar://problem/36085943>
+
+ Reviewed by Saam Barati.
+
+ Add a flaky test expectation to imported/w3c/web-platform-tests/workers/worker-performance.worker.html
+ due to a test bug. The test fix is tracked in https://github.com/w3c/web-platform-tests/pull/8711
+
+ * TestExpectations:
+
+ 2017-12-18 Ryosuke Niwa <[email protected]>
+
+ Reduce the precision of "high" resolution time to 1ms
+ https://bugs.webkit.org/show_bug.cgi?id=180910
+ <rdar://problem/36085943>
+
+ Reviewed by Saam Barati.
+
+ Add a flaky test expectation to imported/w3c/web-platform-tests/workers/worker-performance.worker.html
+ due to a test bug. The test fix is tracked in https://github.com/w3c/web-platform-tests/pull/8711
+
+ * TestExpectations:
+
+2017-12-18 Jason Marcell <[email protected]>
+
Cherry-pick r226065. rdar://problem/36080415
2017-12-18 Zalan Bujtas <[email protected]>
Modified: branches/safari-604.5.100-branch/LayoutTests/TestExpectations (226589 => 226590)
--- branches/safari-604.5.100-branch/LayoutTests/TestExpectations 2018-01-09 00:07:06 UTC (rev 226589)
+++ branches/safari-604.5.100-branch/LayoutTests/TestExpectations 2018-01-09 00:07:10 UTC (rev 226590)
@@ -1404,3 +1404,6 @@
webkit.org/b/177997 webgl/1.0.2/conformance/textures/copy-tex-image-2d-formats.html [ Pass Failure ]
webkit.org/b/174284 media/video-source-before-src.html [ Skip ]
+
+# There is a bug in the test
+imported/w3c/web-platform-tests/workers/worker-performance.worker.html [ Pass Failure ]
Modified: branches/safari-604.5.100-branch/Source/WebCore/ChangeLog (226589 => 226590)
--- branches/safari-604.5.100-branch/Source/WebCore/ChangeLog 2018-01-09 00:07:06 UTC (rev 226589)
+++ branches/safari-604.5.100-branch/Source/WebCore/ChangeLog 2018-01-09 00:07:10 UTC (rev 226590)
@@ -1,5 +1,37 @@
2017-12-18 Jason Marcell <[email protected]>
+ Apply patch. rdar://problem/36257610
+
+ Reduce the precision of "high" resolution time to 1ms
+ https://bugs.webkit.org/show_bug.cgi?id=180910
+ <rdar://problem/36085943>
+
+ Reviewed by Saam Barati.
+
+ Add a flaky test expectation to imported/w3c/web-platform-tests/workers/worker-performance.worker.html
+ due to a test bug. The test fix is tracked in https://github.com/w3c/web-platform-tests/pull/8711
+
+ * TestExpectations:
+
+ 2017-12-18 Ryosuke Niwa <[email protected]>
+
+ Reduce the precision of "high" resolution time to 1ms
+ https://bugs.webkit.org/show_bug.cgi?id=180910
+ <rdar://problem/36085943>
+
+ Reviewed by Saam Barati.
+
+ Reduced the high prevision time's resolution to 1ms, the same precision as Date.now().
+
+ Also fixed the bug in fillRTCStats that we weren't reducing the time resolution in RTCStats dictionaries.
+
+ * Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:
+ (WebCore::fillRTCStats):
+ * page/Performance.cpp:
+ (WebCore::Performance::reduceTimeResolution):
+
+2017-12-18 Jason Marcell <[email protected]>
+
Cherry-pick r226065. rdar://problem/36080415
2017-12-18 Zalan Bujtas <[email protected]>
Modified: branches/safari-604.5.100-branch/Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp (226589 => 226590)
--- branches/safari-604.5.100-branch/Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp 2018-01-09 00:07:06 UTC (rev 226589)
+++ branches/safari-604.5.100-branch/Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp 2018-01-09 00:07:10 UTC (rev 226590)
@@ -35,6 +35,7 @@
#include "Logging.h"
#include "MediaStreamEvent.h"
#include "NotImplemented.h"
+#include "Performance.h"
#include "PlatformStrategies.h"
#include "RTCDataChannel.h"
#include "RTCDataChannelEvent.h"
@@ -50,6 +51,7 @@
#include <webrtc/p2p/base/basicpacketsocketfactory.h>
#include <webrtc/p2p/client/basicportallocator.h>
#include <webrtc/pc/peerconnectionfactory.h>
+#include <wtf/CurrentTime.h>
#include <wtf/MainThread.h>
#include "CoreMediaSoftLink.h"
@@ -303,7 +305,7 @@
static inline void fillRTCStats(RTCStatsReport::Stats& stats, const webrtc::RTCStats& rtcStats)
{
- stats.timestamp = rtcStats.timestamp_us() / 1000.0;
+ stats.timestamp = Performance::reduceTimeResolution(Seconds::fromMicroseconds(rtcStats.timestamp_us())).milliseconds();
stats.id = fromStdString(rtcStats.id());
}
Modified: branches/safari-604.5.100-branch/Source/WebCore/page/Performance.cpp (226589 => 226590)
--- branches/safari-604.5.100-branch/Source/WebCore/page/Performance.cpp 2018-01-09 00:07:06 UTC (rev 226589)
+++ branches/safari-604.5.100-branch/Source/WebCore/page/Performance.cpp 2018-01-09 00:07:10 UTC (rev 226590)
@@ -79,7 +79,7 @@
Seconds Performance::reduceTimeResolution(Seconds seconds)
{
- double resolution = (100_us).seconds();
+ double resolution = (1000_us).seconds();
double reduced = std::floor(seconds.seconds() / resolution) * resolution;
return Seconds(reduced);
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
