reflow comments in modules/[fetch,indexeddb]

third_party/WebKit/Source/modules/indexeddb/IDBTransaction.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 289 matching lines @@
 
     abortOutstandingRequests();
     revertDatabaseMetadata();
 
     m_state = Finishing;
   }
 
   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();
 }
 
 void IDBTransaction::onComplete() {
   IDB_TRACE("IDBTransaction::onComplete");
   if (m_contextStopped) {
     finished();
     return;
   }
 
   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

[dcheng, 2016/10/04 23:03:04]

Nit: same comment about FIXME; alignment

[Nico, 2016/10/05 01:33:17]

Done.

+  //        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;
 }
 
 WebIDBTransactionMode IDBTransaction::stringToMode(const String& modeString) {
   if (modeString == IndexedDBNames::readonly)
     return WebIDBTransactionModeReadOnly;
   if (modeString == IndexedDBNames::readwrite)
     return WebIDBTransactionModeReadWrite;
   if (modeString == IndexedDBNames::versionchange)
     return WebIDBTransactionModeVersionChange;
@@ skipping 50 matching lines @@
   DCHECK_NE(m_state, Finished);
   DCHECK(m_hasPendingActivity);
   DCHECK(getExecutionContext());
   DCHECK_EQ(event->target(), this);
   m_state = Finished;
 
   HeapVector<Member<EventTarget>> targets;
   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 =
       IDBEventDispatcher::dispatch(event, targets);
   // FIXME: Try to construct a test where |this| outlives openDBRequest and we
   // get a crash.
   if (m_openDBRequest) {
     DCHECK(isVersionChange());
     m_openDBRequest->transactionDidFinishAndDispatch();
   }
@@ skipping 63 matching lines @@
   m_objectStoreMap.clear();
   for (auto& it : m_deletedObjectStores)
     it->transactionFinished();
   m_createdObjectStores.clear();
   m_deletedObjectStores.clear();
 
   m_objectStoreCleanupMap.clear();
 }
 
 }  // namespace blink