The default bug view has changed. See this FAQ.

Remove isindex layout code

RESOLVED FIXED in mozilla8

Status

()

Core
Layout: Form Controls
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: sicking, Assigned: sicking)

Tracking

Trunk
mozilla8
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 539093 [details] [diff] [review]
Patch to fix

The HTML5 parser doesn't output isindex elements for <isindex> markup any more. Instead it outputs a couple of <hr>s and an <input>. So layout no longer needs to have support for isindex frames.
Attachment #539093 - Flags: review?(dbaron)

Updated

6 years ago
Blocks: 666665
Comment on attachment 539093 [details] [diff] [review]
Patch to fix

r=dbaron

Once you've removed the content code too, you should do an mxr search to make sure you haven't missed anything.
Attachment #539093 - Flags: review?(dbaron) → review+
Landed on inbound

http://hg.mozilla.org/integration/mozilla-inbound/rev/47d8748daa90
http://hg.mozilla.org/mozilla-central/rev/47d8748daa90
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.