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

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, 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
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 65d6a79d225ae1e53d0d1842c3d19465ac014e6c..1ba064760d0357dc7f566459276e584bac0fee27 100644
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
@@ -106,8 +106,8 @@ namespace blink {
using namespace HTMLNames;
-using WeakMediaElementSet = WillBeHeapHashSet<RawPtrWillBeWeakMember<HTMLMediaElement>>;
-using DocumentElementSetMap = WillBeHeapHashMap<RawPtrWillBeWeakMember<Document>, WeakMediaElementSet>;
+using WeakMediaElementSet = HeapHashSet<WeakMember<HTMLMediaElement>>;
+using DocumentElementSetMap = HeapHashMap<WeakMember<Document>, WeakMediaElementSet>;
namespace {
@@ -140,7 +140,7 @@ const char* boolString(bool val)
DocumentElementSetMap& documentToElementSetMap()
{
- DEFINE_STATIC_LOCAL(OwnPtrWillBePersistent<DocumentElementSetMap>, map, (adoptPtrWillBeNoop(new DocumentElementSetMap())));
+ DEFINE_STATIC_LOCAL(Persistent<DocumentElementSetMap>, map, (new DocumentElementSetMap()));
return *map;
}
@@ -275,9 +275,9 @@ class HTMLMediaElement::AutoplayHelperClientImpl :
public AutoplayExperimentHelper::Client {
public:
- static PassOwnPtrWillBeRawPtr<AutoplayHelperClientImpl> create(HTMLMediaElement* element)
+ static RawPtr<AutoplayHelperClientImpl> create(HTMLMediaElement* element)
{
- return adoptPtrWillBeNoop(new AutoplayHelperClientImpl(element));
+ return new AutoplayHelperClientImpl(element);
}
virtual ~AutoplayHelperClientImpl();
@@ -318,7 +318,7 @@ public:
private:
AutoplayHelperClientImpl(HTMLMediaElement* element) : m_element(element) {}
- RawPtrWillBeMember<HTMLMediaElement> m_element;
+ Member<HTMLMediaElement> m_element;
};
@@ -682,7 +682,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());
@@ -2616,8 +2616,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, DoNothing);
@@ -3296,7 +3296,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.
@@ -3438,7 +3438,7 @@ void HTMLMediaElement::ensureMediaControls()
if (mediaControls())
return;
- RefPtrWillBeRawPtr<MediaControls> mediaControls = MediaControls::create(*this);
+ RawPtr<MediaControls> mediaControls = MediaControls::create(*this);
mediaControls->reset();
if (isFullscreen())
@@ -3477,7 +3477,7 @@ void HTMLMediaElement::configureMediaControls()
CueTimeline& HTMLMediaElement::cueTimeline()
{
if (!m_cueTimeline)
- m_cueTimeline = adoptPtrWillBeNoop(new CueTimeline(*this));
+ m_cueTimeline = new CueTimeline(*this);
return *m_cueTimeline;
}
« no previous file with comments | « third_party/WebKit/Source/core/html/HTMLMediaElement.h ('k') | third_party/WebKit/Source/core/html/HTMLMediaSource.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698