Nika Layzell [:nika] (ni? for response) Reporter
	Description • 8 years ago

This is causing problems with selectevents enabled in OWA. See bug 1280534.

	:Ms2ger (he/him; ⌚ UTC+1/+2)
	Comment 1 • 8 years ago

Spec bug?

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 2 • 8 years ago

This is being discussed in https://github.com/whatwg/dom/issues/211

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 3 • 8 years ago

stone, want to take this ? :)

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 4 • 8 years ago

though, the spec issue is still open.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 5 • 8 years ago

But I think we could still implement this.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 7 • 8 years ago

Comment on attachment 8815657 [details] [diff] [review]
Move UIEvent.cancelBubble to Event

Thanks.

	Ming-Chou Shih [:stone] Assignee
	Comment 8 • 8 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=10962face3c4adbe499fe14a87b86b61d9436d82

	Ming-Chou Shih [:stone] Assignee
	Comment 9 • 8 years ago

Updated the patch summary.

	Pulsebot
	Comment 10 • 8 years ago

Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/033d5532cac7
Move UIEvent.cancelBubble to Event. r=smaug

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 11 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/033d5532cac7

	James Pearson
	Comment 12 • 7 years ago

[Tracking Requested - why for this release]:

This problem also happens with OWA using Firefox 52.0b1

Could this fix please be added to the 52 branch - as 52 will be the next ESR release

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 13 • 7 years ago

Comment on attachment 8815959 [details] [diff] [review]
Bug 1298970 - Move UIEvent.cancelBubble to Event.

Approval Request Comment
[Feature/Bug causing the regression]: NA, but a spec change https://github.com/whatwg/dom/issues/211

[User impact if declined]: Broken pages
[Is this code covered by automated tests?]: yes
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: No 
[List of other uplifts needed for the feature/fix]:
I'd uplift also https://bugzilla.mozilla.org/show_bug.cgi?id=1324380
[Is the change risky?]: No
[Why is the change risky/not risky?]: The property has been in browser for ages but implemented differently. It got recently spec'ed, and this patch makes us follow the spec.
[String changes made/needed]: NA

	Julien Cristau [:jcristau]
	Comment 14 • 7 years ago

Comment on attachment 8815959 [details] [diff] [review]
Bug 1298970 - Move UIEvent.cancelBubble to Event.

follow spec change for cancelBubble attribute, beta52+

	Carsten Book [:Tomcat]
	Comment 15 • 7 years ago

needs rebasing for beta

grafting 378285:033d5532cac7 "Bug 1298970 - Move UIEvent.cancelBubble to Event. r=smaug"
merging dom/events/Event.cpp
merging dom/events/Event.h
merging dom/events/test/mochitest.ini
merging dom/webidl/Event.webidl
warning: conflicts while merging dom/events/test/mochitest.ini! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue

	Ryan VanderMeulen [:RyanVM]
	Comment 16 • 7 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/b130f8af6d55

	Chris Mills [:cmills]
	Comment 17 • 7 years ago

I've updated the affected docs pages:

https://developer.mozilla.org/en-US/docs/Web/API/UIEvent#Browser_compatibility
https://developer.mozilla.org/en-US/docs/Web/API/Event#Browser_compatibility
https://developer.mozilla.org/en-US/docs/Web/API/Event/cancelBubble

I've also added a note to the Fx53 release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/53#DOM_HTML_DOM

Let me know if these changes look OK, or if anything else needs to be done (Yes, I know the Event page looks scrappy and needs cleaning up; that is a job for another day.)

	Chris Rebert
	Comment 18 • 7 years ago

:cmills The spec table in https://developer.mozilla.org/en-US/docs/Web/API/Event/cancelBubble incorrectly refers to the Event interface generally instead of the Event.cancelBubble property specifically.

	Chris Mills [:cmills]
	Comment 19 • 7 years ago

(In reply to Chris Rebert from comment #18)
> :cmills The spec table in
> https://developer.mozilla.org/en-US/docs/Web/API/Event/cancelBubble
> incorrectly refers to the Event interface generally instead of the
> Event.cancelBubble property specifically.

Ooops; fixed now. Thanks for spotting!

	assafdd
	Comment 20 • 7 years ago

:cmills this change was pushed to Fx52 & ESR, I think it is worth updating Fx52 release note as well.

	Chris Mills [:cmills]
	Comment 21 • 7 years ago

(In reply to assafdd from comment #20)
> :cmills this change was pushed to Fx52 & ESR, I think it is worth updating
> Fx52 release note as well.

Thanks for the heads up - I've updated the docs to say this happened from 52 onwards, not 53.