Created attachment 73322 [details] Patch

Let's hold off on landing this until the 3 bugs that this depends on are fixed. Then I believe we are completely in compliance with the spec and can remove the prefix. Just in case I missed something, James, you might want to do a quick pass over the spec just to verify that we are done. Adam, once that is done, do you feel comfortable approving this? Is there anyone else we should check with?

Comment on attachment 73322 [details] Patch I'm following this spec that closely, but I know that you all are.

This patch LGTM. I took another pass over the spec to check that we are all in compliance. After bug 49101, this should be ready to go. We discussed in the working group this morning that if this can land by the end of the year, it will allow us to move forward in the w3c process to that point that will allow Web Timing to be included in the final shipping IE9. We should help make sure that can happen. Darin, do you have any thoughts on removing the vendor prefix?

What about Mozilla's plans? How far are they from implementing this spec?

(In reply to comment #5) > What about Mozilla's plans? How far are they from implementing this spec? Jonas has been participating in the working group list and calls. There was a patch to start landing, but now it appears abandoned and ownerless: https://bugzilla.mozilla.org/show_bug.cgi?id=570341 Sigbjørn from Opera has also recently starting taking an interest in the list discussions, but I don't know the status of their implementation (if any). The latest IE9 beta is in complete compliance w/ the spec (except for the ms prefix).

New requirement: window.performance should be replaceable.

Created attachment 76806 [details] Patch

I'm going to mark the two bugs we found from the IE test suite as blocking this. We should only remove the prefix when it is perfect.

Now that the interface is replaceable and all the blocking bugs are fixed, this looks good to me.

Comment on attachment 76806 [details] Patch Rejecting attachment 76806 [details] from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-sf-cq', 'apply-attachment', '--no-update', '--non-interactive', 76806]" exit_code: 2 Last 500 characters of output: patching file WebCore/page/DOMWindow.cpp patching file WebCore/page/DOMWindow.h Hunk #1 succeeded at 405 (offset 3 lines). patching file WebCore/page/DOMWindow.idl Hunk #1 succeeded at 222 (offset 1 line). patching file WebKit/chromium/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file WebKit/chromium/src/WebFrameImpl.cpp Hunk #1 succeeded at 713 (offset 1 line). Failed to run "[u'/Projects/CommitQueue/Tools/Scripts/svn-apply', u'--reviewer', u'Darin Fisher', u'--force']" exit_code: 1 Full output: http://queues.webkit.org/results/7347052

Created attachment 78137 [details] Patch for landing

Comment on attachment 78137 [details] Patch for landing Clearing flags on attachment: 78137 Committed r75200: <http://trac.webkit.org/changeset/75200>

All reviewed patches have been landed. Closing bug.