Win AcceleratedSurface no longer polls queries on separate thread.

Win AcceleratedSurface no longer polls queries on separate thread.

This is basically a reversion of the final patch set (#4) of this:
http://codereview.chromium.org/8395012

The reason is because tasks posted to the query thread can extend the lifetime of the D3D device such that it is destroyed on the query thread. The D3D thread must be destroyed on the thread it was created on though.

The suggestion was a good one and I might revisit it in the future.

I also added a missing lock and switched it to load the D3D9 DLL rather than assume it is already loaded by something else.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113529

[jbauman, 2011-12-07 23:46:00]

lgtm

[apatrick_chromium, 2011-12-08 01:49:43]

+kbr

[Ken Russell (switch to Gerrit), 2011-12-08 01:55:07]

LGTM; seems much simpler.

http://codereview.chromium.org/8867005/diff/1/ui/gfx/surface/accelerated_surf...
File ui/gfx/surface/accelerated_surface_win.cc (right):

http://codereview.chromium.org/8867005/diff/1/ui/gfx/surface/accelerated_surf...
ui/gfx/surface/accelerated_surface_win.cc:373: } while (hr == S_FALSE);
As an FYI a problem in a similarly structured loop was reported in
http://code.google.com/p/angleproject/issues/detail?id=262 .