Diff
Modified: trunk/LayoutTests/ChangeLog (130379 => 130380)
--- trunk/LayoutTests/ChangeLog 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/LayoutTests/ChangeLog 2012-10-04 10:08:00 UTC (rev 130380)
@@ -1,3 +1,15 @@
+2012-10-04 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r130377.
+ http://trac.webkit.org/changeset/130377
+ https://bugs.webkit.org/show_bug.cgi?id=98392
+
+ Chromium Win compilation is broken (Requested by yurys on
+ #webkit).
+
+ * fast/mediastream/RTCPeerConnection-stats-expected.txt:
+ * fast/mediastream/RTCPeerConnection-stats.html:
+
2012-10-04 Kent Tamura <[email protected]>
Some picker test for input[type=date] are failing
Modified: trunk/LayoutTests/fast/mediastream/RTCPeerConnection-stats-expected.txt (130379 => 130380)
--- trunk/LayoutTests/fast/mediastream/RTCPeerConnection-stats-expected.txt 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/LayoutTests/fast/mediastream/RTCPeerConnection-stats-expected.txt 2012-10-04 10:08:00 UTC (rev 130380)
@@ -14,7 +14,6 @@
PASS statsHandler2 was called
PASS result.length is >= 2
PASS local.length is 1
-PASS timestamp is >= startTime
PASS local[0].stat("type") is "audio"
PASS successfullyParsed is true
Modified: trunk/LayoutTests/fast/mediastream/RTCPeerConnection-stats.html (130379 => 130380)
--- trunk/LayoutTests/fast/mediastream/RTCPeerConnection-stats.html 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/LayoutTests/fast/mediastream/RTCPeerConnection-stats.html 2012-10-04 10:08:00 UTC (rev 130380)
@@ -9,7 +9,6 @@
var pc = null;
var result;
-var timestamp;
var local;
function getUserMedia(dictionary, callback) {
@@ -50,13 +49,10 @@
shouldBeGreaterThanOrEqual('result.length', '2');
local = result[0].local();
shouldBe('local.length', '1');
- timestamp = local[0].timestamp;
- shouldBeGreaterThanOrEqual('timestamp', 'startTime');
shouldBe('local[0].stat("type")', '"audio"');
finishJSTest();
}
-var startTime = new Date().getTime();
shouldNotThrow('pc = new webkitRTCPeerConnection(null)');
shouldNotThrow('pc.getStats(statsHandler1)');
Modified: trunk/Source/Platform/ChangeLog (130379 => 130380)
--- trunk/Source/Platform/ChangeLog 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/Platform/ChangeLog 2012-10-04 10:08:00 UTC (rev 130380)
@@ -1,3 +1,15 @@
+2012-10-04 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r130377.
+ http://trac.webkit.org/changeset/130377
+ https://bugs.webkit.org/show_bug.cgi?id=98392
+
+ Chromium Win compilation is broken (Requested by yurys on
+ #webkit).
+
+ * chromium/public/WebRTCStatsResponse.h:
+ (WebRTCStatsResponse):
+
2012-10-04 Harald Alvestrand <[email protected]>
Change RTCPeerConnection GetStats to use Date timestamp format
Modified: trunk/Source/Platform/chromium/public/WebRTCStatsResponse.h (130379 => 130380)
--- trunk/Source/Platform/chromium/public/WebRTCStatsResponse.h 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/Platform/chromium/public/WebRTCStatsResponse.h 2012-10-04 10:08:00 UTC (rev 130380)
@@ -52,7 +52,7 @@
WEBKIT_EXPORT void reset();
WEBKIT_EXPORT size_t addReport();
- WEBKIT_EXPORT size_t addElement(size_t report, bool isLocal, double timestamp);
+ WEBKIT_EXPORT size_t addElement(size_t report, bool isLocal, long timestamp);
WEBKIT_EXPORT void addStatistic(size_t report, bool isLocal, size_t element, WebString name, WebString value);
#if WEBKIT_IMPLEMENTATION
Modified: trunk/Source/WebCore/ChangeLog (130379 => 130380)
--- trunk/Source/WebCore/ChangeLog 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/ChangeLog 2012-10-04 10:08:00 UTC (rev 130380)
@@ -1,3 +1,32 @@
+2012-10-04 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r130377.
+ http://trac.webkit.org/changeset/130377
+ https://bugs.webkit.org/show_bug.cgi?id=98392
+
+ Chromium Win compilation is broken (Requested by yurys on
+ #webkit).
+
+ * Modules/mediastream/RTCStatsElement.cpp:
+ (WebCore::RTCStatsElement::create):
+ (WebCore::RTCStatsElement::RTCStatsElement):
+ * Modules/mediastream/RTCStatsElement.h:
+ (RTCStatsElement):
+ (WebCore::RTCStatsElement::timestamp):
+ * Modules/mediastream/RTCStatsElement.idl:
+ * Modules/mediastream/RTCStatsReport.cpp:
+ (WebCore::RTCStatsReport::addElement):
+ * Modules/mediastream/RTCStatsReport.h:
+ (RTCStatsReport):
+ * Modules/mediastream/RTCStatsResponse.cpp:
+ (WebCore::RTCStatsResponse::addElement):
+ * Modules/mediastream/RTCStatsResponse.h:
+ (RTCStatsResponse):
+ * platform/chromium/support/WebRTCStatsResponse.cpp:
+ (WebKit::WebRTCStatsResponse::addElement):
+ * platform/mediastream/RTCStatsResponseBase.h:
+ (RTCStatsResponseBase):
+
2012-10-04 Harald Alvestrand <[email protected]>
Change RTCPeerConnection GetStats to use Date timestamp format
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.cpp (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.cpp 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.cpp 2012-10-04 10:08:00 UTC (rev 130380)
@@ -32,12 +32,12 @@
namespace WebCore {
-PassRefPtr<RTCStatsElement> RTCStatsElement::create(double timestamp)
+PassRefPtr<RTCStatsElement> RTCStatsElement::create(long timestamp)
{
return adoptRef(new RTCStatsElement(timestamp));
}
-RTCStatsElement::RTCStatsElement(double timestamp)
+RTCStatsElement::RTCStatsElement(long timestamp)
: m_timestamp(timestamp)
{
}
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.h (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.h 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.h 2012-10-04 10:08:00 UTC (rev 130380)
@@ -34,15 +34,15 @@
class RTCStatsElement : public RefCounted<RTCStatsElement> {
public:
- static PassRefPtr<RTCStatsElement> create(double timestamp);
- double timestamp() const { return m_timestamp; }
+ static PassRefPtr<RTCStatsElement> create(long timestamp);
+ long timestamp() const { return m_timestamp; }
String stat(const String& name) const;
void addStatistic(const String& name, const String& value);
private:
- explicit RTCStatsElement(double timestamp);
+ explicit RTCStatsElement(long timestamp);
- double m_timestamp;
+ long m_timestamp;
HashMap<String, String> m_stats;
};
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.idl (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.idl 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsElement.idl 2012-10-04 10:08:00 UTC (rev 130380)
@@ -27,7 +27,7 @@
interface [
Conditional=MEDIA_STREAM,
] RTCStatsElement {
- readonly attribute Date timestamp;
+ readonly attribute long timestamp;
DOMString stat(in DOMString name);
};
}
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsReport.cpp (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsReport.cpp 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsReport.cpp 2012-10-04 10:08:00 UTC (rev 130380)
@@ -39,7 +39,7 @@
{
}
-size_t RTCStatsReport::addElement(bool isLocal, double timestamp)
+size_t RTCStatsReport::addElement(bool isLocal, long timestamp)
{
if (isLocal) {
m_local.append(RTCStatsElement::create(timestamp));
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsReport.h (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsReport.h 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsReport.h 2012-10-04 10:08:00 UTC (rev 130380)
@@ -40,7 +40,7 @@
const Vector<RefPtr<RTCStatsElement> >& local() const { return m_local; }
const Vector<RefPtr<RTCStatsElement> >& remote() const { return m_remote; }
- size_t addElement(bool isLocal, double timestamp);
+ size_t addElement(bool isLocal, long timestamp);
void addStatistic(bool isLocal, size_t element, String name, String value);
private:
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.cpp (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.cpp 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.cpp 2012-10-04 10:08:00 UTC (rev 130380)
@@ -45,7 +45,7 @@
return m_result.size() - 1;
}
-size_t RTCStatsResponse::addElement(size_t report, bool isLocal, double timestamp)
+size_t RTCStatsResponse::addElement(size_t report, bool isLocal, long timestamp)
{
ASSERT(report >= 0 && report < m_result.size());
return m_result[report]->addElement(isLocal, timestamp);
Modified: trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.h (130379 => 130380)
--- trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.h 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/Modules/mediastream/RTCStatsResponse.h 2012-10-04 10:08:00 UTC (rev 130380)
@@ -45,7 +45,7 @@
const Vector<RefPtr<RTCStatsReport> >& result() const { return m_result; };
virtual size_t addReport() OVERRIDE;
- virtual size_t addElement(size_t report, bool isLocal, double timestamp) OVERRIDE;
+ virtual size_t addElement(size_t report, bool isLocal, long timestamp) OVERRIDE;
virtual void addStatistic(size_t report, bool isLocal, size_t element, String name, String value) OVERRIDE;
private:
Modified: trunk/Source/WebCore/platform/chromium/support/WebRTCStatsResponse.cpp (130379 => 130380)
--- trunk/Source/WebCore/platform/chromium/support/WebRTCStatsResponse.cpp 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/platform/chromium/support/WebRTCStatsResponse.cpp 2012-10-04 10:08:00 UTC (rev 130380)
@@ -60,7 +60,7 @@
return m_private->addReport();
}
-size_t WebRTCStatsResponse::addElement(size_t report, bool isLocal, double timestamp)
+size_t WebRTCStatsResponse::addElement(size_t report, bool isLocal, long timestamp)
{
return m_private->addElement(report, isLocal, timestamp);
}
Modified: trunk/Source/WebCore/platform/mediastream/RTCStatsResponseBase.h (130379 => 130380)
--- trunk/Source/WebCore/platform/mediastream/RTCStatsResponseBase.h 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Source/WebCore/platform/mediastream/RTCStatsResponseBase.h 2012-10-04 10:08:00 UTC (rev 130380)
@@ -38,7 +38,7 @@
virtual ~RTCStatsResponseBase() { }
virtual size_t addReport() = 0;
- virtual size_t addElement(size_t report, bool isLocal, double timestamp) = 0;
+ virtual size_t addElement(size_t report, bool isLocal, long timestamp) = 0;
virtual void addStatistic(size_t report, bool isLocal, size_t element, String name, String value) = 0;
};
Modified: trunk/Tools/ChangeLog (130379 => 130380)
--- trunk/Tools/ChangeLog 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Tools/ChangeLog 2012-10-04 10:08:00 UTC (rev 130380)
@@ -1,3 +1,15 @@
+2012-10-04 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r130377.
+ http://trac.webkit.org/changeset/130377
+ https://bugs.webkit.org/show_bug.cgi?id=98392
+
+ Chromium Win compilation is broken (Requested by yurys on
+ #webkit).
+
+ * DumpRenderTree/chromium/MockWebRTCPeerConnectionHandler.cpp:
+ (MockWebRTCPeerConnectionHandler::getStats):
+
2012-10-04 Harald Alvestrand <[email protected]>
Change RTCPeerConnection GetStats to use Date timestamp format
Modified: trunk/Tools/DumpRenderTree/chromium/MockWebRTCPeerConnectionHandler.cpp (130379 => 130380)
--- trunk/Tools/DumpRenderTree/chromium/MockWebRTCPeerConnectionHandler.cpp 2012-10-04 09:51:44 UTC (rev 130379)
+++ trunk/Tools/DumpRenderTree/chromium/MockWebRTCPeerConnectionHandler.cpp 2012-10-04 10:08:00 UTC (rev 130380)
@@ -43,7 +43,6 @@
#include <public/WebRTCVoidRequest.h>
#include <public/WebString.h>
#include <public/WebVector.h>
-#include <sys/time.h>
using namespace WebKit;
@@ -216,15 +215,12 @@
void MockWebRTCPeerConnectionHandler::getStats(const WebRTCStatsRequest& request)
{
WebRTCStatsResponse response = request.createResponse();
- struct timeval currentTime;
- gettimeofday(¤tTime, 0);
- double currentDate = currentTime.tv_sec * 1000.0 + currentTime.tv_usec / 1000;
for (int i = 0; i < m_streamCount; ++i) {
size_t reportIndex = response.addReport();
- size_t elementIndex = response.addElement(reportIndex, true, currentDate);
+ size_t elementIndex = response.addElement(reportIndex, true, 12345);
response.addStatistic(reportIndex, true, elementIndex, "type", "audio");
reportIndex = response.addReport();
- elementIndex = response.addElement(reportIndex, true, currentDate);
+ elementIndex = response.addElement(reportIndex, true, 12345);
response.addStatistic(reportIndex, true, elementIndex, "type", "video");
}
postTask(new RTCStatsRequestSucceededTask(this, request, response));
