Closed Bug 975206 Opened 6 years ago Closed 6 years ago

clip-path urls on SVG nodes without corresponding clipPath elements don't fire events

Categories

(Core :: SVG, defect)

27 Branch
x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: nross83, Assigned: longsonr)

Details

Attachments

(2 files, 1 obsolete file)

Attached file svg-no-events.html
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/32.0.1700.107 Safari/537.36

Steps to reproduce:

I created an SVG node with a clip-path url attribute that did not match a clipPath node.  In my specific case, there was no clipPath element on the page.  Here's a fiddle which demonstrates the problem:

http://jsfiddle.net/932Um/

Mousing over the black square should fire an event.


Actual results:

The SVG node with the clip-path url attribute did not fire any mouse events such as 'mouseover', 'mouseout', or 'click'.


Expected results:

The SVG node should still fire registered mouse events even if the clip-path url attribute is invalid.  This is how WebKit handles it currently.  It was difficult to figure out why events were not firing and required lots of troubleshooting before the solution was determined.
Component: Untriaged → SVG
Product: Firefox → Core
Attached patch pointer.txt (obsolete) — Splinter Review
Assignee: nobody → longsonr
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attachment #8380145 - Flags: review?(jwatt)
Attached patch pointer.txtSplinter Review
Attachment #8380145 - Attachment is obsolete: true
Attachment #8380145 - Flags: review?(jwatt)
Attachment #8380151 - Flags: review?(jwatt)
Attachment #8380151 - Flags: review?(jwatt) → review+
Simplified the test so it runs more quickly. Code change is the same though.

https://tbpl.mozilla.org/?tree=Try&rev=d43facf0a553
https://hg.mozilla.org/mozilla-central/rev/61c7a0b4ece1
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.