Closed
Bug 1322500
Opened 7 years ago
Closed 7 years ago
Use CanvasClient in WebRenderCanvasLayer
Categories
(Core :: Graphics: WebRender, defect)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: sotaro, Assigned: mtseng)
References
Details
Attachments
(3 files)
28.38 KB,
patch
|
sotaro
:
review+
|
Details | Diff | Splinter Review |
7.87 KB,
patch
|
Details | Diff | Splinter Review | |
34.81 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•7 years ago
|
||
Per discussion with Sotaro, I'm working on using CanvasClient for WebRenderCanvasLayer currently. So I'm going to steal this bug.
Assignee: sotaro.ikeda.g → mtseng
Assignee | ||
Updated•7 years ago
|
Summary: Use TextureClient in WebRenderCanvasLayer → Use CanvasClient in WebRenderCanvasLayer
Assignee | ||
Comment 2•7 years ago
|
||
Having a workable patch for webgl but not for canvas2d. I'll upload patches once the 2d part is done.
Assignee | ||
Comment 3•7 years ago
|
||
I share many codes in ClientCanvasLayer when I try to use CanvasClient in WebRenderCanvasLayer. So I extracted common functions to a based class which called ShareableCanvasLayer. This patch is going to land in m-c.
Attachment #8821419 -
Flags: review?(sotaro.ikeda.g)
Assignee | ||
Comment 4•7 years ago
|
||
Following works on webrender branch.
Assignee | ||
Comment 5•7 years ago
|
||
Try link for first patch https://treeherder.mozilla.org/#/jobs?repo=try&revision=41f10654255f8c2e9e0ccace96368bd90c904260
Reporter | ||
Comment 6•7 years ago
|
||
Comment on attachment 8821419 [details] [diff] [review] Introducing ShareableCanvasLayer. Is it possible to create a patch by using |hg cp| like Bug 1316479 Comment 2?
Attachment #8821419 -
Flags: review?(sotaro.ikeda.g) → review+
Assignee | ||
Comment 7•7 years ago
|
||
Sure, I'll use |hg cp| for creating a new patch.
Assignee | ||
Comment 8•7 years ago
|
||
MozReview-Commit-ID: I22lXigQHZM
Assignee | ||
Comment 9•7 years ago
|
||
(In reply to Morris Tseng [:mtseng] [:Morris] from comment #8) > Created attachment 8821971 [details] [diff] [review] > Introducing ShareableCanvasLayer > > MozReview-Commit-ID: I22lXigQHZM This is patch which generated by |hg cp|
Assignee | ||
Comment 10•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=dc17021ce061d869fd8f47b525506b2a16d4d5aa new try.
Comment 11•7 years ago
|
||
Pushed by mtseng@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c86f74a55fa8 Introducing ShareableCanvasLayer. r=sotaro
Comment 12•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c86f74a55fa8
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Assignee | ||
Comment 13•7 years ago
|
||
Also need webrender part to make everything works. Keep this bug open.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 14•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b63b2bdf6310dd876f330f7257d32bd2669594b Try with webrender part.
Assignee | ||
Comment 15•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c52d2983b08002660125f44cab6af1a40a84c4f8 latest try
Comment 16•7 years ago
|
||
Pushed by mtseng@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/f7db1b467195 Using CanvasClient for WebRenderCanvasLayer. r=sotaro? https://hg.mozilla.org/projects/graphics/rev/8ea83e47ef87 Mark some webgl tests which are passed now. r=gfx?
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•