Closed
Bug 602289
Opened 14 years ago
Closed 14 years ago
Add Vendor() function to GLContext
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| blocking2.0 | --- | beta7+ |
People
(Reporter: bjacob, Assigned: bjacob)
References
Details
Attachments
(2 files)
|
5.28 KB,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
|
6.14 KB,
patch
|
bjacob
:
review+
|
Details | Diff | Splinter Review |
Only one glGetString call; also, adding a global inline DoesVendorStringMatch() function that does this parsing in a more fool-proof way than just calling strstr(), and use it also in GLContextProviderGLX.cpp for the xQueryServerString-returned vendor string.
Attachment #481306 -
Flags: review?(joe)
|
Assignee | |
Updated•14 years ago
|
Assignee: nobody → bjacob
|
||
Comment 1•14 years ago
|
||
Comment on attachment 481306 [details] [diff] [review] GL vendor detection Just create an enum for vendor type rather than making a bunch of bools.
Attachment #481306 -
Flags: review?(joe) → review+
|
|
||
Updated•14 years ago
|
blocking2.0: --- → beta7+
|
|
Assignee | |
Updated•14 years ago
|
Summary: Add IsATI(), IsNVIDIA() and IsIntel() functions to GLContext → Add Vendor() function to GLContext
|
|
Assignee | |
Comment 3•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/17531c7ef2a6
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•