IndexedDB: Clean up boilerplate when throwing TransactionInactiveError

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/indexeddb/IDBObserver.h"
 
 #include <bitset>
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/modules/v8/IDBObserverCallback.h"
@@ skipping 11 matching lines @@
 IDBObserver* IDBObserver::Create(IDBObserverCallback* callback) {
   return new IDBObserver(callback);
 }
 
 IDBObserver::IDBObserver(IDBObserverCallback* callback) : callback_(callback) {}
 
 void IDBObserver::observe(IDBDatabase* database,
                           IDBTransaction* transaction,
                           const IDBObserverInit& options,
                           ExceptionState& exception_state) {
-  if (transaction->IsFinished() || transaction->IsFinishing()) {
-    exception_state.ThrowDOMException(
-        kTransactionInactiveError,
-        IDBDatabase::kTransactionFinishedErrorMessage);
-    return;
-  }
   if (!transaction->IsActive()) {
-    exception_state.ThrowDOMException(
-        kTransactionInactiveError,
-        IDBDatabase::kTransactionInactiveErrorMessage);
+    exception_state.ThrowDOMException(kTransactionInactiveError,
+                                      transaction->InactiveErrorMessage());
     return;
   }
   if (transaction->IsVersionChange()) {
     exception_state.ThrowDOMException(
         kTransactionInactiveError,
         IDBDatabase::kCannotObserveVersionChangeTransaction);
     return;
   }
   if (!database->Backend()) {
     exception_state.ThrowDOMException(kInvalidStateError,
@@ skipping 51 matching lines @@
   if (!observer_ids_to_remove.IsEmpty())
     database->RemoveObservers(observer_ids_to_remove);
 }
 
 DEFINE_TRACE(IDBObserver) {
   visitor->Trace(callback_);
   visitor->Trace(observer_ids_);
 }
 
 }  // namespace blink