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

Unified Diff: Source/core/html/shadow/MediaControlsChromium.cpp

Issue 23886003: Have HTMLElements / SVGElements constructors take a Document reference in argument (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Another Android build fix Created 7 years, 3 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: Source/core/html/shadow/MediaControlsChromium.cpp
diff --git a/Source/core/html/shadow/MediaControlsChromium.cpp b/Source/core/html/shadow/MediaControlsChromium.cpp
index 380b8a693d61abda50436abc700220b108353edb..d4fadf70193de4f3dd936dad5ebd26ab378a5c73 100644
--- a/Source/core/html/shadow/MediaControlsChromium.cpp
+++ b/Source/core/html/shadow/MediaControlsChromium.cpp
@@ -34,7 +34,7 @@ using namespace std;
namespace WebCore {
-MediaControlsChromium::MediaControlsChromium(Document* document)
+MediaControlsChromium::MediaControlsChromium(Document& document)
: MediaControls(document)
, m_durationDisplay(0)
, m_enclosure(0)
@@ -43,15 +43,15 @@ MediaControlsChromium::MediaControlsChromium(Document* document)
// MediaControls::create() for Android is defined in MediaControlsChromiumAndroid.cpp.
#if !OS(ANDROID)
-PassRefPtr<MediaControls> MediaControls::create(Document* document)
+PassRefPtr<MediaControls> MediaControls::create(Document& document)
{
return MediaControlsChromium::createControls(document);
}
#endif
-PassRefPtr<MediaControlsChromium> MediaControlsChromium::createControls(Document* document)
+PassRefPtr<MediaControlsChromium> MediaControlsChromium::createControls(Document& document)
{
- if (!document->page())
+ if (!document.page())
return 0;
RefPtr<MediaControlsChromium> controls = adoptRef(new MediaControlsChromium(document));
@@ -62,7 +62,7 @@ PassRefPtr<MediaControlsChromium> MediaControlsChromium::createControls(Document
return 0;
}
-bool MediaControlsChromium::initializeControls(Document* document)
+bool MediaControlsChromium::initializeControls(Document& document)
{
// Create an enclosing element for the panel so we can visually offset the controls correctly.
RefPtr<MediaControlPanelEnclosureElement> enclosure = MediaControlPanelEnclosureElement::create(document);
@@ -205,7 +205,7 @@ void MediaControlsChromium::createTextTrackDisplay()
if (m_textDisplayContainer)
return;
- RefPtr<MediaControlTextTrackContainerElement> textDisplayContainer = MediaControlTextTrackContainerElement::create(&document());
+ RefPtr<MediaControlTextTrackContainerElement> textDisplayContainer = MediaControlTextTrackContainerElement::create(document());
m_textDisplayContainer = textDisplayContainer.get();
if (m_mediaController)
« no previous file with comments | « Source/core/html/shadow/MediaControlsChromium.h ('k') | Source/core/html/shadow/MediaControlsChromiumAndroid.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698