GCC complains about switches over enums that don't have default cases. In the interest of reducing warnings, these switches should be fixed.
Created attachment 594465 [details] [diff] [review] patch Simple patch. It seemed better to eliminate the duplicate code in SourceSurfaceCairo.cpp than fixing the switches.
Autoland Patchset: Patches: 594465 Branch: mozilla-central => try Destination: http://hg.mozilla.org/try/rev/388039878d56 Try run started, revision 388039878d56. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=388039878d56
Try run for 388039878d56 is complete. Detailed breakdown of the results available here: https://tbpl.mozilla.org/?tree=Try&rev=388039878d56 Results (out of 208 total builds): success: 186 warnings: 21 failure: 1 Builds (or logs if builds failed) available at: http://firstname.lastname@example.org