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

Unified Diff: Source/modules/indexeddb/IDBRequestTest.cpp

Issue 338833002: Factor out NullExecutionContext from unittest files. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Addressed #2 Created 6 years, 6 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
« no previous file with comments | « Source/core/testing/NullExecutionContext.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/indexeddb/IDBRequestTest.cpp
diff --git a/Source/modules/indexeddb/IDBRequestTest.cpp b/Source/modules/indexeddb/IDBRequestTest.cpp
index 00d9f764e07c979c2d1f92a2d328a8e0d899cc80..cfcc1851d834606ee4a5300ae903e33b73801bff 100644
--- a/Source/modules/indexeddb/IDBRequestTest.cpp
+++ b/Source/modules/indexeddb/IDBRequestTest.cpp
@@ -26,17 +26,23 @@
#include "config.h"
#include "modules/indexeddb/IDBRequest.h"
+#include "bindings/v8/ScriptState.h"
+#include "bindings/v8/V8Binding.h"
#include "core/dom/DOMError.h"
-#include "core/dom/Document.h"
-#include "core/dom/SecurityContext.h"
-#include "core/events/EventQueue.h"
+#include "core/dom/ExecutionContext.h"
+#include "core/testing/NullExecutionContext.h"
#include "modules/indexeddb/IDBDatabaseCallbacks.h"
+#include "modules/indexeddb/IDBKey.h"
#include "modules/indexeddb/IDBKeyRange.h"
#include "modules/indexeddb/IDBOpenDBRequest.h"
#include "platform/SharedBuffer.h"
#include "public/platform/WebBlobInfo.h"
#include "public/platform/WebIDBDatabase.h"
+#include "wtf/OwnPtr.h"
#include "wtf/PassOwnPtr.h"
+#include "wtf/PassRefPtr.h"
+#include "wtf/Vector.h"
+#include "wtf/dtoa/utils.h"
#include <gtest/gtest.h>
#include <v8.h>
@@ -45,54 +51,6 @@ using namespace WebCore;
namespace {
-class NullEventQueue FINAL : public EventQueue {
-public:
- NullEventQueue() { }
- virtual ~NullEventQueue() { }
- virtual bool enqueueEvent(PassRefPtrWillBeRawPtr<Event>) OVERRIDE { return true; }
- virtual bool cancelEvent(Event*) OVERRIDE { return true; }
- virtual void close() OVERRIDE { }
-};
-
-class NullExecutionContext FINAL : public RefCountedWillBeGarbageCollectedFinalized<NullExecutionContext>, public SecurityContext, public ExecutionContext {
- WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(NullExecutionContext);
-public:
- NullExecutionContext();
-
- virtual EventQueue* eventQueue() const OVERRIDE { return m_queue.get(); }
-
- void trace(Visitor* visitor)
- {
- visitor->trace(m_queue);
- ExecutionContext::trace(visitor);
- }
-
- virtual void reportBlockedScriptExecutionToInspector(const String& directiveText) OVERRIDE { }
- virtual SecurityContext& securityContext() { return *this; }
-
-#if !ENABLE(OILPAN)
- using RefCounted<NullExecutionContext>::ref;
- using RefCounted<NullExecutionContext>::deref;
-
- virtual void refExecutionContext() OVERRIDE { ref(); }
- virtual void derefExecutionContext() OVERRIDE { deref(); }
-#endif
-
-protected:
- virtual const KURL& virtualURL() const OVERRIDE { return m_dummyURL; }
- virtual KURL virtualCompleteURL(const String&) const OVERRIDE { return m_dummyURL; }
-
-private:
- OwnPtrWillBeMember<EventQueue> m_queue;
-
- KURL m_dummyURL;
-};
-
-NullExecutionContext::NullExecutionContext()
- : m_queue(adoptPtrWillBeNoop(new NullEventQueue()))
-{
-}
-
class IDBRequestTest : public testing::Test {
public:
IDBRequestTest()
« no previous file with comments | « Source/core/testing/NullExecutionContext.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698