OLD | NEW |
1 // Copyright 2015 The Chromium Authors. All rights reserved. | 1 // Copyright 2015 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 #include "core/frame/LocalFrameLifecycleNotifier.h" | 5 #include "core/frame/LocalFrameLifecycleNotifier.h" |
6 | 6 |
7 #include "core/frame/LocalFrameLifecycleObserver.h" | 7 #include "core/frame/LocalFrameLifecycleObserver.h" |
8 | 8 |
9 namespace blink { | 9 namespace blink { |
10 | 10 |
11 void LocalFrameLifecycleNotifier::notifyWillDetachFrameHost() | 11 void LocalFrameLifecycleNotifier::notifyWillDetachFrameHost() |
12 { | 12 { |
13 TemporaryChange<IterationType> scope(m_iterating, IteratingOverAll); | 13 TemporaryChange<IterationType> scope(m_iterating, IteratingOverAll); |
14 #if !ENABLE(OILPAN) | |
15 // Notifications perform unknown amounts of heap allocations, | |
16 // which might trigger (conservative) GCs. This will flush out | |
17 // dead observers, causing the _non-heap_ set be updated. Snapshot | |
18 // the observers and explicitly check if they're still alive before | |
19 // notifying. | |
20 Vector<RawPtr<LocalFrameLifecycleObserver>> snapshotOfObservers; | |
21 copyToVector(m_observers, snapshotOfObservers); | |
22 for (LocalFrameLifecycleObserver* observer : snapshotOfObservers) { | |
23 if (m_observers.contains(observer)) | |
24 observer->willDetachFrameHost(); | |
25 } | |
26 #else | |
27 for (LocalFrameLifecycleObserver* observer : m_observers) | 14 for (LocalFrameLifecycleObserver* observer : m_observers) |
28 observer->willDetachFrameHost(); | 15 observer->willDetachFrameHost(); |
29 #endif | |
30 } | 16 } |
31 | 17 |
32 } // namespace blink | 18 } // namespace blink |
OLD | NEW |