Chromium Code Reviews| Index: Source/core/css/MediaQueryEvaluator.cpp |
| diff --git a/Source/core/css/MediaQueryEvaluator.cpp b/Source/core/css/MediaQueryEvaluator.cpp |
| index 5517dca39b0eaaefb05df229c598570cadb36815..776427e910929e49be4e95f6215d9a0068513a27 100644 |
| --- a/Source/core/css/MediaQueryEvaluator.cpp |
| +++ b/Source/core/css/MediaQueryEvaluator.cpp |
| @@ -70,7 +70,7 @@ MediaQueryEvaluator::MediaQueryEvaluator(bool mediaFeatureResult) |
| { |
| } |
| -MediaQueryEvaluator::MediaQueryEvaluator(const AtomicString& acceptedMediaType, bool mediaFeatureResult) |
| +MediaQueryEvaluator::MediaQueryEvaluator(const String& acceptedMediaType, bool mediaFeatureResult) |
| : m_mediaType(acceptedMediaType) |
| , m_frame(0) |
| , m_style(nullptr) |
| @@ -86,7 +86,7 @@ MediaQueryEvaluator::MediaQueryEvaluator(const char* acceptedMediaType, bool med |
| { |
| } |
| -MediaQueryEvaluator::MediaQueryEvaluator(const AtomicString& acceptedMediaType, LocalFrame* frame, RenderStyle* style) |
| +MediaQueryEvaluator::MediaQueryEvaluator(const String& acceptedMediaType, LocalFrame* frame, RenderStyle* style) |
| : m_mediaType(acceptedMediaType) |
| , m_frame(frame) |
| , m_style(style) |
| @@ -98,7 +98,7 @@ MediaQueryEvaluator::~MediaQueryEvaluator() |
| { |
| } |
| -bool MediaQueryEvaluator::mediaTypeMatch(const AtomicString& mediaTypeToMatch) const |
| +bool MediaQueryEvaluator::mediaTypeMatch(const String& mediaTypeToMatch) const |
| { |
| return mediaTypeToMatch.isEmpty() |
| || equalIgnoringCase(mediaTypeToMatch, "all") |
| @@ -110,7 +110,7 @@ bool MediaQueryEvaluator::mediaTypeMatchSpecific(const char* mediaTypeToMatch) c |
| // Like mediaTypeMatch, but without the special cases for "" and "all". |
| ASSERT(mediaTypeToMatch); |
| ASSERT(mediaTypeToMatch[0] != '\0'); |
| - ASSERT(!equalIgnoringCase(mediaTypeToMatch, AtomicString("all"))); |
| + ASSERT(!equalIgnoringCase(mediaTypeToMatch, String("all"))); |
|
abarth-chromium
2014/03/06 08:03:57
Didn't you create a static string for all? We sho
|
| return equalIgnoringCase(mediaTypeToMatch, m_mediaType); |
| } |