Move to using std::unique_ptr for KeyboardEvent, ScriptExecutionContext::PendingException
Created attachment 224675 [details] Patch
Comment on attachment 224675 [details] Patch Clearing flags on attachment: 224675 Committed r164422: <http://trac.webkit.org/changeset/164422>
All reviewed patches have been landed. Closing bug.
(In reply to comment #2) > (From update of attachment 224675 [details]) > Clearing flags on attachment: 224675 > > Committed r164422: <http://trac.webkit.org/changeset/164422> It caused regressions on Apple Mac debug bots: http://build.webkit.org/builders/Apple%20Mavericks%20Debug%20WK1%20%28Tests%29/builds/3145
Re-opened since this is blocked by bug 129102
Created attachment 225755 [details] Patch
Comment on attachment 225755 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=225755&action=review > Source/WebCore/html/track/LoadableTextTrack.cpp:107 > void LoadableTextTrack::newCuesAvailable(TextTrackLoader* loader) > { > - ASSERT_UNUSED(loader, m_loader == loader); > + ASSERT_UNUSED(loader, m_loader.get() == loader); > > Vector<RefPtr<TextTrackCue>> newCues; > m_loader->getNewCues(newCues); Isn't this part of your text track patch?
Comment on attachment 225755 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=225755&action=review >> Source/WebCore/html/track/LoadableTextTrack.cpp:107 >> m_loader->getNewCues(newCues); > > Isn't this part of your text track patch? It should be. This got included by accident.
Created attachment 225787 [details] Patch
Comment on attachment 225787 [details] Patch Clearing flags on attachment: 225787 Committed r165177: <http://trac.webkit.org/changeset/165177>