Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(792)

Unified Diff: third_party/WebKit/Source/core/events/Event.idl

Issue 1352523002: Use high precision timestamp for Event.timestamp (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebase Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/events/Event.idl
diff --git a/third_party/WebKit/Source/core/events/Event.idl b/third_party/WebKit/Source/core/events/Event.idl
index 07398122bdfe586dfe239ebb2290f36e479dab37..1db19f6f5c94f071ff64efa1670944e9804eb507 100644
--- a/third_party/WebKit/Source/core/events/Event.idl
+++ b/third_party/WebKit/Source/core/events/Event.idl
@@ -44,7 +44,17 @@
readonly attribute boolean defaultPrevented;
[RuntimeEnabled=TrustedEvents, Unforgeable] readonly attribute boolean isTrusted;
- readonly attribute DOMTimeStamp timeStamp;
+
+ // TODO(majidvp): At the moment the actual return value type can either
+ // be:
+ // - DOMTimeStamp (i.e. long long): legacy type
+ // - DOMHighResTimeStamp (i.e. double): HighResEventTimeStamp REF is
+ // enabled
+ // Below IDL definition uses DOMHighResTimeStamp because all DOMTimeStamp
+ // values can be represented in double type without any loss of precision.
+ // Once the feature is enabled by default the return value type will always
+ // match the definition here.
+ [CallWith=ScriptState] readonly attribute DOMHighResTimeStamp timeStamp;
// FIXME: initEvent()'s arguments should not be optional.
[Measure] void initEvent([Default=Undefined] optional DOMString type,
« no previous file with comments | « third_party/WebKit/Source/core/events/Event.cpp ('k') | third_party/WebKit/Source/core/events/GestureEvent.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698