No other browser has this particular quirk, and the testcases in bug 264624 all pass without it. (I think thanks to the HTML5 parser.)
Created attachment 466139 [details] [diff] [review] Patch v1
Comment on attachment 466139 [details] [diff] [review] Patch v1 Since Mats fixed Bug 264624, he could review this.
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+
Created attachment 483413 [details] [diff] [review] Patch v2 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
Created attachment 521566 [details] [diff] [review] Patch for checkin
Attachment #483413 - Attachment is obsolete: true
Whiteboard: [needs landing]
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
(FWIW, I claim the dromaeo improvement on that push <http://groups.google.com/group/mozilla.dev.tree-management/browse_thread/thread/47eec39b9eb490cf/8c652ab00d54a991>.)
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?
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.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.