[Webkit-unassigned] [Bug 31838] Nested start/stop events in dispatchDidChangeLocationWithinPage for pages that do fragment redirect in window.onload

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 24 12:08:51 PST 2009


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


Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #43780|                            |review-
               Flag|                            |




--- Comment #2 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2009-11-24 12:08:50 PST ---
(From update of attachment 43780)
> Index: third_party/WebKit/WebKit/chromium/src/FrameLoaderClientImpl.cpp
...
> +    // Flag of whether frame loader is completed. Generate didStartLoading and
> +    // didStopLoading only when loader is completed so that we don't fire
> +    // them for fragment redirection that happens in window.onload handler.
> +    // See http://crbug.com/15782

nit: please link to the bugs.webkit.org bug instead.  that bug can have a
reference
to the chromium bug if you think it would be helpful.

also, you need to include a modification to WebCore/ChangeLog.  please see the
instructions here: http://webkit.org/coding/contributing.html

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list