Closed Bug 1318066 Opened 4 years ago Closed 4 years ago
Package up the webrender shaders so that they are in the installer
Right now the webrender binding code just reads the WR shaders from the res/ subfolder under the cargo manifest path, but this is evaluated at compile-time. If the build is compiled on one machine and moved to another machine then it fails on startup because it can't find the path. https://github.com/servo/webrender/issues/352 covers a webrender change to compile in the shaders but until that is done we can work around it by packaging the shaders as part of the firefox packaging process. This can be backed out later once we get a webrender update that includes compiled-in shaders.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
That was quicker than I anticipated: vlad landed a patch in webrender to compile in the shaders in https://github.com/servo/webrender/pull/563 When we update our copy of webrender to a version that includes that, we can back out these patches.
ni myself so I don't forget to do this backout after the update.
Depends on: 1318440
Pushed by email@example.com: https://hg.mozilla.org/projects/graphics/rev/d17759dd3b74 Revert the changes for packing the webrender shaders. r=kats?
Hi Kats, Bug 1318613 includes the PR: https://github.com/servo/webrender/pull/563 . So, comment 4 do the reversion.
Resolution: FIXED → WORKSFORME
You need to log in before you can comment on or make changes to this bug.