Bug 196514

Summary: HTML Parser: Remove conditional parsing of <noembed> content
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: DOMAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, ews-watchlist, ggaren, gyuyoung.kim, rniwa, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: https://html.spec.whatwg.org/multipage/parsing.html#parsing-html-fragments:noembed
Attachments:
Description Flags
Patch none

Description Chris Dumez 2019-04-02 16:41:19 PDT 
Our HTML Parser has raw text handling for <noembed> content only if plugins are runnable. However, the HTML specification doesn't ask such behavior [1], and it doesn't match to our HTML serializer. We should always handle it as raw text.
Blink already made this change in https://chromium-review.googlesource.com/c/1477556.

[1] https://html.spec.whatwg.org/multipage/parsing.html#parsing-html-fragments:noembed
Comment 1 Chris Dumez 2019-04-02 16:45:40 PDT 
Created attachment 366551 [details]
Patch
Comment 2 Geoffrey Garen 2019-04-02 16:59:57 PDT 
Comment on attachment 366551 [details]
Patch

r=me
Comment 3 WebKit Commit Bot 2019-04-02 18:55:43 PDT 
Comment on attachment 366551 [details]
Patch

Clearing flags on attachment: 366551

Committed r243782: <https://trac.webkit.org/changeset/243782>
Comment 4 WebKit Commit Bot 2019-04-02 18:55:45 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2019-04-02 18:56:21 PDT 
<rdar://problem/49544147>