Skip to content
This repository has been archived by the owner on Nov 23, 2017. It is now read-only.

Add support for console.time() #207

Closed
nchevobbe opened this issue Aug 10, 2016 · 2 comments
Closed

Add support for console.time() #207

nchevobbe opened this issue Aug 10, 2016 · 2 comments
Assignees

Comments

@nchevobbe
Copy link
Member

Add support for console.time() and console.timeEnd()

I'm working on it right now

@nchevobbe
Copy link
Member Author

nchevobbe commented Aug 10, 2016

I was curious to see what the code below output in the current console :

console.time();
console.timeEnd();

-> We don't show anything.
In Chrome, they output :
default: 2544.671ms

I think it might make sense to have this too in our console. I think this might require some change server side, because in the code above, the message packet does not have the timer object at all.


Another test I ran was :
console.timeEnd("foo");
i.e. , a timeEnd() call without a prior time() call, and if in Firefox it does not show anything, in Chrome it awkwardly show a duration (of what ? I don't know)

Maybe for this case we could output a message saying that prior time() call was not found allowing the user to know what's happening if they made a typo ?

@linclark
Copy link

This will land in m-c in Bug 1295347

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants