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

Unified Diff: Source/modules/crypto/CryptoResultImpl.cpp

Issue 26004002: Decouple ScriptPromise creation from ScriptPromiseResolver. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebase Created 7 years, 2 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
Index: Source/modules/crypto/CryptoResultImpl.cpp
diff --git a/Source/modules/crypto/CryptoResultImpl.cpp b/Source/modules/crypto/CryptoResultImpl.cpp
index 7b00de5eca2f5595d3d9ea462cfc3c9521a37a2d..e291b50ab3d4442e7ee8c35030dedc5370d0f0ee 100644
--- a/Source/modules/crypto/CryptoResultImpl.cpp
+++ b/Source/modules/crypto/CryptoResultImpl.cpp
@@ -50,9 +50,9 @@ CryptoResultImpl::~CryptoResultImpl()
ASSERT(m_finished);
}
-PassRefPtr<CryptoResultImpl> CryptoResultImpl::create()
+PassRefPtr<CryptoResultImpl> CryptoResultImpl::create(ScriptPromise promise)
{
- return adoptRef(new CryptoResultImpl);
+ return adoptRef(new CryptoResultImpl(promise));
}
void CryptoResultImpl::completeWithError()
@@ -63,35 +63,30 @@ void CryptoResultImpl::completeWithError()
void CryptoResultImpl::completeWithBuffer(const WebKit::WebArrayBuffer& buffer)
{
- m_promiseResolver->fulfill(PassRefPtr<ArrayBuffer>(buffer));
+ m_promiseResolver->resolve(PassRefPtr<ArrayBuffer>(buffer));
finish();
}
void CryptoResultImpl::completeWithBoolean(bool b)
{
- m_promiseResolver->fulfill(ScriptValue::createBoolean(b));
+ m_promiseResolver->resolve(ScriptValue::createBoolean(b));
finish();
}
void CryptoResultImpl::completeWithKey(const WebKit::WebCryptoKey& key)
{
- m_promiseResolver->fulfill(Key::create(key));
+ m_promiseResolver->resolve(Key::create(key));
finish();
}
void CryptoResultImpl::completeWithKeyPair(const WebKit::WebCryptoKey& publicKey, const WebKit::WebCryptoKey& privateKey)
{
- m_promiseResolver->fulfill(KeyPair::create(publicKey, privateKey));
+ m_promiseResolver->resolve(KeyPair::create(publicKey, privateKey));
finish();
}
-ScriptPromise CryptoResultImpl::promise()
-{
- return m_promiseResolver->promise();
-}
-
-CryptoResultImpl::CryptoResultImpl()
- : m_promiseResolver(ScriptPromiseResolver::create())
+CryptoResultImpl::CryptoResultImpl(ScriptPromise promise)
+ : m_promiseResolver(ScriptPromiseResolver::create(promise))
, m_finished(false) { }
void CryptoResultImpl::finish()

Powered by Google App Engine
This is Rietveld 408576698