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

Unified Diff: Source/core/rendering/CompositedLayerMappingPtr.h

Issue 59063003: Don't coerce pointers to compositor layer mappings to booleans. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebaaaase Created 7 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
Index: Source/core/rendering/CompositedLayerMappingPtr.h
diff --git a/Source/core/rendering/CompositedLayerMappingPtr.h b/Source/core/rendering/CompositedLayerMappingPtr.h
new file mode 100644
index 0000000000000000000000000000000000000000..540781160529b83e2a23f812473caa400dc0b4d0
--- /dev/null
+++ b/Source/core/rendering/CompositedLayerMappingPtr.h
@@ -0,0 +1,28 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CompositedLayerMappingPtr_h
+#define CompositedLayerMappingPtr_h
+
+namespace WebCore {
+
+class CompositedLayerMapping;
+
+class CompositedLayerMappingPtr {
+public:
+ CompositedLayerMappingPtr(CompositedLayerMapping* mapping)
+ : m_mapping(mapping)
+ {
+ }
+
+ CompositedLayerMapping& operator*() const { return *m_mapping; }
shawnsingh 2013/11/05 10:20:27 should we ASSERT(m_mapping) here? I don't feel st
Ian Vollick 2013/11/08 03:35:12 Sure, done. I've also added the assert to 'operato
+ CompositedLayerMapping* operator->() const { return m_mapping; }
+
+private:
+ CompositedLayerMapping* m_mapping;
+};
+
+} // namespace WebCore
+
+#endif // CompositedLayerMappingPtr_h

Powered by Google App Engine
This is Rietveld 408576698