Closed Bug 695912 Opened 12 years ago Closed 12 years ago

Add antialiasing to the downloadable blocklist

Categories

(Core :: Graphics: CanvasWebGL, defect)

Product:
Core
Component:
Graphics: CanvasWebGL
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla10

People

(Reporter: drs, Assigned: drs)

References

Details

Attachments

(2 files, 3 obsolete files)

Patch v1.0, force MSAA.
2.03 KB, patch
jgilbert
: review+
Details | Diff | Splinter Review
Patch v1.1, blocklisting for MSAA.
7.63 KB, patch
drs
: review+
Details | Diff | Splinter Review 
Support for blocking antialiasing if vendor/device/OS match a pre-determined bad setup must be added to the downloadable blocklist.
Assignee: nobody → dsherk
Depends on: 615976
Allows blocklisting of MSAA through downloaded blocklist or static code.
Attachment #568268 - Flags: review?(jgilbert)
Allows users to force MSAA on by setting the webgl.msaa-force pref to true.
Attachment #568269 - Flags: review?(jgilbert)
Comment on attachment 568269 [details] [diff] [review]
Patch v1.0, force MSAA.

># HG changeset patch
># Parent 54159f8b6372997bc05b2bfae2de89fb02c6e1d9
># User Doug Sherk <dsherk@mozilla.com>
>Bug 695912: allow forcing MSAA on r=jgilbert
[...]
>+    bool forceMSAA =
>+        Preferences::GetBool("webgl.msaa-force", false);

Since this should never change, I'd rather we use 'const bool' here. R+ though.
Attachment #568269 - Flags: review?(jgilbert) → review+
Updated to include ATI blocks.
Attachment #568268 - Attachment is obsolete: true
Attachment #568268 - Flags: review?(jgilbert)
Attachment #568548 - Flags: review?(jgilbert)
Removed accidentally added character in a comment.
Attachment #568548 - Attachment is obsolete: true
Attachment #568548 - Flags: review?(jgilbert)
Attachment #568552 - Flags: review?(jgilbert)
Comment on attachment 568552 [details] [diff] [review]
Patch v1.1, blocklisting for MSAA.

Good, R+ but:
>diff --git a/widget/src/xpwidgets/GfxInfoBase.cpp b/widget/src/xpwidgets/GfxInfoBase.cpp
>--- a/widget/src/xpwidgets/GfxInfoBase.cpp
>+++ b/widget/src/xpwidgets/GfxInfoBase.cpp
>@@ -152,16 +152,18 @@ GetPrefNameForFeature(PRInt32 aFeature)
>       name = BLACKLIST_PREF_BRANCH "layers.opengl";
>       break;
>     case nsIGfxInfo::FEATURE_WEBGL_OPENGL:
>       name = BLACKLIST_PREF_BRANCH "webgl.opengl";
>       break;
>     case nsIGfxInfo::FEATURE_WEBGL_ANGLE:
>       name = BLACKLIST_PREF_BRANCH "webgl.angle";
>       break;
>+    case nsIGfxInfo::FEATURE_WEBGL_MSAA:
>+      name = BLACKLIST_PREF_BRANCH "webgl.msaa";
>     default:
>       break;
>   };
> 
>   return name;
> }

Even though it falls through to 'default: break;', it should have its own 'break', for future-proofing.
Attachment #568552 - Flags: review?(jgilbert) → review+
Previous comment fixed, +r carried.
Attachment #568552 - Attachment is obsolete: true
Attachment #568562 - Flags: review+
Depends on: 668004
https://hg.mozilla.org/mozilla-central/rev/b7a96e8bec7e
https://hg.mozilla.org/mozilla-central/rev/5119eb1f191b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.