Closed Bug 778595 Opened 13 years ago Closed 13 years ago

[Azure] Add a reftest sandbox flag for a mismatch between Azure backends

Categories

(Core :: Graphics, defect)

15 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: nrc, Assigned: nrc)

References

Details

Attachments

(1 file, 2 obsolete files)

We would like to be able to add fuzz to tests if canvas and content are rendered using different backends.
Attached patch patch (obsolete) — Splinter Review
Attachment #647048 - Flags: review?(roc)
Comment on attachment 647048 [details] [diff] [review] patch Review of attachment 647048 [details] [diff] [review]: ----------------------------------------------------------------- ::: layout/tools/reftest/reftest.js @@ +489,5 @@ > sandbox.azureQuartz = gfxInfo.getInfo().AzureCanvasBackend == "quartz"; > sandbox.azureSkia = gfxInfo.getInfo().AzureCanvasBackend == "skia"; > + // true if we are using a different Azure backend for rendering canvas and content > + sandbox.azureMismatch = gfxInfo.getInfo().AzureContentBackend != gfxInfo.getInfo().AzureCanvasBackend > + && !(gfxInfo.getInfo().AzureContentBackend == "none" && gfxInfo.getInfo().AzureCanvasBackend == "cairo"); How about "sandbox.contentSameGfxBackendAsCanvas"?
Attached patch patch (obsolete) — Splinter Review
changed the name of the flag
Attachment #647048 - Attachment is obsolete: true
Attachment #647048 - Flags: review?(roc)
Attachment #647069 - Flags: review?(roc)
Comment on attachment 647069 [details] [diff] [review] patch Review of attachment 647069 [details] [diff] [review]: ----------------------------------------------------------------- Hoist gfxInfo.getInfo() into a local variable, eh?
Attachment #647069 - Flags: review?(roc) → review+
Attached patch patchSplinter Review
hoisted gfxInfo.GetInfo(), carrying r=roc
Attachment #647069 - Attachment is obsolete: true
Attachment #647298 - Flags: review+
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: