Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3186)

Unified Diff: base/win/scoped_comptr.h

Issue 719363002: Remove implicit conversions from scoped_refptr to T* in base/win/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | base/win/scoped_comptr_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/win/scoped_comptr.h
diff --git a/base/win/scoped_comptr.h b/base/win/scoped_comptr.h
index 98cea0ff4c8a45f8612cdab90e7882d85b7e25c1..28906714ec45c4d3b9a9de4a6b155478368eaeca 100644
--- a/base/win/scoped_comptr.h
+++ b/base/win/scoped_comptr.h
@@ -133,8 +133,8 @@ class ScopedComPtr : public scoped_refptr<Interface> {
ScopedComPtr<IUnknown> other_identity;
other->QueryInterface(other_identity.Receive());
- return static_cast<IUnknown*>(my_identity) ==
- static_cast<IUnknown*>(other_identity);
+ return static_cast<IUnknown*>(my_identity.get()) ==
+ static_cast<IUnknown*>(other_identity.get());
dcheng 2014/11/12 23:37:54 I'm reverting this, because once the implicit conv
}
// Provides direct access to the interface.
« no previous file with comments | « no previous file | base/win/scoped_comptr_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698