IndexedDB: Throw exception if no valid context in IDBRequest.result.

IndexedDB: Throw exception if no valid context in IDBRequest.result.

This is a speculative fix since unable to repro with local Android ASan build.

BUG=588352
Committed: https://crrev.com/bdb6c202d739663d6e934de598e3347ee26d215e
Cr-Commit-Position: refs/heads/master@{#396907}

[cmumford, 2016-05-31 17:49:39]

FYI: I didn't want to use a new error message for a condition that shouldn't
exist.

[jsbell, 2016-05-31 17:56:00]

jsbell@chromium.org changed reviewers:
+ jsbell@chromium.org

[jsbell, 2016-05-31 17:56:00]

lgtm

https://codereview.chromium.org/2022873003/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp (right):

https://codereview.chromium.org/2022873003/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp:97:
exceptionState.throwDOMException(InvalidStateError,
IDBDatabase::databaseClosedErrorMessage);
Maybe add comment: "Must throw if returning an empty value. Message is arbitrary
since it will never be seen." ?

[cmumford, 2016-05-31 18:03:22]

The CQ bit was checked by cmumford@chromium.org

[cmumford, 2016-05-31 18:03:23]

The patchset sent to the CQ was uploaded after l-g-t-m from jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/2022873003/#ps20001
(title: "Added exception comment.")

[commit-bot: I haz the power, 2016-05-31 18:04:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2022873003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2022873003/20001

[commit-bot: I haz the power, 2016-05-31 20:06:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-31 20:44:58]

Description was changed from

==========
IndexedDB: Throw exception if no valid context in IDBRequest.result.

This is a speculative fix since unable to repro with local Android ASan build.

BUG=588352
==========

to

==========
IndexedDB: Throw exception if no valid context in IDBRequest.result.

This is a speculative fix since unable to repro with local Android ASan build.

BUG=588352

Committed: https://crrev.com/bdb6c202d739663d6e934de598e3347ee26d215e
Cr-Commit-Position: refs/heads/master@{#396907}
==========

[commit-bot: I haz the power, 2016-05-31 20:44:59]

Patchset 2 (id:??) landed as
https://crrev.com/bdb6c202d739663d6e934de598e3347ee26d215e
Cr-Commit-Position: refs/heads/master@{#396907}