From 56cb90a482737ae5252ac8b7ccbc0aff0b309410 Mon Sep 17 00:00:00 2001 From: Moonchild Date: Mon, 4 May 2020 13:11:28 +0000 Subject: Issue #1517 - Remove dom.event.highrestimestamp.enabled pref This resolves #1517 --- dom/events/Event.cpp | 14 -------------- dom/events/test/test_eventTimeStamp.html | 6 ------ dom/smil/test/test_smilTimeEvents.xhtml | 12 ++++-------- 3 files changed, 4 insertions(+), 28 deletions(-) (limited to 'dom') diff --git a/dom/events/Event.cpp b/dom/events/Event.cpp index 280e40ad5..91417499b 100755 --- a/dom/events/Event.cpp +++ b/dom/events/Event.cpp @@ -44,9 +44,6 @@ extern bool IsCurrentThreadRunningChromeWorker(); static char *sPopupAllowedEvents; -static bool sReturnHighResTimeStamp = false; -static bool sReturnHighResTimeStampIsSet = false; - Event::Event(EventTarget* aOwner, nsPresContext* aPresContext, WidgetEvent* aEvent) @@ -67,13 +64,6 @@ Event::ConstructorInit(EventTarget* aOwner, SetOwner(aOwner); mIsMainThreadEvent = NS_IsMainThread(); - if (mIsMainThreadEvent && !sReturnHighResTimeStampIsSet) { - Preferences::AddBoolVarCache(&sReturnHighResTimeStamp, - "dom.event.highrestimestamp.enabled", - sReturnHighResTimeStamp); - sReturnHighResTimeStampIsSet = true; - } - mPrivateDataDuplicated = false; mWantsPopupControlCheck = false; @@ -1109,10 +1099,6 @@ Event::TimeStamp() const double Event::TimeStampImpl() const { - if (!sReturnHighResTimeStamp) { - return static_cast(mEvent->mTime); - } - if (mEvent->mTimeStamp.IsNull()) { return 0.0; } diff --git a/dom/events/test/test_eventTimeStamp.html b/dom/events/test/test_eventTimeStamp.html index 056203e92..5ca5cd47d 100644 --- a/dom/events/test/test_eventTimeStamp.html +++ b/dom/events/test/test_eventTimeStamp.html @@ -35,12 +35,6 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=77992 SimpleTest.waitForExplicitFinish(); SimpleTest.requestFlakyTimeout("untriaged"); -// We don't use SpecialPowers.pushPrefEnv since it can delay the test -// function until after the load event has fired which means we can't -// test the timestamp of the load event. -const kPrefName = "dom.event.highrestimestamp.enabled"; -var prevPrefValue = SpecialPowers.getBoolPref(kPrefName); -SpecialPowers.setBoolPref(kPrefName, true); testRegularEvents(); // Event.timeStamp should be relative to the time origin which is: diff --git a/dom/smil/test/test_smilTimeEvents.xhtml b/dom/smil/test/test_smilTimeEvents.xhtml index bf6924ddb..002be5539 100644 --- a/dom/smil/test/test_smilTimeEvents.xhtml +++ b/dom/smil/test/test_smilTimeEvents.xhtml @@ -236,14 +236,10 @@ function sanityCheckEvent(evt) is(evt.eventPhase, evt.AT_TARGET); is(evt.bubbles, false, "Event should not bubble"); is(evt.cancelable, false, "Event should not be cancelable"); - if (SpecialPowers.getBoolPref("dom.event.highrestimestamp.enabled")) { - var now = window.performance.now(); - ok(evt.timeStamp > 0 && evt.timeStamp < now, - "Event timeStamp (" + evt.timeStamp + ") should be > 0 but " + - "before the current time (" + now + ")"); - } else { - is(evt.timeStamp, 0, "Event timeStamp should be 0"); - } + var now = window.performance.now(); + ok(evt.timeStamp > 0 && evt.timeStamp < now, + "Event timeStamp (" + evt.timeStamp + ") should be > 0 but " + + "before the current time (" + now + ")"); ok(evt.view !== null, "Event view not set"); } -- cgit v1.2.3