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

Unified Diff: third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp

Issue 2388423003: reflow comments in modules/[fetch,indexeddb] (Closed)
Patch Set: rebase 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/modules/indexeddb/IDBTransactionTest.cpp
diff --git a/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp b/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
index fe9684c3f2eba5805c006c5f7afe3c69c1f7bb89..d04f3200f3ba343ae1ebec720ff3f85accb3002d 100644
--- a/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
+++ b/third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp
@@ -91,8 +91,8 @@ TEST(IDBTransactionTest, EnsureLifetime) {
ThreadState::current()->collectAllGarbage();
EXPECT_EQ(1u, set.size());
- // This will generate an abort() call to the back end which is dropped by the fake proxy,
- // so an explicit onAbort call is made.
+ // This will generate an abort() call to the back end which is dropped by the
+ // fake proxy, so an explicit onAbort call is made.
scope.getExecutionContext()->notifyContextDestroyed();
transaction->onAbort(DOMException::create(AbortError, "Aborted"));
transaction.clear();
@@ -134,11 +134,12 @@ TEST(IDBTransactionTest, TransactionFinish) {
ThreadState::current()->collectAllGarbage();
EXPECT_EQ(1u, set.size());
- // Stop the context, so events don't get queued (which would keep the transaction alive).
+ // Stop the context, so events don't get queued (which would keep the
+ // transaction alive).
scope.getExecutionContext()->notifyContextDestroyed();
- // Fire an abort to make sure this doesn't free the transaction during use. The test
- // will not fail if it is, but ASAN would notice the error.
+ // Fire an abort to make sure this doesn't free the transaction during use.
+ // The test will not fail if it is, but ASAN would notice the error.
db->onAbort(transactionId, DOMException::create(AbortError, "Aborted"));
// onAbort() should have cleared the transaction's reference to the database.

Powered by Google App Engine
This is Rietveld 408576698