Bug 44315 - HISTORY_ALWAYS_ASYNC should be removed (history should always be async)
Summary: HISTORY_ALWAYS_ASYNC should be removed (history should always be async)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: History (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Mihai Parparita
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-19 17:31 PDT by Mihai Parparita
Modified: 2011-02-27 09:21 PST (History)
3 users (show)

See Also:


Attachments
Patch (5.46 KB, patch)
2010-08-24 14:04 PDT, Mihai Parparita
no flags Details | Formatted Diff | Diff
Patch (10.36 KB, patch)
2010-08-24 14:47 PDT, Mihai Parparita
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mihai Parparita 2010-08-19 17:31:56 PDT
It looks like the HISTORY_ALWAYS_ASYNC #define was added for bug 33538. At that point there was some debate as to the intent of the HTML5 spec. However, it looks like there was a WHATWG thread to discuss this (http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2010-January/thread.html#24786) and the outcome was that the spec now says "To traverse the history by a delta delta, the user agent must queue a task to run the following steps. " (from http://www.whatwg.org/specs/web-apps/current-work/multipage/history.html#traverse-the-history-by-a-delta)

Brady, I realize that you were opposed to this change initially. Given that the spec has been updated (as as far as I can tell, no existing layout tests depend on navigation being synchronous), would you be OK with it now?
Comment 1 Mihai Parparita 2010-08-24 14:04:32 PDT
Created attachment 65319 [details]
Patch
Comment 2 Mihai Parparita 2010-08-24 14:25:04 PDT
Comment on attachment 65319 [details]
Patch

Actually, fast/dom/location-hash.html fails with this change. I'll need to update that too.
Comment 3 Brady Eidson 2010-08-24 14:38:08 PDT
Wonder what the change to that test will be.
Comment 4 Mihai Parparita 2010-08-24 14:47:28 PDT
Created attachment 65328 [details]
Patch
Comment 5 Mihai Parparita 2010-08-24 14:50:37 PDT
(In reply to comment #3)
> Wonder what the change to that test will be.

It's basically reverting the changes to the test made by r53950. With my changes the test won't pass if traversal is synchronous, if you'd like I can make it pass with either sync or async traversal (since there's an separatel explicit test for async behavior).
Comment 6 Adam Barth 2010-08-30 16:49:09 PDT
Comment on attachment 65328 [details]
Patch

Following HTML5 is a good idea.  Brady, feel free to let us know if you're still opposed to this patch.
Comment 7 WebKit Commit Bot 2010-08-30 22:39:59 PDT
Comment on attachment 65328 [details]
Patch

Clearing flags on attachment: 65328

Committed r66458: <http://trac.webkit.org/changeset/66458>
Comment 8 WebKit Commit Bot 2010-08-30 22:40:03 PDT
All reviewed patches have been landed.  Closing bug.