Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 496200

Summary: Avoid using 'new Date().getTime()'
Product: [ECD] Orion Reporter: Michael Rennie <Michael_Rennie>
Component: ClientAssignee: Michael Rennie <Michael_Rennie>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 10.0   
Target Milestone: 13.0   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Michael Rennie CLA 2016-06-15 12:37:58 EDT
Instead we should be using Date.now() - it avoids creating spurious instances of Date, and depending on which browser you run the jsperf tests in, it can be twice as fast.

https://jsperf.com/date-now-vs-new-date-gettime/8
Comment 1 Michael Rennie CLA 2016-07-19 09:24:32 EDT
Fixed in:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=1f8a2f16947507cf6c4db6ee19c1f4b5d3ea5714

There are a few occurrences of its use left in some of the embedded requirejs's. I left those ones alone.