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

Unified Diff: Source/core/html/HTMLMediaElement.cpp

Issue 108313015: Make calls to AtomicString(const String&) explicit in html/ (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 12 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/core/html/HTMLMediaElement.cpp
diff --git a/Source/core/html/HTMLMediaElement.cpp b/Source/core/html/HTMLMediaElement.cpp
index d0654b41cbff8d044e34b5f2b2786eec99212547..a04e46243c9a05c1bc0456af6eccd1b8566f04aa 100644
--- a/Source/core/html/HTMLMediaElement.cpp
+++ b/Source/core/html/HTMLMediaElement.cpp
@@ -2081,9 +2081,9 @@ String HTMLMediaElement::preload() const
return String();
}
-void HTMLMediaElement::setPreload(const String& preload)
+void HTMLMediaElement::setPreload(const AtomicString& preload)
{
- WTF_LOG(Media, "HTMLMediaElement::setPreload(%s)", preload.utf8().data());
+ WTF_LOG(Media, "HTMLMediaElement::setPreload(%s)", preload.string().utf8().data());
eseidel 2013/12/31 23:22:35 I'm a little surprsied that AtomicSTring doens't h
Inactive 2014/01/01 19:26:29 Right, this would be convenient. I will propose th
setAttribute(preloadAttr, preload);
}
@@ -2534,7 +2534,7 @@ void HTMLMediaElement::removeAllInbandTracks()
}
}
-PassRefPtr<TextTrack> HTMLMediaElement::addTextTrack(const String& kind, const String& label, const String& language, ExceptionState& exceptionState)
+PassRefPtr<TextTrack> HTMLMediaElement::addTextTrack(const AtomicString& kind, const AtomicString& label, const AtomicString& language, ExceptionState& exceptionState)
{
ASSERT(RuntimeEnabledFeatures::videoTrackEnabled());

Powered by Google App Engine
This is Rietveld 408576698