Closed
Bug 1391136
Opened 7 years ago
Closed 7 years ago
Handle WebGL context lost in layers-free mode.
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: mtseng, Assigned: mtseng)
References
Details
Attachments
(1 file)
Originally, we check IsContextLost in WebGLContext::GetCanvasLayer(). However, in layers-free mode, we don't use GetCanvasLayer, we'll call InitializeCanvasRenderer. So I move IsContextLost check to InitializeCanvasRenderer.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8898085 [details] Bug 1391136 - Handle context lost in layers-free mode. https://reviewboard.mozilla.org/r/169416/#review174992 ::: layout/generic/nsHTMLCanvasFrame.cpp:144 (Diff revision 1) > aManager->CreateOrRecycleWebRenderUserData<WebRenderCanvasData>(this, &isRecycled); > WebRenderCanvasRendererAsync* data = > static_cast<WebRenderCanvasRendererAsync*>(canvasData->GetCanvasRenderer()); > > if (!isRecycled) { > - static_cast<nsHTMLCanvasFrame*>(mFrame)->InitializeCanvasRenderer(aDisplayListBuilder, data); > + nsHTMLCanvasFrame *canvasFrame = static_cast<nsHTMLCanvasFrame*>(mFrame); Star to the left, against the type.
Attachment #8898085 -
Flags: review?(jgilbert) → review+
Assignee | ||
Comment 3•7 years ago
|
||
try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1d935248b404c5532efa54d00137528723bf624f
Pushed by mtseng@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/682c2b840724 Handle context lost in layers-free mode. r=jgilbert
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/682c2b840724
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•