Closed Bug 1344433 Opened 3 years ago Closed 3 years ago

Cleanup gfx/gl library loading

Categories

(Core :: Graphics, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jgilbert, Assigned: jgilbert)

Details

(Whiteboard: gfx-noted)

Attachments

(6 files)

No description provided.
Comment on attachment 8843624 [details]
Bug 1344433 - Remove HasDXInterop(), but not HasDXInterop2(). -

https://reviewboard.mozilla.org/r/117258/#review119044
Attachment #8843624 - Flags: review?(dmu) → review+
Comment on attachment 8843625 [details]
Bug 1344433 - Require GLX 1.4 (released 2005) -

https://reviewboard.mozilla.org/r/117260/#review119048

::: gfx/gl/GLContextProviderGLX.cpp:1320
(Diff revision 1)
>          return nullptr;
>      }
>  
>      // Handle slightly different signature between glXCreatePixmap and
>      // its pre-GLX-1.3 extension equivalent (though given the ABI, we
>      // might not need to).

You may need to modify the comments here if you wanna upgrade to support GLX 1.4 only.
Attachment #8843625 - Flags: review?(dmu) → review+
Comment on attachment 8843625 [details]
Bug 1344433 - Require GLX 1.4 (released 2005) -

https://reviewboard.mozilla.org/r/117260/#review119050

r=me for fixing the nits of the comments.
Comment on attachment 8843626 [details]
Bug 1344433 - Reformat. -

https://reviewboard.mozilla.org/r/117262/#review119098

LGTM
Attachment #8843626 - Flags: review?(dmu) → review+
Comment on attachment 8843627 [details]
Bug 1344433 - Use WRAP() macros. -

https://reviewboard.mozilla.org/r/117264/#review119102

LGTM
Attachment #8843627 - Flags: review?(dmu) → review+
Comment on attachment 8843628 [details]
Bug 1344433 - Fix ANGLE symbol loading. -

https://reviewboard.mozilla.org/r/117266/#review119116
Attachment #8843628 - Flags: review?(dmu) → review+
Comment on attachment 8844158 [details]
Bug 1344433 - Allow GLX 1.3+glXGetProcAddressARB. -

https://reviewboard.mozilla.org/r/117694/#review119392
Attachment #8844158 - Flags: review?(jmuizelaar) → review+
Pushed by jgilbert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/96bb57eaa0e2
Remove HasDXInterop(), but not HasDXInterop2(). - r=daoshengmu
https://hg.mozilla.org/integration/mozilla-inbound/rev/5dd77a2bc068
Require GLX 1.4 (released 2005) - r=daoshengmu
https://hg.mozilla.org/integration/mozilla-inbound/rev/a2624aa679c1
Reformat. - r=daoshengmu
https://hg.mozilla.org/integration/mozilla-inbound/rev/b90bf52c69ea
Use WRAP() macros. - r=daoshengmu
https://hg.mozilla.org/integration/mozilla-inbound/rev/11ca2ac87fbd
Fix ANGLE symbol loading. - r=daoshengmu
https://hg.mozilla.org/integration/mozilla-inbound/rev/c726adc9b217
Allow GLX 1.3+glXGetProcAddressARB. - r=jrmuizel
You need to log in before you can comment on or make changes to this bug.