Use nv contrl x extension to query nvidia driver version.

Use nv contrl x extension to query nvidia driver version.

So we can do it in browser process, not wait until gpu process launches and a gl context is created.

This is tested on my linux box with NVIDIA driver.

We need to put more code to tell chrome (Linux/NVIDIA) that we have enough information for blacklisting, so at GPU launch time we don't need to collect GPUInfo.

Will do it in a followup CL.

BUG=122912
TEST=tree green
TBR=jam
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=143323

[Zhenyao Mo, 2012-06-20 23:44:52]

This CL depends on the landing of libXNVCtrl
(http://codereview.chromium.org/10578052/), but it can be reviewed without it.

Please have a look.

[piman, 2012-06-21 00:08:32]

LGTM + nit

https://chromiumcodereview.appspot.com/10536232/diff/3001/content/gpu/gpu_inf...
File content/gpu/gpu_info_collector_linux.cc (right):

https://chromiumcodereview.appspot.com/10536232/diff/3001/content/gpu/gpu_inf...
content/gpu/gpu_info_collector_linux.cc:179: int event_base, error_base;
nit: =0

[Ken Russell (switch to Gerrit), 2012-06-21 00:30:54]

LGTM too

https://chromiumcodereview.appspot.com/10536232/diff/3001/content/gpu/gpu_inf...
File content/gpu/gpu_info_collector_linux.cc (right):

https://chromiumcodereview.appspot.com/10536232/diff/3001/content/gpu/gpu_inf...
content/gpu/gpu_info_collector_linux.cc:174: Display* display =
base::MessagePumpForUI::GetDefaultXDisplay();
Please make sure there is an assertion somewhere (if not in the internals of the
above call, add one) that it's being called on the correct thread. There's been
a lot of churn about which thread the GpuInfoCollector calls are made upon.

[Zhenyao Mo, 2012-06-21 00:45:43]

http://codereview.chromium.org/10536232/diff/3001/content/gpu/gpu_info_collec...
File content/gpu/gpu_info_collector_linux.cc (right):

http://codereview.chromium.org/10536232/diff/3001/content/gpu/gpu_info_collec...
content/gpu/gpu_info_collector_linux.cc:174: Display* display =
base::MessagePumpForUI::GetDefaultXDisplay();
On 2012/06/21 00:30:54, kbr wrote:
> Please make sure there is an assertion somewhere (if not in the internals of
the
> above call, add one) that it's being called on the correct thread. There's
been
> a lot of churn about which thread the GpuInfoCollector calls are made upon.

Done.

http://codereview.chromium.org/10536232/diff/3001/content/gpu/gpu_info_collec...
content/gpu/gpu_info_collector_linux.cc:179: int event_base, error_base;
On 2012/06/21 00:08:32, piman wrote:
> nit: =0

Done.

[jam, 2012-06-21 02:48:14]

please don't commit without running a try run or CQ. This change broke the tree.