implement external image handler in RendererOGL

RESOLVED FIXED in Firefox 54

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jerry, Assigned: jerry)

Tracking

unspecified
mozilla54
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(2 attachments, 2 obsolete attachments)

When we try to pass a external gl texture to WR, the gl texture should use the same gl context as the RendererOGL has. So, I think it's better to implement the handler in RendererOGL.
Created attachment 8839089 [details] [diff] [review]
rename the external image related structure in webrender_ffi.

The name of "WrExternalImageIdId" is a little bit strange.

MozReview-Commit-ID: 8pfWxg5XuYS
Attachment #8839089 - Flags: review?(nical.bugzilla)
Created attachment 8839090 [details] [diff] [review]
add external image callback in RendererOGL.

MozReview-Commit-ID: LfjnTRwUCao
Attachment #8839090 - Flags: review?(nical.bugzilla)
Comment on attachment 8839090 [details] [diff] [review]
add external image callback in RendererOGL.

Review of attachment 8839090 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/webrender_bindings/WebRenderAPI.cpp
@@ +47,5 @@
>  
>      gl->fGetIntegerv(LOCAL_GL_MAX_TEXTURE_SIZE, mMaxTextureSize);
>  
>      WrRenderer* wrRenderer = nullptr;
> +    if (!wr_window_new(aWindowId, gl.get(), this->mEnableProfiler, mWrApi, &wrRenderer)) {

When we call wr_window_new, the rendererOGL is not ready here. So, move the setting of externalImageHandler to another function.

Updated

2 years ago
See Also: → bug 1341235

Updated

2 years ago
Blocks: 1341235
Created attachment 8842323 [details] [diff] [review]
rebase
Attachment #8842323 - Flags: review?(sotaro.ikeda.g)
(Assignee)

Updated

2 years ago
Attachment #8839089 - Attachment is obsolete: true
Attachment #8839089 - Flags: review?(nical.bugzilla)
Created attachment 8842324 [details] [diff] [review]
rebase
Attachment #8842324 - Flags: review?(sotaro.ikeda.g)
(Assignee)

Updated

2 years ago
Attachment #8839090 - Attachment is obsolete: true
Attachment #8839090 - Flags: review?(nical.bugzilla)

Updated

2 years ago
Attachment #8842323 - Flags: review?(sotaro.ikeda.g) → review+

Updated

2 years ago
Attachment #8842324 - Flags: review?(sotaro.ikeda.g) → review+

Comment 6

2 years ago
Pushed by hshih@mozilla.com:
https://hg.mozilla.org/projects/graphics/rev/169c2f5176bb
rename the external image related structure in webrender_ffi. r=sotaro
https://hg.mozilla.org/projects/graphics/rev/e306b40ff823
add external image callback in RendererOGL. r=sotaro
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.