Clean up scoped_com_initializer.h. Remove #ifdefs for non-Windows (this is in base/win, no one sho…

Clean up scoped_com_initializer.h, primarily by removing #ifdefs for non-Windows.  Instead change files that use this to specifically #ifdef it for OS_WIN, like we do most other Windows-specific code.

I plan to factor out the "subclass base::Thread to init MTA on Windows" pattern to its own file in a followup change.

BUG=none
TEST=none
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=159545

[Peter Kasting, 2012-09-29 21:47:16]

ping

[cpu_(ooo_6.6-7.5), 2012-10-01 19:47:27]

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
File content/browser/renderer_host/media/media_stream_manager.cc (left):

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
content/browser/renderer_host/media/media_stream_manager.cc:59: Stop();
did you mean to remove this stop()?

I don't see an override here so I thinking is ok but just checking.

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
File content/browser/renderer_host/media/media_stream_manager.cc (right):

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
content/browser/renderer_host/media/media_stream_manager.cc:57: #if
defined(OS_WIN)
does block 58-62 need to be ifdef(os_win) ?

[cpu_(ooo_6.6-7.5), 2012-10-01 19:55:19]

I swear I made comments before from home. Maybe I used the google account and
forgot to submit.

lgtm

[Peter Kasting, 2012-10-01 20:23:17]

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
File content/browser/renderer_host/media/media_stream_manager.cc (left):

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
content/browser/renderer_host/media/media_stream_manager.cc:59: Stop();
On 2012/10/01 19:47:27, cpu wrote:
> did you mean to remove this stop()?
> 
> I don't see an override here so I thinking is ok but just checking.

I did mean to remove it, because the superclass destructor already calls it, so
it's just redundant.

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
File content/browser/renderer_host/media/media_stream_manager.cc (right):

http://codereview.chromium.org/10990079/diff/1005/content/browser/renderer_ho...
content/browser/renderer_host/media/media_stream_manager.cc:57: #if
defined(OS_WIN)
On 2012/10/01 19:47:27, cpu wrote:
> does block 58-62 need to be ifdef(os_win) ?

Yes, see the header file changes.

[Peter Kasting, 2012-10-01 20:37:09]

brettw, can you OWNERS r+?

[brettw, 2012-10-01 20:39:54]

lgtm rubberstamp