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

Unified Diff: third_party/WebKit/Source/core/css/MediaList.cpp

Issue 1686483002: Oilpan: Remove most WillBe types from the code base (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
« no previous file with comments | « third_party/WebKit/Source/core/css/MediaList.h ('k') | third_party/WebKit/Source/core/css/MediaList.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/css/MediaList.cpp
diff --git a/third_party/WebKit/Source/core/css/MediaList.cpp b/third_party/WebKit/Source/core/css/MediaList.cpp
index 1c825853dc91ef1aee9382e594d605ae7d36be92..9dbd736283ce7cf851b40367c9da6fb53c1253f7 100644
--- a/third_party/WebKit/Source/core/css/MediaList.cpp
+++ b/third_party/WebKit/Source/core/css/MediaList.cpp
@@ -59,9 +59,7 @@ MediaQuerySet::MediaQuerySet(const MediaQuerySet& o)
m_queries[i] = o.m_queries[i]->copy();
}
-DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(MediaQuerySet);
-
-PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQuerySet::create(const String& mediaString)
+RawPtr<MediaQuerySet> MediaQuerySet::create(const String& mediaString)
{
if (mediaString.isEmpty())
return MediaQuerySet::create();
@@ -69,7 +67,7 @@ PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQuerySet::create(const String& mediaS
return MediaQueryParser::parseMediaQuerySet(mediaString);
}
-PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQuerySet::createOffMainThread(const String& mediaString)
+RawPtr<MediaQuerySet> MediaQuerySet::createOffMainThread(const String& mediaString)
{
if (mediaString.isEmpty())
return MediaQuerySet::create();
@@ -79,7 +77,7 @@ PassRefPtrWillBeRawPtr<MediaQuerySet> MediaQuerySet::createOffMainThread(const S
bool MediaQuerySet::set(const String& mediaString)
{
- RefPtrWillBeRawPtr<MediaQuerySet> result = create(mediaString);
+ RawPtr<MediaQuerySet> result = create(mediaString);
m_queries.swap(result->m_queries);
return true;
}
@@ -89,13 +87,13 @@ bool MediaQuerySet::add(const String& queryString)
// To "parse a media query" for a given string means to follow "the parse
// a media query list" steps and return "null" if more than one media query
// is returned, or else the returned media query.
- RefPtrWillBeRawPtr<MediaQuerySet> result = create(queryString);
+ RawPtr<MediaQuerySet> result = create(queryString);
// Only continue if exactly one media query is found, as described above.
if (result->m_queries.size() != 1)
return true;
- OwnPtrWillBeRawPtr<MediaQuery> newQuery = result->m_queries[0].release();
+ RawPtr<MediaQuery> newQuery = result->m_queries[0].release();
ASSERT(newQuery);
// If comparing with any of the media queries in the collection of media
@@ -115,13 +113,13 @@ bool MediaQuerySet::remove(const String& queryStringToRemove)
// To "parse a media query" for a given string means to follow "the parse
// a media query list" steps and return "null" if more than one media query
// is returned, or else the returned media query.
- RefPtrWillBeRawPtr<MediaQuerySet> result = create(queryStringToRemove);
+ RawPtr<MediaQuerySet> result = create(queryStringToRemove);
// Only continue if exactly one media query is found, as described above.
if (result->m_queries.size() != 1)
return true;
- OwnPtrWillBeRawPtr<MediaQuery> newQuery = result->m_queries[0].release();
+ RawPtr<MediaQuery> newQuery = result->m_queries[0].release();
ASSERT(newQuery);
// Remove any media query from the collection of media queries for which
@@ -139,7 +137,7 @@ bool MediaQuerySet::remove(const String& queryStringToRemove)
return found;
}
-void MediaQuerySet::addMediaQuery(PassOwnPtrWillBeRawPtr<MediaQuery> mediaQuery)
+void MediaQuerySet::addMediaQuery(RawPtr<MediaQuery> mediaQuery)
{
m_queries.append(mediaQuery);
}
@@ -182,8 +180,6 @@ MediaList::MediaList(MediaQuerySet* mediaQueries, CSSRule* parentRule)
{
}
-DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(MediaList);
-
void MediaList::setMediaText(const String& value)
{
CSSStyleSheet::RuleMutationScope mutationScope(m_parentRule);
@@ -196,7 +192,7 @@ void MediaList::setMediaText(const String& value)
String MediaList::item(unsigned index) const
{
- const WillBeHeapVector<OwnPtrWillBeMember<MediaQuery>>& queries = m_mediaQueries->queryVector();
+ const HeapVector<Member<MediaQuery>>& queries = m_mediaQueries->queryVector();
if (index < queries.size())
return queries[index]->cssText();
return String();
« no previous file with comments | « third_party/WebKit/Source/core/css/MediaList.h ('k') | third_party/WebKit/Source/core/css/MediaList.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698