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

Side by Side Diff: third_party/WebKit/Source/core/html/HTMLMediaElement.cpp

Issue 1851743002: Simplify Supplementables post Oilpan. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix component build Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved. 2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 3592 matching lines...) Expand 10 before | Expand all | Expand 10 after
3603 { 3603 {
3604 if (event->type() == EventTypeNames::focusin) { 3604 if (event->type() == EventTypeNames::focusin) {
3605 if (mediaControls()) 3605 if (mediaControls())
3606 mediaControls()->mediaElementFocused(); 3606 mediaControls()->mediaElementFocused();
3607 } 3607 }
3608 HTMLElement::defaultEventHandler(event); 3608 HTMLElement::defaultEventHandler(event);
3609 } 3609 }
3610 3610
3611 DEFINE_TRACE(HTMLMediaElement) 3611 DEFINE_TRACE(HTMLMediaElement)
3612 { 3612 {
3613 #if ENABLE(OILPAN)
3614 visitor->trace(m_playedTimeRanges); 3613 visitor->trace(m_playedTimeRanges);
3615 visitor->trace(m_asyncEventQueue); 3614 visitor->trace(m_asyncEventQueue);
3616 visitor->trace(m_error); 3615 visitor->trace(m_error);
3617 visitor->trace(m_currentSourceNode); 3616 visitor->trace(m_currentSourceNode);
3618 visitor->trace(m_nextChildNodeToConsider); 3617 visitor->trace(m_nextChildNodeToConsider);
3619 visitor->trace(m_mediaSource); 3618 visitor->trace(m_mediaSource);
3620 visitor->trace(m_audioTracks); 3619 visitor->trace(m_audioTracks);
3621 visitor->trace(m_videoTracks); 3620 visitor->trace(m_videoTracks);
3622 visitor->trace(m_cueTimeline); 3621 visitor->trace(m_cueTimeline);
3623 visitor->trace(m_textTracks); 3622 visitor->trace(m_textTracks);
3624 visitor->trace(m_textTracksWhenResourceSelectionBegan); 3623 visitor->trace(m_textTracksWhenResourceSelectionBegan);
3625 visitor->trace(m_playResolvers); 3624 visitor->trace(m_playResolvers);
3626 visitor->trace(m_audioSourceProvider); 3625 visitor->trace(m_audioSourceProvider);
3627 visitor->trace(m_autoplayHelperClient); 3626 visitor->trace(m_autoplayHelperClient);
3628 visitor->trace(m_autoplayHelper); 3627 visitor->trace(m_autoplayHelper);
3629 visitor->template registerWeakMembers<HTMLMediaElement, &HTMLMediaElement::c learWeakMembers>(this); 3628 visitor->template registerWeakMembers<HTMLMediaElement, &HTMLMediaElement::c learWeakMembers>(this);
3630 HeapSupplementable<HTMLMediaElement>::trace(visitor); 3629 Supplementable<HTMLMediaElement>::trace(visitor);
3631 #endif
3632 HTMLElement::trace(visitor); 3630 HTMLElement::trace(visitor);
3633 ActiveDOMObject::trace(visitor); 3631 ActiveDOMObject::trace(visitor);
3634 } 3632 }
3635 3633
3636 void HTMLMediaElement::createPlaceholderTracksIfNecessary() 3634 void HTMLMediaElement::createPlaceholderTracksIfNecessary()
3637 { 3635 {
3638 if (!RuntimeEnabledFeatures::audioVideoTracksEnabled()) 3636 if (!RuntimeEnabledFeatures::audioVideoTracksEnabled())
3639 return; 3637 return;
3640 3638
3641 // Create a placeholder audio track if the player says it has audio but it d idn't explicitly announce the tracks. 3639 // Create a placeholder audio track if the player says it has audio but it d idn't explicitly announce the tracks.
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
3872 } 3870 }
3873 3871
3874 #if !ENABLE(OILPAN) 3872 #if !ENABLE(OILPAN)
3875 WeakPtr<HTMLMediaElement> HTMLMediaElement::createWeakPtr() 3873 WeakPtr<HTMLMediaElement> HTMLMediaElement::createWeakPtr()
3876 { 3874 {
3877 return m_weakPtrFactory.createWeakPtr(); 3875 return m_weakPtrFactory.createWeakPtr();
3878 } 3876 }
3879 #endif 3877 #endif
3880 3878
3881 } // namespace blink 3879 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698