2009-11-12 Anantanarayanan G Iyengar <ananta@chromium.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Nov 2009 06:42:43 +0000 (06:42 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Nov 2009 06:42:43 +0000 (06:42 +0000)
commit88bc21a5b6f1232327e3620add6c438a31a817e8
tree3c001c08ad3aea723d0cc382daee5ac5eb26f8a2
parent503c7b59876391ab22f1ca57e92be98dcb99e2ca
2009-11-12  Anantanarayanan G Iyengar  <ananta@chromium.org>

        Reviewed by Adam Barth.

        The document-html and window-html tests don't use a setTimeout to notify that the
        test is completed. The layout test plugin now issues the notifyDone call via
        javascript.

        https://bugs.webkit.org/show_bug.cgi?id=31067

        * plugins/document-open.html:
        * plugins/window-open.html:
2009-11-12  Anantanarayanan G Iyengar  <ananta@chromium.org>

        Reviewed by Adam Barth.

        The document-open.html test was flaky at times. The test invokes the layout test plugin
        which in its destroy stream handler opens a new document. This basically tears down the
        stream and the associated plugin instance, which causes a crash when the plugin stream
        dereferences an invalid m_client pointer which points to the PluginView instance which
        is invalid at this time. Fix is to set the m_client pointer to NULL in the stop function
        and check for the same.

        https://bugs.webkit.org/show_bug.cgi?id=31067

        * plugins/PluginStream.cpp:
        (WebCore::PluginStream::stop):
        (WebCore::PluginStream::destroyStream):
2009-11-12  Anantanarayanan G Iyengar  <ananta@chromium.org>

        Reviewed by Adam Barth.

        The document-open.html test was flaky at times. The test invokes the layout test plugin
        which in its destroy stream handler opens a new document. This basically tears down the
        stream and the associated plugin instance. The pluginLog function in the layout test
        plugin attempts to retrieve the window script object on a torn down plugin instance
        which crashed consistently on windows in the debugger. The functions which issue
        these logs already have a valid window script object. We now have variants of the pluginLog
        function which take in a window script object with and without variable arguments.

        https://bugs.webkit.org/show_bug.cgi?id=31067

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
        (pluginLogWithWindowObject):
        (pluginLogWithWindowObjectVariableArgs):
        (pluginLog):
        (notifyTestCompletion):
        (testDocumentOpen):
        (testWindowOpen):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@50929 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/plugins/document-open.html
LayoutTests/plugins/window-open.html
WebCore/ChangeLog
WebCore/plugins/PluginStream.cpp
WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp