Check cancellation before reporting crypto result.

Check cancellation before reporting crypto result.

Should Blink have cancelled the crypto operation, do not go ahead and
try to report the result. The objects involved cannot be assumed to be
a valid state.

R=
BUG=440450

[sof, 2015-07-15 12:47:53]

sigbjornf@opera.com changed reviewers:
+ eroman@chromium.org, haraken@chromium.org

[sof, 2015-07-15 12:47:54]

please take a look.

Makes sense on its own, but needed to make
https://codereview.chromium.org/1228373006/ reliable.

[haraken, 2015-07-15 13:56:06]

non-owner LGTM

[eroman, 2015-07-15 20:39:32]

not lgtm.

If additional checks for cancellation are needed, please add them as part of
WebCryptoResult / WebCryptoResultImpl instead, rather than at each callsite to
WebCryptoResult.

[sof, 2015-07-15 21:01:59]

On 2015/07/15 20:39:32, eroman wrote:
> not lgtm.
> 
> If additional checks for cancellation are needed, please add them as part of
> WebCryptoResult / WebCryptoResultImpl instead, rather than at each callsite to
> WebCryptoResult.

ok, the commit*() methods on WebCryptoResult gate on cancellation status instead
( https://codereview.chromium.org/1228373006/ )