Closed
Bug 1314133
Opened 8 years ago
Closed 8 years ago
Enable the GPU process in nightly, on Windows
Categories
(Core :: Graphics, defect)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox52 | --- | fixed |
People
(Reporter: dvander, Assigned: dvander)
References
(Depends on 2 open bugs)
Details
Attachments
(2 files)
1.15 KB,
patch
|
milan
:
review+
|
Details | Diff | Splinter Review |
8.65 KB,
application/x-zip-compressed
|
Details |
This will just be for Nightly, and only for Windows users.
Assignee | ||
Comment 1•8 years ago
|
||
Enables out-of-process compositing for Nightly users that meet the following conditions: (1) Use Windows 7 with the Platform Update, or higher, and (2) Are able to use Direct3D 11 compositing. Intent is not to check this in until blocking bugs are fixed.
Attachment #8806109 -
Flags: review?(milan)
Comment 2•8 years ago
|
||
Comment on attachment 8806109 [details] [diff] [review] patch Review of attachment 8806109 [details] [diff] [review]: ----------------------------------------------------------------- ::: modules/libpref/init/all.js @@ +4533,5 @@ > // Preference that when switched at runtime will run a series of benchmarks > // and output the result to stderr. > pref("layers.bench.enabled", false); > > +#ifdef XP_WIN && defined(NIGHTLY_BUILD) Drive-by: this syntax seems wrong. shouldn't be it #if defined(XP_WIN) && defined(NIGHTLY_BUILD) ?
Assignee | ||
Comment 3•8 years ago
|
||
It looks weird, but I literally copied it from elsewhere in the file. I can change it though.
Comment 4•8 years ago
|
||
Might be broken in the other place too, who knows. Or maybe our in-house preprocessor accidentally does the right thing on this syntax?
Assignee | ||
Comment 5•8 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #4) > Might be broken in the other place too, who knows. Or maybe our in-house > preprocessor accidentally does the right thing on this syntax? It parses fine but seems to evaluate to #if 0. Hah. Thanks, I'll put a new patch up.
Comment on attachment 8806109 [details] [diff] [review] patch Review of attachment 8806109 [details] [diff] [review]: ----------------------------------------------------------------- \o/ Kats' comment non-withstanding.
Attachment #8806109 -
Flags: review?(milan) → review+
Pushed by danderson@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/59d33178b0cd Enable out-of-process D3D11 compositing for Nightly Windows users. (bug 1314133, r=milan)
Comment 8•8 years ago
|
||
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/28ed60ea7e1117144e10ed931df1dfc9437dacbf for https://treeherder.mozilla.org/logviewer.html#?job_id=38536570&repo=mozilla-inbound
Assignee | ||
Comment 9•8 years ago
|
||
I can reproduce this, investigating.
Comment 10•8 years ago
|
||
When I manually add "layers.gpu-process.dev.enabled = true" I get this in about:support graphics section: Hardware H264 Decoding Yes; Failed to create D3D11 device for decoder; Using D3D9 API
Comment 11•8 years ago
|
||
(In reply to Gary [:streetwolf] from comment #10) > When I manually add "layers.gpu-process.dev.enabled = true" I get this in > about:support graphics section: > > Hardware H264 Decoding Yes; Failed to create D3D11 device for decoder; Using > D3D9 API Can you please attach your full about:support from when this happens?
Comment 12•8 years ago
|
||
The zip files contains: support_false.txt = about:support with "layers.gpu-process.dev.enabled = false" support_true.txt = about:support with "layers.gpu-process.dev.enabled = true"
Comment 13•8 years ago
|
||
(In reply to Gary [:streetwolf] from comment #12) > Created attachment 8806898 [details] > Support.zip > > The zip files contains: > > support_false.txt = about:support with "layers.gpu-process.dev.enabled = > false" > support_true.txt = about:support with "layers.gpu-process.dev.enabled = true" Thanks! Looks like this is a mainly a reporting problem, you should still be getting D3D11 DXVA for your videos. I've filed bug 1314803 and bug 1314804 to track fixing this.
Comment 14•8 years ago
|
||
Pushed by danderson@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b4ade2b0841c Enable the GPU process for Nightly Windows users. (bug 1314133, r=milan)
Comment 15•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b4ade2b0841c
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Comment 16•8 years ago
|
||
Although I have no idea why, and I personally cannot reproduce it, there are reports from Japanese people that this causes issue with printing.
Depends on: 1318845
You need to log in
before you can comment on or make changes to this bug.
Description
•