Changeset 41563 in webkit


Ignore:
Timestamp:
Mar 10, 2009 12:49:47 PM (15 years ago)
Author:
cwzwarich@webkit.org
Message:

2009-03-10 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Geoff Garen.

Bug 23736: WebKit Crashes on http://g-conquest.fr/~server2
<https://bugs.webkit.org/show_bug.cgi?id=23736>
<rdar://problem/6560278>

Before replacing the document, FrameLoader::executeIfJavaScriptURL()
should call stopAllLoaders(), just like continueLoadAfterNavigationPolicy().

WebCore:

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::executeIfJavaScriptURL):

LayoutTests:

  • http/tests/misc/javascript-url-stop-loaders-expected.txt: Added.
  • http/tests/misc/javascript-url-stop-loaders.html: Added.
Location:
trunk
Files:
2 added
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r41562 r41563  
     12009-03-10  Cameron Zwarich  <cwzwarich@uwaterloo.ca>
     2
     3        Reviewed by Geoff Garen.
     4
     5        Test for bug 23736: WebKit Crashes on http://g-conquest.fr/~server2
     6        <https://bugs.webkit.org/show_bug.cgi?id=23736>
     7        <rdar://problem/6560278>
     8
     9        * http/tests/misc/javascript-url-stop-loaders-expected.txt: Added.
     10        * http/tests/misc/javascript-url-stop-loaders.html: Added.
     11
    1122009-03-10  Darin Adler  <darin@apple.com>
    213
  • trunk/WebCore/ChangeLog

    r41562 r41563  
     12009-03-10  Cameron Zwarich  <cwzwarich@uwaterloo.ca>
     2
     3        Reviewed by Geoff Garen.
     4
     5        Bug 23736: WebKit Crashes on http://g-conquest.fr/~server2
     6        <https://bugs.webkit.org/show_bug.cgi?id=23736>
     7        <rdar://problem/6560278>
     8
     9        Before replacing the document, FrameLoader::executeIfJavaScriptURL()
     10        should call stopAllLoaders(), just like continueLoadAfterNavigationPolicy().
     11
     12        * loader/FrameLoader.cpp:
     13        (WebCore::FrameLoader::executeIfJavaScriptURL):
     14
    1152009-03-10  Darin Adler  <darin@apple.com>
    216
  • trunk/WebCore/loader/FrameLoader.cpp

    r41555 r41563  
    767767    //        http://bugs.webkit.org/show_bug.cgi?id=16782
    768768    if (replaceDocument) {
     769        stopAllLoaders();
    769770        begin(m_URL, true, currentSecurityOrigin);
    770771        write(scriptResult);
Note: See TracChangeset for help on using the changeset viewer.