Disable implicit conversion of scoped_refptr<T> to T* on Windows.

Disable implicit conversion of scoped_refptr<T> to T* on Windows.

Finally!!!

BUG=110610
Committed: https://crrev.com/1c3841d1c87f5332ba6550ed3b4f113c026b0ce3
Cr-Commit-Position: refs/heads/master@{#305558}

[dcheng, 2014-11-22 01:13:58]

dcheng@chromium.org changed reviewers:
+ thakis@chromium.org

[dcheng, 2014-11-22 01:14:45]

Note: I'm not sure if this is actually landable yet (I'm still finishing a local
Windows build), but it's made it 33% of the way without errors so far!

[Nico, 2014-11-22 01:19:25]

lgtm if the remaining 67% are happy too

https://codereview.chromium.org/748073004/diff/20001/base/memory/ref_counted.h
File base/memory/ref_counted.h (right):

https://codereview.chromium.org/748073004/diff/20001/base/memory/ref_counted....
base/memory/ref_counted.h:20: #if defined(OS_LINUX) || defined(OS_MACOSX) ||
defined(OS_IOS) || defined(OS_ANDROID) || defined(OS_WIN)
Is anything missing from this list? Might want to remove the macro completely?
(but can be a in a follow-up too)

[dcheng, 2014-11-22 01:20:22]

https://codereview.chromium.org/748073004/diff/20001/base/memory/ref_counted.h
File base/memory/ref_counted.h (right):

https://codereview.chromium.org/748073004/diff/20001/base/memory/ref_counted....
base/memory/ref_counted.h:20: #if defined(OS_LINUX) || defined(OS_MACOSX) ||
defined(OS_IOS) || defined(OS_ANDROID) || defined(OS_WIN)
On 2014/11/22 01:19:24, Nico wrote:
> Is anything missing from this list? Might want to remove the macro completely?
> (but can be a in a follow-up too)

I wanted to do that in a followup--just in case there's obscure platform I
forgot to handle.

[dcheng, 2014-11-24 23:07:49]

The CQ bit was checked by dcheng@chromium.org

[dcheng, 2014-11-24 23:07:50]

I've landed all blockers to this patch that I found over the weekend...
hopefully nothing else snuck in since then.

[commit-bot: I haz the power, 2014-11-24 23:08:24]

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

[commit-bot: I haz the power, 2014-11-25 01:49:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-25 01:50:35]

Patchset 2 (id:??) landed as
https://crrev.com/1c3841d1c87f5332ba6550ed3b4f113c026b0ce3
Cr-Commit-Position: refs/heads/master@{#305558}