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

Side by Side Diff: third_party/WebKit/Source/core/observer/ResizeObserverController.h

Issue 2005593002: Initial ResizeObserver implementation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef ResizeObserverController_h
6 #define ResizeObserverController_h
7
8 #include "platform/heap/Handle.h"
9
10 namespace blink {
11
12 class ResizeObserver;
13
14 // ResizeObserverController keeps track of all ResizeObservers
15 // in a single Document.
16 class ResizeObserverController : public GarbageCollectedFinalized<ResizeObserver Controller> {
17
18 public:
19
20 static const size_t kRenderLoopLimit = 16;
21
22 ResizeObserverController();
23 virtual ~ResizeObserverController() {};
szager1 2016/05/23 19:49:12 Ditto comment about unnecessary destructors.
atotic1 2016/05/25 00:15:54 Done.
24
25 void addObserver(ResizeObserver&);
26
szager1 2016/05/23 19:49:12 Too many blank lines.
atotic1 2016/05/25 00:15:54 Done.
27 void gatherObservations();
28
29 bool hasObservations();
30
31 void deliverObservations();
32
33 DECLARE_TRACE();
34
35 private:
36
37 // Active observers
38 HeapHashSet<Member<ResizeObserver>> m_observers;
szager1 2016/05/23 19:49:12 This should probably be HeapHashSet<WeakMember<Res
atotic1 2016/05/25 00:15:54 Done.
39 };
40
41 } // namespace blink
42
43 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698