Remove image map quirk from bug 264624

RESOLVED FIXED in mozilla5

Status

()

defect
RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

({dev-doc-complete, html5})

Trunk
mozilla5
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

No other browser has this particular quirk, and the testcases in bug 264624 all pass without it. (I think thanks to the HTML5 parser.)
Assignee

Comment 1

9 years ago
Posted patch Patch v1 (obsolete) — Splinter Review
Attachment #466139 - Flags: review?(Olli.Pettay)
Comment on attachment 466139 [details] [diff] [review]
Patch v1

Since Mats fixed Bug 264624, he could review this.
Attachment #466139 - Flags: review?(Olli.Pettay)
Assignee

Comment 3

9 years ago
Comment on attachment 466139 [details] [diff] [review]
Patch v1

Sure.
Attachment #466139 - Flags: review?(matspal)
Comment on attachment 466139 [details] [diff] [review]
Patch v1

The code change looks good, but the test doesn't do what it's
supposed to do.  First, you need to include the script
"/tests/SimpleTest/EventUtils.js" for synthesizeMouse,
then the test also passes without the code change because
50,50 is actually outside the image (which is the builtin
"broken URL" image).  Clicking at 2,2 should work.
r=mats with that
Attachment #466139 - Flags: review?(matspal) → review+
Assignee

Comment 5

9 years ago
Posted patch Patch v2 (obsolete) — Splinter Review
EventUtils.js was already included, and I fixed the src and the coords to ensure I hit the right thing.
Attachment #466139 - Attachment is obsolete: true
Assignee

Updated

9 years ago
Whiteboard: [needs landing]
Assignee

Updated

9 years ago
Depends on: post2.0
Assignee

Comment 6

8 years ago
Attachment #483413 - Attachment is obsolete: true
Assignee

Updated

8 years ago
Keywords: checkin-needed
Whiteboard: [needs landing]
http://hg.mozilla.org/mozilla-central/rev/31464345fc6e
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Flags: in-testsuite+
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
Am I correctly interpreting this code as we no longer skip over empty maps in favor of non-empty ones if we find a matching empty map before a non-empty one, when in quirks mode?
You are.
OK, I need to clarify further. Are we talking about empty vs. non-empty map areas or maps?
maps, as in the test:

<map name=a></map>
<map name=a><area shape=rect coords=25,25,75,75 href=#fail></map>
<img usemap=#a src=image.png>
Documentation updated:

https://developer.mozilla.org/en/HTML/Element/map

Also mentioned on Firefox 5 for developers.
You need to log in before you can comment on or make changes to this bug.