Closed Bug 583857 Opened 10 years ago Closed 9 years ago

Support disabling reftests on D2D

Categories

(Core :: Graphics, defect)

x86
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: jrmuizel, Assigned: jrmuizel)

References

(Depends on 2 open bugs, Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
Blocks: d2d-reftest
Comment on attachment 462502 [details] [diff] [review]
First cut at an implementation of the gfxinfo service

This adds GfxInfo service that will let us get information like whether d2d is enabled. Right now there's a single implementation that lives in widget/windows.

Whether or not d2d is enabled is exposed as a global boolean. e.g.
random-if(d2d) == boxshadow-rounded-spread.html boxshadow-rounded-spread-ref.html
Attachment #462502 - Flags: feedback?(vladimir)
Attachment #462502 - Flags: feedback?(dbaron)
Blocks: 549353
Comment on attachment 462502 [details] [diff] [review]
First cut at an implementation of the gfxinfo service

I'd be happy to call this r=dbaron if you want (though a gfx peer should probably also look at it).
Attachment #462502 - Flags: feedback?(dbaron) → feedback+
OS: Mac OS X → Windows 7
Attachment #462502 - Flags: review?(roc)
http://hg.mozilla.org/mozilla-central/rev/d3293a155e32
http://hg.mozilla.org/mozilla-central/rev/3fdcf0ef7e82
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 462502 [details] [diff] [review]
First cut at an implementation of the gfxinfo service

>+namespace mozilla {
>+namespace widget {
>+NS_GENERIC_FACTORY_CONSTRUCTOR(GfxInfo)
>+}
>+}
...
>+  { &kNS_GFXINFO_CID, false, NULL, GfxInfoConstructor },
My build doesn't like this, but it likes mozilla::widget::GfxInfoConstructor
(In reply to comment #5)
> >+  { &kNS_GFXINFO_CID, false, NULL, GfxInfoConstructor },
> My build doesn't like this, but it likes mozilla::widget::GfxInfoConstructor

See bug 587322
Depends on: 587322
Attachment #462502 - Flags: feedback?(vladimir)
Depends on: 629617
Depends on: 629957
You need to log in before you can comment on or make changes to this bug.