Bug 54180
| Summary: | Change <ISINDEX> to use the new shadow DOM model | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Roland Steiner <rolandsteiner> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | abarth, annevk, dglazkov, dominicc, morrita |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | |||
| Bug Blocks: | 48698 | ||
Roland Steiner
Might be doable together with other text-based <input> types, see https://bugs.webkit.org/show_bug.cgi?id=54179
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Adam Barth
I don't think this is correct. <isindex> is speced to uses a non-shadow DOM.
Anne van Kesteren
Specifically, it is handled as a HTML parser "macro".
Dominic Cooney
The prompt and hr are a parser macro; I assume Roland is referring to cleaning up the text input field itself.
Looking at IsIndexInputType, it looks like if TextFieldInputType uses the new shadow DOM model, there is nothing that needs to be done for isindex. So I think we should close this bug.
Roland Steiner
I also don't expect that we have to do anything special here, I added this bug mainly for bookkeeping reasons. However, i would prefer if we close only after the conversion for the other text types is actually done, as unforeseen gotchas lurk everywhere </pessimist>.
Roland Steiner
Looking with Dominic at the parser-generated DOM again, it's probably safe to already close the bug after all.