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

Unified Diff: third_party/WebKit/Source/modules/indexeddb/IDBTransaction.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/IDBTransaction.cpp
diff --git a/third_party/WebKit/Source/modules/indexeddb/IDBTransaction.cpp b/third_party/WebKit/Source/modules/indexeddb/IDBTransaction.cpp
index 4d4bd220b494930c7b692c2dcb3f35e2ab255b7a..709f09d2af09898b63c804cfc3a359350e74b646 100644
--- a/third_party/WebKit/Source/modules/indexeddb/IDBTransaction.cpp
+++ b/third_party/WebKit/Source/modules/indexeddb/IDBTransaction.cpp
@@ -307,7 +307,8 @@ void IDBTransaction::onAbort(DOMException* error) {
if (isVersionChange())
m_database->close();
- // Enqueue events before notifying database, as database may close which enqueues more events and order matters.
+ // Enqueue events before notifying database, as database may close which
+ // enqueues more events and order matters.
enqueueEvent(Event::createBubble(EventTypeNames::abort));
finished();
}
@@ -322,15 +323,17 @@ void IDBTransaction::onComplete() {
DCHECK_NE(m_state, Finished);
m_state = Finishing;
- // Enqueue events before notifying database, as database may close which enqueues more events and order matters.
+ // Enqueue events before notifying database, as database may close which
+ // enqueues more events and order matters.
enqueueEvent(Event::create(EventTypeNames::complete));
finished();
}
bool IDBTransaction::hasPendingActivity() const {
- // FIXME: In an ideal world, we should return true as long as anyone has a or can
- // get a handle to us or any child request object and any of those have
- // event listeners. This is in order to handle user generated events properly.
+ // FIXME: In an ideal world, we should return true as long as anyone has a or
+ // can get a handle to us or any child request object and any of those have
+ // event listeners. This is in order to handle user generated events
+ // properly.
return m_hasPendingActivity && !m_contextStopped;
}
@@ -401,7 +404,8 @@ DispatchEventResult IDBTransaction::dispatchEventInternal(Event* event) {
targets.append(this);
targets.append(db());
- // FIXME: When we allow custom event dispatching, this will probably need to change.
+ // FIXME: When we allow custom event dispatching, this will probably need to
+ // change.
DCHECK(event->type() == EventTypeNames::complete ||
event->type() == EventTypeNames::abort);
DispatchEventResult dispatchResult =

Powered by Google App Engine
This is Rietveld 408576698