Disable DRI3 temporarily on Linux desktop.

Disable DRI3 temporarily on Linux desktop.

The GPU process is crashing on DRI3-enabled desktop Linux systems.

BUG=415681
TEST=On a DRI3 system (Ubuntu 14.10, Intel GPU), GPU process doesn't crash.

Committed: https://crrev.com/caa8c58cb570091ea27d8d75a406022ef8e0e82d
Cr-Commit-Position: refs/heads/master@{#303269}

[Jorge Lucangeli Obes, 2014-11-07 06:26:09]

jorgelo@chromium.org changed reviewers:
+ jln@chromium.org, kbr@chromium.org, leecam@chromium.org, marcheu@chromium.org,
mdempsky@chromium.org

[Jorge Lucangeli Obes, 2014-11-07 06:26:32]

This is probably the smallest fix we can do.

[mdempsky, 2014-11-07 06:40:33]

lgtm

https://chromiumcodereview.appspot.com/708043002/diff/20001/content/gpu/gpu_m...
File content/gpu/gpu_main.cc (right):

https://chromiumcodereview.appspot.com/708043002/diff/20001/content/gpu/gpu_m...
content/gpu/gpu_main.cc:220: scoped_ptr<base::Environment>
env(base::Environment::Create());
nit: Since GpuMain() doesn't return, it might be worth wrapping these two
statements in a block?  Otherwise I think the Environment object will never get
destroyed.

[Jorge Lucangeli Obes, 2014-11-07 15:53:19]

I have confirmed this works on 14.10. As soon as I get an el-gee-tee-em from
gpu_main.cc owners, I can submit.

[Jorge Lucangeli Obes, 2014-11-07 15:53:48]

https://chromiumcodereview.appspot.com/708043002/diff/20001/content/gpu/gpu_m...
File content/gpu/gpu_main.cc (right):

https://chromiumcodereview.appspot.com/708043002/diff/20001/content/gpu/gpu_m...
content/gpu/gpu_main.cc:220: scoped_ptr<base::Environment>
env(base::Environment::Create());
On 2014/11/07 06:40:33, mdempsky (Tokyo until Nov 14) wrote:
> nit: Since GpuMain() doesn't return, it might be worth wrapping these two
> statements in a block?  Otherwise I think the Environment object will never
get
> destroyed.

Done.

[marcheu, 2014-11-07 17:37:18]

On 2014/11/07 15:53:48, Jorge Lucangeli Obes wrote:
>
https://chromiumcodereview.appspot.com/708043002/diff/20001/content/gpu/gpu_m...
> File content/gpu/gpu_main.cc (right):
> 
>
https://chromiumcodereview.appspot.com/708043002/diff/20001/content/gpu/gpu_m...
> content/gpu/gpu_main.cc:220: scoped_ptr<base::Environment>
> env(base::Environment::Create());
> On 2014/11/07 06:40:33, mdempsky (Tokyo until Nov 14) wrote:
> > nit: Since GpuMain() doesn't return, it might be worth wrapping these two
> > statements in a block?  Otherwise I think the Environment object will never
> get
> > destroyed.
> 
> Done.

lgtm but I'm not an owner

[Ken Russell (switch to Gerrit), 2014-11-07 18:13:55]

lgtm

[Jorge Lucangeli Obes, 2014-11-07 18:17:39]

The CQ bit was checked by jorgelo@chromium.org

[commit-bot: I haz the power, 2014-11-07 18:18:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/708043002/40001

[commit-bot: I haz the power, 2014-11-07 19:21:10]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-11-07 19:21:58]

Patchset 3 (id:??) landed as
https://crrev.com/caa8c58cb570091ea27d8d75a406022ef8e0e82d
Cr-Commit-Position: refs/heads/master@{#303269}