Changeset 140850 in webkit
- Timestamp:
- Jan 25, 2013 11:05:44 AM (11 years ago)
- Location:
- trunk/Source
- Files:
-
- 14 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r140848 r140850 1 2013-01-25 Alec Flett <alecflett@chromium.org> 2 3 IndexedDB: Move TaskType enum to IDBDatabaseBackendInterface 4 https://bugs.webkit.org/show_bug.cgi?id=107960 5 6 Reviewed by Tony Chang. 7 8 The IDBTransactionBackendInterface class is going away, 9 this is an easy search-and-replace patch to keep these 10 changes out of more complex patches. 11 12 No new tests, just part of a refactor. 13 14 * Modules/indexeddb/IDBCursorBackendImpl.cpp: 15 (WebCore::IDBCursorBackendImpl::IDBCursorBackendImpl): 16 * Modules/indexeddb/IDBCursorBackendImpl.h: 17 (WebCore::IDBCursorBackendImpl::create): 18 (IDBCursorBackendImpl): 19 * Modules/indexeddb/IDBDatabaseBackendImpl.cpp: 20 (WebCore::IDBDatabaseBackendImpl::setIndexesReady): 21 (WebCore::OpenCursorOperation::perform): 22 * Modules/indexeddb/IDBObjectStore.cpp: 23 (WebCore::IDBObjectStore::createIndex): 24 (WebCore::IDBObjectStore::openCursor): 25 * Modules/indexeddb/IDBObjectStore.h: 26 (WebCore::IDBObjectStore::openCursor): 27 (IDBObjectStore): 28 * Modules/indexeddb/IDBOpenDBRequest.cpp: 29 (WebCore::IDBOpenDBRequest::IDBOpenDBRequest): 30 * Modules/indexeddb/IDBRequest.cpp: 31 (WebCore::IDBRequest::create): 32 (WebCore::IDBRequest::IDBRequest): 33 * Modules/indexeddb/IDBRequest.h: 34 (IDBRequest): 35 (WebCore::IDBRequest::taskType): 36 * Modules/indexeddb/IDBTransactionBackendImpl.cpp: 37 (WebCore::IDBTransactionBackendImpl::scheduleTask): 38 * Modules/indexeddb/IDBTransactionBackendImpl.h: 39 (WebCore::IDBTransactionBackendImpl::scheduleTask): 40 (IDBTransactionBackendImpl): 41 * Modules/indexeddb/IDBTransactionBackendInterface.h: 42 1 43 2013-01-25 Abhishek Arya <inferno@chromium.org> 2 44 -
trunk/Source/WebCore/Modules/indexeddb/IDBCursorBackendImpl.cpp
r140457 r140850 102 102 }; 103 103 104 IDBCursorBackendImpl::IDBCursorBackendImpl(PassRefPtr<IDBBackingStore::Cursor> cursor, CursorType cursorType, IDB TransactionBackendInterface::TaskType taskType, IDBTransactionBackendImpl* transaction, int64_t objectStoreId)104 IDBCursorBackendImpl::IDBCursorBackendImpl(PassRefPtr<IDBBackingStore::Cursor> cursor, CursorType cursorType, IDBDatabaseBackendInterface::TaskType taskType, IDBTransactionBackendImpl* transaction, int64_t objectStoreId) 105 105 : m_taskType(taskType) 106 106 , m_cursorType(cursorType) -
trunk/Source/WebCore/Modules/indexeddb/IDBCursorBackendImpl.h
r139289 r140850 48 48 static PassRefPtr<IDBCursorBackendImpl> create(PassRefPtr<IDBBackingStore::Cursor> cursor, CursorType cursorType, IDBTransactionBackendImpl* transaction, int64_t objectStoreId) 49 49 { 50 return adoptRef(new IDBCursorBackendImpl(cursor, cursorType, IDB TransactionBackendInterface::NormalTask, transaction, objectStoreId));50 return adoptRef(new IDBCursorBackendImpl(cursor, cursorType, IDBDatabaseBackendInterface::NormalTask, transaction, objectStoreId)); 51 51 } 52 static PassRefPtr<IDBCursorBackendImpl> create(PassRefPtr<IDBBackingStore::Cursor> cursor, CursorType cursorType, IDB TransactionBackendInterface::TaskType taskType, IDBTransactionBackendImpl* transaction, int64_t objectStoreId)52 static PassRefPtr<IDBCursorBackendImpl> create(PassRefPtr<IDBBackingStore::Cursor> cursor, CursorType cursorType, IDBDatabaseBackendInterface::TaskType taskType, IDBTransactionBackendImpl* transaction, int64_t objectStoreId) 53 53 { 54 54 return adoptRef(new IDBCursorBackendImpl(cursor, cursorType, taskType, transaction, objectStoreId)); … … 70 70 71 71 private: 72 IDBCursorBackendImpl(PassRefPtr<IDBBackingStore::Cursor>, CursorType, IDB TransactionBackendInterface::TaskType, IDBTransactionBackendImpl*, int64_t objectStoreId);72 IDBCursorBackendImpl(PassRefPtr<IDBBackingStore::Cursor>, CursorType, IDBDatabaseBackendInterface::TaskType, IDBTransactionBackendImpl*, int64_t objectStoreId); 73 73 74 74 class CursorIterationOperation; … … 76 76 class CursorPrefetchIterationOperation; 77 77 78 IDB TransactionBackendInterface::TaskType m_taskType;78 IDBDatabaseBackendInterface::TaskType m_taskType; 79 79 CursorType m_cursorType; 80 80 const RefPtr<IDBDatabaseBackendImpl> m_database; -
trunk/Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.cpp
r140602 r140850 915 915 return; 916 916 917 if (!transaction->scheduleTask(IDB TransactionBackendInterface::PreemptiveTask, SetIndexesReadyOperation::create(indexIds.size())))917 if (!transaction->scheduleTask(IDBDatabaseBackendInterface::PreemptiveTask, SetIndexesReadyOperation::create(indexIds.size()))) 918 918 ASSERT_NOT_REACHED(); 919 919 } … … 964 964 } 965 965 966 IDB TransactionBackendInterface::TaskType taskType(static_cast<IDBTransactionBackendInterface::TaskType>(m_taskType));966 IDBDatabaseBackendInterface::TaskType taskType(static_cast<IDBDatabaseBackendInterface::TaskType>(m_taskType)); 967 967 RefPtr<IDBCursorBackendImpl> cursor = IDBCursorBackendImpl::create(backingStoreCursor.get(), m_cursorType, taskType, transaction, m_objectStoreId); 968 968 m_callbacks->onSuccess(cursor, cursor->key(), cursor->primaryKey(), cursor->value()); -
trunk/Source/WebCore/Modules/indexeddb/IDBObjectStore.cpp
r140457 r140850 416 416 return 0; 417 417 418 RefPtr<IDBRequest> indexRequest = openCursor(context, static_cast<IDBKeyRange*>(0), IDBCursor::directionNext(), IDB TransactionBackendInterface::PreemptiveTask, ec);418 RefPtr<IDBRequest> indexRequest = openCursor(context, static_cast<IDBKeyRange*>(0), IDBCursor::directionNext(), IDBDatabaseBackendInterface::PreemptiveTask, ec); 419 419 ASSERT(!ec); 420 420 if (ec) … … 492 492 } 493 493 494 PassRefPtr<IDBRequest> IDBObjectStore::openCursor(ScriptExecutionContext* context, PassRefPtr<IDBKeyRange> range, const String& directionString, IDB TransactionBackendInterface::TaskType taskType, ExceptionCode& ec)494 PassRefPtr<IDBRequest> IDBObjectStore::openCursor(ScriptExecutionContext* context, PassRefPtr<IDBKeyRange> range, const String& directionString, IDBDatabaseBackendInterface::TaskType taskType, ExceptionCode& ec) 495 495 { 496 496 IDB_TRACE("IDBObjectStore::openCursor"); -
trunk/Source/WebCore/Modules/indexeddb/IDBObjectStore.h
r140457 r140850 71 71 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext* context, PassRefPtr<IDBKeyRange> keyRange, ExceptionCode& ec) { return openCursor(context, keyRange, IDBCursor::directionNext(), ec); } 72 72 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext* context, const ScriptValue& key, ExceptionCode& ec) { return openCursor(context, key, IDBCursor::directionNext(), ec); } 73 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext* context, PassRefPtr<IDBKeyRange> range, const String& direction, ExceptionCode& ec) { return openCursor(context, range, direction, IDB TransactionBackendInterface::NormalTask, ec); }74 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext*, PassRefPtr<IDBKeyRange>, const String& direction, IDB TransactionBackendInterface::TaskType, ExceptionCode&);73 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext* context, PassRefPtr<IDBKeyRange> range, const String& direction, ExceptionCode& ec) { return openCursor(context, range, direction, IDBDatabaseBackendInterface::NormalTask, ec); } 74 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext*, PassRefPtr<IDBKeyRange>, const String& direction, IDBDatabaseBackendInterface::TaskType, ExceptionCode&); 75 75 PassRefPtr<IDBRequest> openCursor(ScriptExecutionContext*, const ScriptValue& key, const String& direction, ExceptionCode&); 76 76 -
trunk/Source/WebCore/Modules/indexeddb/IDBOpenDBRequest.cpp
r140741 r140850 46 46 47 47 IDBOpenDBRequest::IDBOpenDBRequest(ScriptExecutionContext* context, PassRefPtr<IDBDatabaseCallbacksImpl> callbacks, int64_t transactionId, int64_t version) 48 : IDBRequest(context, IDBAny::createNull(), IDB TransactionBackendInterface::NormalTask, 0)48 : IDBRequest(context, IDBAny::createNull(), IDBDatabaseBackendInterface::NormalTask, 0) 49 49 , m_databaseCallbacks(callbacks) 50 50 , m_transactionId(transactionId) -
trunk/Source/WebCore/Modules/indexeddb/IDBRequest.cpp
r140457 r140850 48 48 PassRefPtr<IDBRequest> IDBRequest::create(ScriptExecutionContext* context, PassRefPtr<IDBAny> source, IDBTransaction* transaction) 49 49 { 50 RefPtr<IDBRequest> request(adoptRef(new IDBRequest(context, source, IDB TransactionBackendInterface::NormalTask, transaction)));50 RefPtr<IDBRequest> request(adoptRef(new IDBRequest(context, source, IDBDatabaseBackendInterface::NormalTask, transaction))); 51 51 request->suspendIfNeeded(); 52 52 // Requests associated with IDBFactory (open/deleteDatabase/getDatabaseNames) are not associated with transactions. … … 56 56 } 57 57 58 PassRefPtr<IDBRequest> IDBRequest::create(ScriptExecutionContext* context, PassRefPtr<IDBAny> source, IDB TransactionBackendInterface::TaskType taskType, IDBTransaction* transaction)58 PassRefPtr<IDBRequest> IDBRequest::create(ScriptExecutionContext* context, PassRefPtr<IDBAny> source, IDBDatabaseBackendInterface::TaskType taskType, IDBTransaction* transaction) 59 59 { 60 60 RefPtr<IDBRequest> request(adoptRef(new IDBRequest(context, source, taskType, transaction))); … … 66 66 } 67 67 68 IDBRequest::IDBRequest(ScriptExecutionContext* context, PassRefPtr<IDBAny> source, IDB TransactionBackendInterface::TaskType taskType, IDBTransaction* transaction)68 IDBRequest::IDBRequest(ScriptExecutionContext* context, PassRefPtr<IDBAny> source, IDBDatabaseBackendInterface::TaskType taskType, IDBTransaction* transaction) 69 69 : ActiveDOMObject(context, this) 70 70 , m_result(0) -
trunk/Source/WebCore/Modules/indexeddb/IDBRequest.h
r138674 r140850 58 58 public: 59 59 static PassRefPtr<IDBRequest> create(ScriptExecutionContext*, PassRefPtr<IDBAny> source, IDBTransaction*); 60 static PassRefPtr<IDBRequest> create(ScriptExecutionContext*, PassRefPtr<IDBAny> source, IDB TransactionBackendInterface::TaskType, IDBTransaction*);60 static PassRefPtr<IDBRequest> create(ScriptExecutionContext*, PassRefPtr<IDBAny> source, IDBDatabaseBackendInterface::TaskType, IDBTransaction*); 61 61 virtual ~IDBRequest(); 62 62 … … 116 116 using RefCounted<IDBCallbacks>::deref; 117 117 118 IDB TransactionBackendInterface::TaskType taskType() { return m_taskType; }118 IDBDatabaseBackendInterface::TaskType taskType() { return m_taskType; } 119 119 120 120 DOMRequestState* requestState() { return &m_requestState; } 121 121 122 122 protected: 123 IDBRequest(ScriptExecutionContext*, PassRefPtr<IDBAny> source, IDB TransactionBackendInterface::TaskType, IDBTransaction*);123 IDBRequest(ScriptExecutionContext*, PassRefPtr<IDBAny> source, IDBDatabaseBackendInterface::TaskType, IDBTransaction*); 124 124 void enqueueEvent(PassRefPtr<Event>); 125 125 virtual bool shouldEnqueueEvent() const; … … 146 146 147 147 RefPtr<IDBAny> m_source; 148 const IDB TransactionBackendInterface::TaskType m_taskType;148 const IDBDatabaseBackendInterface::TaskType m_taskType; 149 149 150 150 bool m_hasPendingActivity; -
trunk/Source/WebCore/Modules/indexeddb/IDBTransactionBackendImpl.cpp
r140219 r140850 71 71 } 72 72 73 bool IDBTransactionBackendImpl::scheduleTask( TaskType type, PassOwnPtr<Operation> task, PassOwnPtr<Operation> abortTask)73 bool IDBTransactionBackendImpl::scheduleTask(IDBDatabaseBackendInterface::TaskType type, PassOwnPtr<Operation> task, PassOwnPtr<Operation> abortTask) 74 74 { 75 75 if (m_state == Finished) 76 76 return false; 77 77 78 if (type == NormalTask)78 if (type == IDBDatabaseBackendInterface::NormalTask) 79 79 m_taskQueue.append(task); 80 80 else -
trunk/Source/WebCore/Modules/indexeddb/IDBTransactionBackendImpl.h
r140465 r140850 68 68 const HashSet<int64_t>& scope() const { return m_objectStoreIds; } 69 69 bool isFinished() const { return m_state == Finished; } 70 bool scheduleTask(PassOwnPtr<Operation> task, PassOwnPtr<Operation> abortTask = nullptr) { return scheduleTask( NormalTask, task, abortTask); }71 bool scheduleTask( TaskType, PassOwnPtr<Operation>, PassOwnPtr<Operation> abortTask = nullptr);70 bool scheduleTask(PassOwnPtr<Operation> task, PassOwnPtr<Operation> abortTask = nullptr) { return scheduleTask(IDBDatabaseBackendInterface::NormalTask, task, abortTask); } 71 bool scheduleTask(IDBDatabaseBackendInterface::TaskType, PassOwnPtr<Operation>, PassOwnPtr<Operation> abortTask = nullptr); 72 72 void registerOpenCursor(IDBCursorBackendImpl*); 73 73 void unregisterOpenCursor(IDBCursorBackendImpl*); -
trunk/Source/WebCore/Modules/indexeddb/IDBTransactionBackendInterface.h
r139666 r140850 47 47 public: 48 48 virtual ~IDBTransactionBackendInterface() { } 49 50 enum TaskType {51 NormalTask = 0,52 PreemptiveTask53 };54 55 49 virtual void commit() = 0; 56 50 virtual void abort() = 0; -
trunk/Source/WebKit/chromium/ChangeLog
r140808 r140850 1 2013-01-25 Alec Flett <alecflett@chromium.org> 2 3 IndexedDB: Move TaskType enum to IDBDatabaseBackendInterface 4 https://bugs.webkit.org/show_bug.cgi?id=107960 5 6 Reviewed by Tony Chang. 7 8 Update asserts to reflect new enum location. 9 10 * src/AssertMatchingEnums.cpp: 11 1 12 2013-01-25 Sheriff Bot <webkit.review.bot@gmail.com> 2 13 -
trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp
r140310 r140850 502 502 COMPILE_ASSERT_MATCHING_ENUM(WebIDBCursor::PrevNoDuplicate, IDBCursor::PREV_NO_DUPLICATE); 503 503 504 COMPILE_ASSERT_MATCHING_ENUM(WebIDBTransaction::PreemptiveTask, IDB TransactionBackendInterface::PreemptiveTask);505 COMPILE_ASSERT_MATCHING_ENUM(WebIDBTransaction::NormalTask, IDB TransactionBackendInterface::NormalTask);504 COMPILE_ASSERT_MATCHING_ENUM(WebIDBTransaction::PreemptiveTask, IDBDatabaseBackendInterface::PreemptiveTask); 505 COMPILE_ASSERT_MATCHING_ENUM(WebIDBTransaction::NormalTask, IDBDatabaseBackendInterface::NormalTask); 506 506 #endif 507 507
Note: See TracChangeset
for help on using the changeset viewer.