Switch content tests to use BrowserThreadImpl directly.

Switch content tests to use BrowserThreadImpl directly.

BUG=98716
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107964

[Jói, 2011-10-28 13:40:54]

I wanted your opinion on the using clauses before committing.  It seems to me
that this approach makes it easier for us once we move everything under content/
to the content namespace, then we can just do a search for "using content::",
delete those lines, and be done, without perturbing any formatting in the file.

Cheers,
Jói

[jam, 2011-10-28 17:00:21]

lgtm

This is the approach I've tried to use as well. Even in chrome code, it's
convenient to use the using keyword so that we have BrowserThread::PostTask(..
BrowserThread::UI instead of content::BrowserThread::PostTask(...
content::BrowserThread::UI). this is the convention that we use in the WebKit
API btw.

http://codereview.chromium.org/8400060/diff/1/content/browser/renderer_host/r...
File content/browser/renderer_host/render_widget_host_view_mac_unittest.mm
(right):

http://codereview.chromium.org/8400060/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_mac_unittest.mm:91:
scoped_ptr<content::BrowserThreadImpl> ui_thread_(
add using in this file?

[Jói, 2011-10-31 15:13:15]

http://codereview.chromium.org/8400060/diff/1/content/browser/renderer_host/r...
File content/browser/renderer_host/render_widget_host_view_mac_unittest.mm
(right):

http://codereview.chromium.org/8400060/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_mac_unittest.mm:91:
scoped_ptr<content::BrowserThreadImpl> ui_thread_(
On 2011/10/28 17:00:21, John Abd-El-Malek wrote:
> add using in this file?

Done.

[commit-bot: I haz the power, 2011-10-31 15:13:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/joi@chromium.org/8400060/6001

[commit-bot: I haz the power, 2011-10-31 16:28:05]

Change committed as 107964