Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Description • 7 years ago

Spotted this while reading code

	David Anderson [:dvander] - inactive, e-mail if emergency
	Comment 2 • 7 years ago
mozreview-review

Comment on attachment 8893383 [details]
Bug 1387046 - Add missing field to operator== in CompositorOptions.

https://reviewboard.mozilla.org/r/164498/#review169868

	Pulsebot
	Comment 3 • 7 years ago

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3758e58e7846
Add missing field to operator== in CompositorOptions. r=dvander

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

https://hg.mozilla.org/mozilla-central/rev/3758e58e7846

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 5 • 7 years ago

Comment on attachment 8893383 [details]
Bug 1387046 - Add missing field to operator== in CompositorOptions.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1365879
[User impact if declined]: unsure, probably nothing. it might result in subtle errors when advanced layers is toggled because the state might not get propagated properly.
[Is this code covered by automated tests?]: yes
[Has the fix been verified in Nightly?]: there's no STR or visible issues, so nothing to verify really
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: trivial correction
[String changes made/needed]: none

	Gerry Chang [:gchang]
	Comment 6 • 7 years ago

Comment on attachment 8893383 [details]
Bug 1387046 - Add missing field to operator== in CompositorOptions.

A trivial fix. Should be low risk. Beta56+. Should be in 56 beta 2.

	Carsten Book [:Tomcat]
	Comment 7 • 7 years ago
bugherder uplift

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