Reviewed by Beth.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Aug 2006 22:05:57 +0000 (22:05 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Aug 2006 22:05:57 +0000 (22:05 +0000)
        - revert part of my last fix that broke the Safari bookmarks view

        * WebView/WebFrame.m:
        (-[WebFrame _checkLoadCompleteForThisFrame]): still send layout message for non-HTML views

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

WebKit/ChangeLog
WebKit/WebView/WebFrame.m

index ccbf229e94dac2355c263afef5543b97ac423bab..9847d9c8779369a8c3e0667f5f7c74c45f19e0cd 100644 (file)
@@ -1,3 +1,12 @@
+2006-08-01  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Beth.
+
+        - revert part of my last fix that broke the Safari bookmarks view
+        
+        * WebView/WebFrame.m:
+        (-[WebFrame _checkLoadCompleteForThisFrame]): still send layout message for non-HTML views
+
 2006-08-01  Tim Omernick  <timo@apple.com>
 
         Reviewed by Anders.
index 0f6f4f5b9f1333ed3e5a12ffd86b53024086f980..e272e79dbd05384d0eb9f9af6dc439d1b882281f 100644 (file)
@@ -1170,6 +1170,14 @@ static CFAbsoluteTime _timeOfLastCompletedLoad;
                 // Maybe there are bugs because of that, or extra work we can skip because
                 // the new page is ready.
 
+                // Tell the just loaded document to layout.  This may be necessary
+                // for non-html content that needs a layout message.
+                if (!([[self dataSource] _isDocumentHTML])) {
+                    [thisDocumentView setNeedsLayout:YES];
+                    [thisDocumentView layout];
+                    [thisDocumentView setNeedsDisplay:YES];
+                }
+                 
                 // If the user had a scroll point scroll to it.  This will override
                 // the anchor point.  After much discussion it was decided by folks
                 // that the user scroll point should override the anchor point.