[Webkit-unassigned] [Bug 64369] PreloadScanner should reuse tokens from the parser

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 13 07:46:38 PDT 2011


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


Balazs Kelemen <kbalazs at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |DUPLICATE




--- Comment #7 from Balazs Kelemen <kbalazs at webkit.org>  2011-07-13 07:46:38 PST ---
(In reply to comment #6)
> (In reply to comment #5)
> > After some investigation I think the whole idea is wrong. I thought that the preload scanner scanning the input again after the parser but now it seems to be it does not. The preload scanner only scans text that the parser has not seen yet. In this case it is useless to feed it with "old" tokens. Adam, could you validate this for me? Thanks.
> 
> It is the other way around. The preload scanner should feed its tokens to the parser. There is no case where it would make sense for the parser to feed tokens to the preload scanner.
> 
> For example, consider:
> 1. <img/>
> 2. <script src="slow.js"></script>
> 3. <img/>
> 
> The Parser would tokenize #1 and #2, then the parser is forced to pause while the script downloads. In the mean time, the PreloadScanner would tokenize #3. After the script runs, the parser would have to retokenize #3 unless we save the work done by the preload scanner.

Thanks for the clarificatoin.

> 
> There is a little more information in this bug:
> https://bugs.webkit.org/show_bug.cgi?id=57376
> 
> We should dupe one of them.

Yep, I am going to mark this one as a duplicate.

*** This bug has been marked as a duplicate of bug 57376 ***

-- 
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