Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2010 Google Inc. All rights reserved. | 2 * Copyright (C) 2010 Google Inc. All rights reserved. |
| 3 * | 3 * |
| 4 * Redistribution and use in source and binary forms, with or without | 4 * Redistribution and use in source and binary forms, with or without |
| 5 * modification, are permitted provided that the following conditions | 5 * modification, are permitted provided that the following conditions |
| 6 * are met: | 6 * are met: |
| 7 * | 7 * |
| 8 * 1. Redistributions of source code must retain the above copyright | 8 * 1. Redistributions of source code must retain the above copyright |
| 9 * notice, this list of conditions and the following disclaimer. | 9 * notice, this list of conditions and the following disclaimer. |
| 10 * 2. Redistributions in binary form must reproduce the above copyright | 10 * 2. Redistributions in binary form must reproduce the above copyright |
| (...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 307 primaryKey->isLessThan(m_primaryKey.get())); | 307 primaryKey->isLessThan(m_primaryKey.get())); |
| 308 if (!ok) { | 308 if (!ok) { |
| 309 exceptionState.throwDOMException(DataError, | 309 exceptionState.throwDOMException(DataError, |
| 310 "The parameter is greater than or " | 310 "The parameter is greater than or " |
| 311 "equal to this cursor's position."); | 311 "equal to this cursor's position."); |
| 312 return; | 312 return; |
| 313 } | 313 } |
| 314 } | 314 } |
| 315 } | 315 } |
| 316 | 316 |
| 317 // FIXME: We're not using the context from when continue was called, which mea ns the callback | 317 // FIXME: We're not using the context from when continue was called, which |
| 318 // will be on the original context openCursor was called on. Is this ri ght? | 318 // means the callback will be on the original context openCursor was |
|
dcheng
2016/10/04 23:03:04
Nit: FIXMEs aren't usually indented like this
Nico
2016/10/05 01:33:17
Done.
| |
| 319 // called on. Is this right? | |
| 319 m_request->setPendingCursor(this); | 320 m_request->setPendingCursor(this); |
| 320 m_gotValue = false; | 321 m_gotValue = false; |
| 321 m_backend->continueFunction(key, primaryKey, | 322 m_backend->continueFunction(key, primaryKey, |
| 322 WebIDBCallbacksImpl::create(m_request).release()); | 323 WebIDBCallbacksImpl::create(m_request).release()); |
| 323 } | 324 } |
| 324 | 325 |
| 325 IDBRequest* IDBCursor::deleteFunction(ScriptState* scriptState, | 326 IDBRequest* IDBCursor::deleteFunction(ScriptState* scriptState, |
| 326 ExceptionState& exceptionState) { | 327 ExceptionState& exceptionState) { |
| 327 IDB_TRACE("IDBCursor::delete"); | 328 IDB_TRACE("IDBCursor::delete"); |
| 328 if (m_transaction->isFinished() || m_transaction->isFinishing()) { | 329 if (m_transaction->isFinished() || m_transaction->isFinishing()) { |
| (...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 480 case WebIDBCursorDirectionPrevNoDuplicate: | 481 case WebIDBCursorDirectionPrevNoDuplicate: |
| 481 return IndexedDBNames::prevunique; | 482 return IndexedDBNames::prevunique; |
| 482 | 483 |
| 483 default: | 484 default: |
| 484 NOTREACHED(); | 485 NOTREACHED(); |
| 485 return IndexedDBNames::next; | 486 return IndexedDBNames::next; |
| 486 } | 487 } |
| 487 } | 488 } |
| 488 | 489 |
| 489 } // namespace blink | 490 } // namespace blink |
| OLD | NEW |