Chromium Code Reviews| Index: Source/core/html/canvas/CanvasRenderingContext2D.h |
| diff --git a/Source/core/html/canvas/CanvasRenderingContext2D.h b/Source/core/html/canvas/CanvasRenderingContext2D.h |
| index bb836747792bc4b3c77c54a9be40c87583622d6c..6511d86d4216df0633272745c14e08c482bd1603 100644 |
| --- a/Source/core/html/canvas/CanvasRenderingContext2D.h |
| +++ b/Source/core/html/canvas/CanvasRenderingContext2D.h |
| @@ -63,13 +63,13 @@ class ImageBitmap; |
| class ImageData; |
| class TextMetrics; |
| -typedef WillBePersistentHeapHashMap<String, RefPtrWillBeMember<MutableStylePropertySet> > MutableStylePropertyMap; |
| +typedef WillBeHeapHashMap<String, RefPtrWillBeMember<MutableStylePropertySet> > MutableStylePropertyMap; |
| -class CanvasRenderingContext2D FINAL: public ScriptWrappable, public CanvasRenderingContext, public CanvasPathMethods { |
| +class CanvasRenderingContext2D FINAL: public CanvasRenderingContext, public ScriptWrappable, public CanvasPathMethods { |
| public: |
| - static PassOwnPtr<CanvasRenderingContext2D> create(HTMLCanvasElement* canvas, const Canvas2DContextAttributes* attrs, bool usesCSSCompatibilityParseMode) |
| + static PassOwnPtrWillBeRawPtr<CanvasRenderingContext2D> create(HTMLCanvasElement* canvas, const Canvas2DContextAttributes* attrs, bool usesCSSCompatibilityParseMode) |
| { |
| - return adoptPtr(new CanvasRenderingContext2D(canvas, attrs, usesCSSCompatibilityParseMode)); |
| + return adoptPtrWillBeNoop(new CanvasRenderingContext2D(canvas, attrs, usesCSSCompatibilityParseMode)); |
| } |
| virtual ~CanvasRenderingContext2D(); |
| @@ -233,6 +233,8 @@ public: |
| void loseContext(); |
| void restoreContext(); |
| + virtual void trace(Visitor*) OVERRIDE; |
| + |
| private: |
| class State FINAL : public CSSFontSelectorClient { |
| public: |
| @@ -345,7 +347,7 @@ private: |
| // FIXME: Oilpan: Make this a vector of embedded State objects rather than pointers |
| // once we support having vectors with objects using a vtable in oilpan. |
|
Erik Corry
2014/04/25 09:23:48
Does this comment make sense? There are lots of su
Mads Ager (chromium)
2014/04/25 10:58:25
I'll remove the comment (which is unrelated to my
|
| - WillBePersistentHeapVector<OwnPtrWillBeMember<State> > m_stateStack; |
| + WillBeHeapVector<OwnPtrWillBeMember<State> > m_stateStack; |
| bool m_usesCSSCompatibilityParseMode; |
| bool m_hasAlpha; |
| bool m_isContextLost; |