IndexedDB Observers

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 "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/ExceptionStatePlaceholder.h"
+#include "bindings/core/v8/ScriptState.h"
+#include "bindings/core/v8/SerializedScriptValueFactory.h"
+#include "bindings/modules/v8/ToV8ForModules.h"
+#include "bindings/modules/v8/V8BindingForModules.h"
+#include "core/dom/DOMStringList.h"
+#include "core/dom/ExceptionCode.h"
+#include "core/dom/ExecutionContext.h"
+#include "modules/indexeddb/IDBObserverCallback.h"
+#include "modules/indexeddb/IDBObserverChanges.h"
+#include "modules/indexeddb/IDBObserverInit.h"
+#include "platform/SharedBuffer.h"
+#include <v8.h>
+
+namespace blink {
+
+IDBObserver* IDBObserver::create(IDBObserverCallback& callback, const IDBObserverInit& options)
+{
+    return new IDBObserver(callback, options.transaction(), options.values(), options.noRecords());
+}
+
+IDBObserver::IDBObserver(IDBObserverCallback& callback, bool transaction, bool values, bool noRecords)
+    : m_callback(&callback)
+    , m_transaction(transaction)
+    , m_values(values)
+    , m_noRecords(noRecords)
+{
+    id = 1;
+}
+
+void IDBObserver::clearWeakMembers(Visitor* visitor)
+{
+}
+
+void IDBObserver::observe(IDBDatabase* db, IDBTransaction* tx, ExceptionState& exceptionState)

[Marijn Kruisselbrink, 2016/06/02 23:51:24]

Blink coding style doesn't like using abbreviations (point 2 in
https://www.chromium.org/blink/coding-style#TOC-Names). So call these
"database", "transaction" etc, rather than db and tx.

[palakj1, 2016/06/03 01:21:53]

Done. Thanks for the tip.

+{
+}
+
+DEFINE_TRACE(IDBObserver)
+{
+    visitor->template registerWeakMembers<IDBObserver, &IDBObserver::clearWeakMembers>(this);

[dmurph, 2016/06/02 23:48:37]

I did not even know that this syntax was a thing haha.

[palakj1, 2016/06/03 01:21:52]

I don't think we'd need this. Removing it.

+    visitor->trace(m_callback);
+}
+
+} // namespace blink