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

Unified Diff: third_party/WebKit/Source/platform/heap/PersistentNode.cpp

Issue 2384213003: reflow comments in platform/heap (Closed)
Patch Set: Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/heap/PersistentNode.cpp
diff --git a/third_party/WebKit/Source/platform/heap/PersistentNode.cpp b/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
index 0235ce47b7df6cf70776e62c87299ff6dff73876..cb47bcf10b39766b61a1f4f99d3d733d0381e78f 100644
--- a/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
+++ b/third_party/WebKit/Source/platform/heap/PersistentNode.cpp
@@ -135,21 +135,22 @@ bool CrossThreadPersistentRegion::shouldTracePersistentNode(
void CrossThreadPersistentRegion::prepareForThreadStateTermination(
ThreadState* threadState) {
- // For heaps belonging to a thread that's detaching, any cross-thread persistents
- // pointing into them needs to be disabled. Do that by clearing out the underlying
- // heap reference.
+ // For heaps belonging to a thread that's detaching, any cross-thread
+ // persistents pointing into them needs to be disabled. Do that by clearing
+ // out the underlying heap reference.
MutexLocker lock(m_mutex);
- // TODO(sof): consider ways of reducing overhead. (e.g., tracking number of active
- // CrossThreadPersistent<>s pointing into the heaps of each ThreadState and use that
- // count to bail out early.)
+ // TODO(sof): consider ways of reducing overhead. (e.g., tracking number of
+ // active CrossThreadPersistent<>s pointing into the heaps of each ThreadState
+ // and use that count to bail out early.)
PersistentNodeSlots* slots = m_persistentRegion->m_slots;
while (slots) {
for (int i = 0; i < PersistentNodeSlots::slotCount; ++i) {
if (slots->m_slot[i].isUnused())
continue;
- // 'self' is in use, containing the cross-thread persistent wrapper object.
+ // 'self' is in use, containing the cross-thread persistent wrapper
+ // object.
CrossThreadPersistent<DummyGCBase>* persistent =
reinterpret_cast<CrossThreadPersistent<DummyGCBase>*>(
slots->m_slot[i].self());
« no previous file with comments | « third_party/WebKit/Source/platform/heap/PersistentNode.h ('k') | third_party/WebKit/Source/platform/heap/SafePoint.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698