Reviewed by Darin.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Jan 2004 01:12:45 +0000 (01:12 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Jan 2004 01:12:45 +0000 (01:12 +0000)
<rdar://problem/3527853>: REGRESSION (119-120): Peoplesoft menus do not work

        * kwq/KWQKHTMLPart.mm:
        (KWQKHTMLPart::scrollToAnchor): Right before calling checkCompleted(), set
m_bComplete to false, so it can get set right back to true if we're actually done
loading and fire all the right signals.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@5896 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQKHTMLPart.mm

index 53027b1..e1cde92 100644 (file)
@@ -1,3 +1,14 @@
+2004-01-13  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Darin.
+
+       <rdar://problem/3527853>: REGRESSION (119-120): Peoplesoft menus do not work
+       
+        * kwq/KWQKHTMLPart.mm:
+        (KWQKHTMLPart::scrollToAnchor): Right before calling checkCompleted(), set
+       m_bComplete to false, so it can get set right back to true if we're actually done
+       loading and fire all the right signals.
+
 2004-01-13  David Hyatt  <hyatt@apple.com>
 
        Fix for 3526710, applets need to have the correct initial size after a layout has been done, since our
index 4bc4d31..e762a88 100644 (file)
@@ -1566,6 +1566,9 @@ void KWQKHTMLPart::scrollToAnchor(const KURL &URL)
     if (!gotoAnchor(URL.encodedHtmlRef()))
         gotoAnchor(URL.htmlRef());
 
+    // It's important to model this as a load that starts and immediately finishes.
+    // Otherwise, the parent frame may think we never finished loading.
+    d->m_bComplete = false;
     checkCompleted();
 }