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

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

Issue 18033004: WebCrypto: Add WebKit API structure for keys. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 6 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/Algorithm.cpp
diff --git a/Source/modules/crypto/Algorithm.cpp b/Source/modules/crypto/Algorithm.cpp
index c6a9b8e3b5296b68683bbf241b1745938bba2630..d61cf7d5fc9b787e68ae3beed65c4aed533d1199 100644
--- a/Source/modules/crypto/Algorithm.cpp
+++ b/Source/modules/crypto/Algorithm.cpp
@@ -39,11 +39,9 @@ Algorithm::Algorithm(const WebKit::WebCryptoAlgorithm& algorithm)
ScriptWrappable::init(this);
}
-const String& Algorithm::name()
+String Algorithm::name()
{
- if (m_name.isNull())
- m_name = m_algorithm.algorithmName();
- return m_name;
+ return ASCIILiteral(m_algorithm.algorithmName());
eroman 2013/07/03 02:22:57 Unrelated cleanup. I can extract these to a separa
abarth-chromium 2013/07/03 03:47:37 I don't think this works. ASCIILiteral needs to b
abarth-chromium 2013/07/03 04:07:00 Oh, actually, you're right. ASCIILiteral doesn't
}
} // namespace WebCore

Powered by Google App Engine
This is Rietveld 408576698