[Webkit-unassigned] [Bug 119322] javascript: urls should always be parsed synchronously

[bugzilla-daemon at webkit.org]

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

Ahmad Saleem <ahmad.saleem792 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ahmad.saleem792 at gmail.com

--- Comment #1 from Ahmad Saleem <ahmad.saleem792 at gmail.com> ---
In Webkit -> Renamed isLayoutTimerActive to isLayoutPending:

In Document.cpp:

Just another renamed to bool: https://github.com/WebKit/WebKit/blob/f1bf39d41c844a64adcee49a7d83afe5d174697b/Source/WebCore/dom/Document.cpp#L3349

In Document.h:

Another renamed:

https://github.com/WebKit/WebKit/blob/552bfd33df1642d5d889a7d649cb7b91eafb27b7/Source/WebCore/dom/Document.h#L781

In DocumentParser.h:

Webkit does not have pinToMainThread to add comment.

In HTMLDocumentParser.

Add asset based on "pinToMainThread" or equivalent here including FIXME and other bits:

https://github.com/WebKit/WebKit/blob/03b5974b58684a094f81c69015126e8bcbfa15e7/Source/WebCore/html/parser/HTMLDocumentParser.cpp#L202

Not able to find XSS related anything here.

In HTMLParserScheduler.cpp:

Update here with new FIXME and renamed from above:

https://github.com/WebKit/WebKit/blob/03b5974b58684a094f81c69015126e8bcbfa15e7/Source/WebCore/html/parser/HTMLParserScheduler.cpp#L94

Not able to find paint related change equivalent.

In DocumentLoader.cpp & DocumentWriter.cpp:

nothing changed in Blink patch except comments.

In WebViewImpl.cpp - just new FIXME comment.

______

Is it something worth pursuing this time considering, it would add more FIXME?

Appreciate if someone can add more context and details. Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220924/492f07c2/attachment.htm>