Reviewed by Hyatt.

        Fix for <rdar://problem/5026141> REGRESSION: Aperture Viewer Flashes when switching from Index Page to Detail page in Web Gallery

        This disables the paint that occurs after WebCore finishes loading the document.  We should reenable this when we
        move the dispatchDidFinishLoad delegate callback to happen around the same time as the JS onload call.  See <rdar://problem/5092361>.

        * dom/Document.cpp: (WebCore::Document::implicitClose):

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

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 636b1c39b63e61aa266e7445a1027c4a6de361cb..43817cc6d44e2033a1b87564f3c646b429a7b026 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2007-03-27  Adele Peterson  <adele@apple.com>
+
+        Reviewed by Hyatt.
+
+        Fix for <rdar://problem/5026141> REGRESSION: Aperture Viewer Flashes when switching from Index Page to Detail page in Web Gallery
+
+        This disables the paint that occurs after WebCore finishes loading the document.  We should reenable this when we
+        move the dispatchDidFinishLoad delegate callback to happen around the same time as the JS onload call.  See <rdar://problem/5092361>.
+
+        * dom/Document.cpp: (WebCore::Document::implicitClose):
+
 2007-03-26  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by darin

diff --git a/WebCore/dom/Document.cpp b/WebCore/dom/Document.cpp
index e1f3ea884642a40c420f81d141ee30e9fdd4c84f..ca78b3ddd81276fee7f929d3619ebca4babfa0ee 100644 (file)
--- a/WebCore/dom/Document.cpp
+++ b/WebCore/dom/Document.cpp
@@ -1421,8 +1421,12 @@ void Document::implicitClose()
         // Paint immediately after the document is ready.  We do this to ensure that any timers set by the
         // onload don't have a chance to fire before we would have painted.  To avoid over-flushing we only
         // worry about this for the top-level document.
+#if !PLATFORM(MAC)
+        // FIXME: This causes a timing issue with the dispatchDidFinishLoad delegate callback.
+        // See <rdar://problem/5092361>
         if (view() && !ownerElement())
             view()->update();
+#endif
     }
 
 #if PLATFORM(MAC)