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

Side by Side Diff: third_party/WebKit/Source/modules/canvas2d/HitRegion.h

Issue 1929493002: Remove unnecessary uses of GarbageCollectedFinalized<>. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add two leftover classes needing same treatment Created 4 years, 7 months 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef HitRegion_h 5 #ifndef HitRegion_h
6 #define HitRegion_h 6 #define HitRegion_h
7 7
8 #include "core/dom/Element.h" 8 #include "core/dom/Element.h"
9 #include "modules/canvas2d/HitRegionOptions.h" 9 #include "modules/canvas2d/HitRegionOptions.h"
10 #include "platform/graphics/Path.h" 10 #include "platform/graphics/Path.h"
(...skipping 27 matching lines...) Expand all
38 38
39 private: 39 private:
40 HitRegion(const Path&, const HitRegionOptions&); 40 HitRegion(const Path&, const HitRegionOptions&);
41 41
42 String m_id; 42 String m_id;
43 Member<Element> m_control; 43 Member<Element> m_control;
44 Path m_path; 44 Path m_path;
45 WindRule m_fillRule; 45 WindRule m_fillRule;
46 }; 46 };
47 47
48 class HitRegionManager final : public GarbageCollectedFinalized<HitRegionManager > { 48 class HitRegionManager final : public GarbageCollected<HitRegionManager> {
49 WTF_MAKE_NONCOPYABLE(HitRegionManager); 49 WTF_MAKE_NONCOPYABLE(HitRegionManager);
50 public: 50 public:
51 static HitRegionManager* create() { return new HitRegionManager; } 51 static HitRegionManager* create() { return new HitRegionManager; }
52 ~HitRegionManager() { }
53 52
54 void addHitRegion(HitRegion*); 53 void addHitRegion(HitRegion*);
55 54
56 void removeHitRegion(HitRegion*); 55 void removeHitRegion(HitRegion*);
57 void removeHitRegionById(const String& id); 56 void removeHitRegionById(const String& id);
58 void removeHitRegionByControl(const Element*); 57 void removeHitRegionByControl(const Element*);
59 void removeHitRegionsInRect(const FloatRect&, const AffineTransform&); 58 void removeHitRegionsInRect(const FloatRect&, const AffineTransform&);
60 void removeAllHitRegions(); 59 void removeAllHitRegions();
61 60
62 HitRegion* getHitRegionById(const String& id) const; 61 HitRegion* getHitRegionById(const String& id) const;
(...skipping 13 matching lines...) Expand all
76 typedef HeapHashMap<Member<const Element>, Member<HitRegion>> HitRegionContr olMap; 75 typedef HeapHashMap<Member<const Element>, Member<HitRegion>> HitRegionContr olMap;
77 76
78 HitRegionList m_hitRegionList; 77 HitRegionList m_hitRegionList;
79 HitRegionIdMap m_hitRegionIdMap; 78 HitRegionIdMap m_hitRegionIdMap;
80 HitRegionControlMap m_hitRegionControlMap; 79 HitRegionControlMap m_hitRegionControlMap;
81 }; 80 };
82 81
83 } // namespace blink 82 } // namespace blink
84 83
85 #endif 84 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698