Unprefix MutationObserver constructor
Created attachment 188240 [details] Patch
Comment on attachment 188240 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188240&action=review > Source/WebCore/page/DOMWindow.idl:802 > + attribute MutationObserverConstructor WebKitMutationObserver; // FIXME: Add metrics to determine when we can remove this. Should we add a test that the prefixed version still works?
Comment on attachment 188240 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188240&action=review >> Source/WebCore/page/DOMWindow.idl:802 >> + attribute MutationObserverConstructor WebKitMutationObserver; // FIXME: Add metrics to determine when we can remove this. > > Should we add a test that the prefixed version still works? I did, that's what the mutation-observer-prefix.html test is in the ChangeLog.
Comment on attachment 188240 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=188240&action=review >>> Source/WebCore/page/DOMWindow.idl:802 >>> + attribute MutationObserverConstructor WebKitMutationObserver; // FIXME: Add metrics to determine when we can remove this. >> >> Should we add a test that the prefixed version still works? > > I did, that's what the mutation-observer-prefix.html test is in the ChangeLog. Ah. I missed that obviously. :)
Comment on attachment 188240 [details] Patch Clearing flags on attachment: 188240 Committed r143386: <http://trac.webkit.org/changeset/143386>
All reviewed patches have been landed. Closing bug.