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

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

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.h
diff --git a/third_party/WebKit/Source/platform/heap/PersistentNode.h b/third_party/WebKit/Source/platform/heap/PersistentNode.h
index ff37d5d743378bf64e7c0f8820cea7475f87b727..0e518a39e295041b229de7028b427acaa2c6ea22 100644
--- a/third_party/WebKit/Source/platform/heap/PersistentNode.h
+++ b/third_party/WebKit/Source/platform/heap/PersistentNode.h
@@ -180,13 +180,16 @@ class CrossThreadPersistentRegion final {
void freePersistentNode(PersistentNode*& persistentNode) {
MutexLocker lock(m_mutex);
- // When the thread that holds the heap object that the cross-thread persistent shuts down,
- // prepareForThreadStateTermination() will clear out the associated CrossThreadPersistent<>
- // and PersistentNode so as to avoid unsafe access. This can overlap with a holder of
- // the CrossThreadPersistent<> also clearing the persistent and freeing the PersistentNode.
+ // When the thread that holds the heap object that the cross-thread
+ // persistent shuts down, prepareForThreadStateTermination() will clear out
+ // the associated CrossThreadPersistent<> and PersistentNode so as to avoid
+ // unsafe access. This can overlap with a holder of the
+ // CrossThreadPersistent<> also clearing the persistent and freeing the
+ // PersistentNode.
//
- // The lock ensures the updating is ordered, but by the time lock has been acquired the
- // PersistentNode reference may have been cleared out already; check for this.
+ // The lock ensures the updating is ordered, but by the time lock has been
+ // acquired the PersistentNode reference may have been cleared out already;
+ // check for this.
if (!persistentNode)
return;
m_persistentRegion->freePersistentNode(persistentNode);
« no previous file with comments | « third_party/WebKit/Source/platform/heap/Persistent.h ('k') | third_party/WebKit/Source/platform/heap/PersistentNode.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698