Bobby Holley (:bholley) Assignee
	Description • 10 years ago

This is a longstanding bug, but the compat impact is a regression from bug 856067. We should take this patch on 33.

	Bobby Holley (:bholley) Assignee
	Comment 1 • 10 years ago

[Tracking Requested - why for this release]: See comment 0. The patch should land on m-i within a date and is very safe.

	Bobby Holley (:bholley) Assignee
	Comment 2 • 10 years ago

(In reply to Bobby Holley (:bholley) from comment #1)
> land on m-i within a date

*within a day.

	Bobby Holley (:bholley) Assignee
	Comment 3 • 10 years ago

I thought we had overrides for all the proxy traps that returned non-primitive
values, but it looks like we missed one.

	Bobby Holley (:bholley) Assignee
	Comment 5 • 10 years ago

For more context - the issue here is that our Xray waivers (which are used from chrome and addon code) miss a certain corner case (custom iterators). This has always been the case, but wasn't noticeable before bug 856067. Now that we deny access to non-waived objects, the lack of a waiver here becomes a problem.

The potential for addon-compat fallout is low, but the fix is really safe and we just need it on aurora.

https://tbpl.mozilla.org/?tree=Try&rev=e69c167c6fd6

	Gabor Krizsanits (INACTIVE)
	Comment 6 • 10 years ago

Comment on attachment 8460642 [details] [diff] [review]
Add a WaiveXrayWrapper override for ::iterate. v1

Review of attachment 8460642 [details] [diff] [review]:
-----------------------------------------------------------------

Uhh... I was not even aware of that trap :(

	Bobby Holley (:bholley) Assignee
	Comment 7 • 10 years ago

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/79d2db9a2a2c
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/88860bed8c96

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

Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/deff412c2d45 and https://hg.mozilla.org/integration/mozilla-inbound/rev/66cb4d556959 for build bustage:

https://tbpl.mozilla.org/php/getParsedLog.php?id=44456475&tree=Mozilla-Inbound

	Bobby Holley (:bholley) Assignee
	Comment 9 • 10 years ago

https://tbpl.mozilla.org/?tree=Try&rev=b7337de650b6

	Bobby Holley (:bholley) Assignee
	Comment 10 • 10 years ago

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/25608b44a408
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/6878d55c8dda

	Carsten Book [:Tomcat]
	Comment 11 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/25608b44a408
https://hg.mozilla.org/mozilla-central/rev/6878d55c8dda

	Bobby Holley (:bholley) Assignee
	Comment 12 • 10 years ago

Comment on attachment 8460642 [details] [diff] [review]
Add a WaiveXrayWrapper override for ::iterate. v1

Approval Request Comment
[Feature/regressing bug #]: See comment 5.
[User impact if declined]: Small potential for addon breakage.
[Describe test coverage new/current, TBPL]: Automated test included, and in general we have lots of test coverage for this stuff.
[Risks and why]: Extremely low risk.
[String/UUID change made/needed]: None

	Ryan VanderMeulen [:RyanVM]
	Comment 13 • 10 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/8b4670845e03
https://hg.mozilla.org/releases/mozilla-aurora/rev/822b25aca158

	Matt Wobensmith [:mwobensmith][:matt:]
	Comment 14 • 10 years ago

Tests look good in Fx33/Fx34. They also look good in Fx31/Fx32, so I can't say if I've truly verified a change, but they pass nonetheless.