Honour gfx.canvas.azure.prefer-skia pref on OS X

RESOLVED FIXED in mozilla12

Status

()

Core
Graphics
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: gw280, Assigned: gw280)

Tracking

unspecified
mozilla12
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

We should honour that pref on OS X. Currently it's hard coded to use CG.
Created attachment 592239 [details] [diff] [review]
Use pref on os x
Attachment #592239 - Flags: review?(jmuizelaar)
(Assignee)

Updated

6 years ago
Attachment #592239 - Attachment is patch: true
Comment on attachment 592239 [details] [diff] [review]
Use pref on os x

Use GetBoolVarCache?
Attachment #592239 - Flags: review?(jmuizelaar) → review-
AddBoolVarCache
Created attachment 592251 [details] [diff] [review]
updated patch
Attachment #592239 - Attachment is obsolete: true
Attachment #592251 - Flags: review?(jmuizelaar)
Comment on attachment 592251 [details] [diff] [review]
updated patch

This would be better off in gfxPlatform and can be initialized in gfxPlatform's constructor.
Attachment #592251 - Flags: review?(jmuizelaar) → review-
Created attachment 592264 [details] [diff] [review]
refactor the wooooorld
Attachment #592251 - Attachment is obsolete: true
Attachment #592264 - Flags: review?(jmuizelaar)
(Assignee)

Updated

6 years ago
Attachment #592264 - Flags: review?(joe)
Attachment #592264 - Flags: review?(jmuizelaar) → review+
Comment on attachment 592264 [details] [diff] [review]
refactor the wooooorld

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

::: gfx/thebes/gfxD2DSurface.cpp
@@ +127,5 @@
>  const gfxIntSize gfxD2DSurface::GetSize() const
>  { 
>      return gfxIntSize(cairo_d2d_surface_get_width(mSurface),
>                        cairo_d2d_surface_get_height(mSurface));
> +}

THIS IS NOT PERMITTED IN THIS PATCH

::: gfx/thebes/gfxPlatform.h
@@ +449,5 @@
>  
>      // which scripts should be shaped with harfbuzz
>      PRInt32 mUseHarfBuzzScripts;
>  
> +    // The preferred azure backend to use

_DRAW TARGET_ backend

::: gfx/thebes/gfxWindowsPlatform.cpp
@@ +559,3 @@
>    }
> +
> +  return true;

Put the return with the aBackend assignment, and return false outside the if.
Attachment #592264 - Flags: review?(joe)
Attachment #592264 - Flags: review?(jmuizelaar)
Attachment #592264 - Flags: review+
Created attachment 592280 [details] [diff] [review]
finally final patch

Pushed to try at https://tbpl.mozilla.org/?tree=Try&rev=28888d108556
Attachment #592264 - Attachment is obsolete: true
Attachment #592264 - Flags: review?(jmuizelaar)
Looks like this can be checked in. The TBPL shows that the only test failures are either known intermittent failures, or are already/also present on other completely unrelated tests that have also been pushed to try.
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → gwright
https://hg.mozilla.org/integration/mozilla-inbound/rev/68b7ab93105f
Keywords: checkin-needed
Target Milestone: --- → mozilla12
https://hg.mozilla.org/mozilla-central/rev/68b7ab93105f
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.