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

Unified Diff: third_party/WebKit/Source/core/html/HTMLMediaElement.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, 10 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: third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
diff --git a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
index ac9168d2f444499ae3a72956d1e57c6db109d421..f38e4b6877a1d16701a239a3fd45880c8125a2e1 100644
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
@@ -122,11 +122,11 @@ static const char mediaSourceBlobProtocol[] = "blob";
using namespace HTMLNames;
-typedef WillBeHeapHashSet<RawPtrWillBeWeakMember<HTMLMediaElement>> WeakMediaElementSet;
-typedef WillBeHeapHashMap<RawPtrWillBeWeakMember<Document>, WeakMediaElementSet> DocumentElementSetMap;
+typedef HeapHashSet<WeakMember<HTMLMediaElement>> WeakMediaElementSet;
+typedef HeapHashMap<WeakMember<Document>, WeakMediaElementSet> DocumentElementSetMap;
static DocumentElementSetMap& documentToElementSetMap()
{
- DEFINE_STATIC_LOCAL(OwnPtrWillBePersistent<DocumentElementSetMap>, map, (adoptPtrWillBeNoop(new DocumentElementSetMap())));
+ DEFINE_STATIC_LOCAL(Persistent<DocumentElementSetMap>, map, ((new DocumentElementSetMap())));
return *map;
}
@@ -595,7 +595,7 @@ void HTMLMediaElement::scheduleEvent(const AtomicString& eventName)
scheduleEvent(Event::createCancelable(eventName));
}
-void HTMLMediaElement::scheduleEvent(PassRefPtrWillBeRawPtr<Event> event)
+void HTMLMediaElement::scheduleEvent(RawPtr<Event> event)
{
#if LOG_MEDIA_EVENTS
WTF_LOG(Media, "HTMLMediaElement::scheduleEvent(%p) - scheduling '%s'", this, event->type().ascii().data());
@@ -2516,8 +2516,8 @@ bool HTMLMediaElement::havePotentialSourceChild()
{
// Stash the current <source> node and next nodes so we can restore them after checking
// to see there is another potential.
- RefPtrWillBeRawPtr<HTMLSourceElement> currentSourceNode = m_currentSourceNode;
- RefPtrWillBeRawPtr<Node> nextNode = m_nextChildNodeToConsider;
+ RawPtr<HTMLSourceElement> currentSourceNode = m_currentSourceNode;
+ RawPtr<Node> nextNode = m_nextChildNodeToConsider;
KURL nextURL = selectNextSourceChild(0, 0, DoNothing);
@@ -3173,7 +3173,7 @@ TextTrackContainer& HTMLMediaElement::ensureTextTrackContainer()
if (firstChild && firstChild->isTextTrackContainer())
return toTextTrackContainer(*firstChild);
- RefPtrWillBeRawPtr<TextTrackContainer> textTrackContainer = TextTrackContainer::create(document());
+ RawPtr<TextTrackContainer> textTrackContainer = TextTrackContainer::create(document());
// The text track container should be inserted before the media controls,
// so that they are rendered behind them.
@@ -3315,7 +3315,7 @@ void HTMLMediaElement::ensureMediaControls()
if (mediaControls())
return;
- RefPtrWillBeRawPtr<MediaControls> mediaControls = MediaControls::create(*this);
+ RawPtr<MediaControls> mediaControls = MediaControls::create(*this);
mediaControls->reset();
if (isFullscreen())
@@ -3353,7 +3353,7 @@ void HTMLMediaElement::configureMediaControls()
CueTimeline& HTMLMediaElement::cueTimeline()
{
if (!m_cueTimeline)
- m_cueTimeline = adoptPtrWillBeNoop(new CueTimeline(*this));
+ m_cueTimeline = (new CueTimeline(*this));
return *m_cueTimeline;
}

Powered by Google App Engine
This is Rietveld 408576698