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

Unified Diff: Source/WebCore/html/HTMLMediaElement.cpp

Issue 13861033: Remove Apple's unused implementation of private browsing from WebCore (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: Source/WebCore/html/HTMLMediaElement.cpp
diff --git a/Source/WebCore/html/HTMLMediaElement.cpp b/Source/WebCore/html/HTMLMediaElement.cpp
index cb20c1b4b4eadc10cd132e86ff0b6811bc78c997..94fa6cbf711cad3749061e88cd9183d16d45d535 100644
--- a/Source/WebCore/html/HTMLMediaElement.cpp
+++ b/Source/WebCore/html/HTMLMediaElement.cpp
@@ -284,7 +284,6 @@ HTMLMediaElement::HTMLMediaElement(const QualifiedName& tagName, Document* docum
{
LOG(Media, "HTMLMediaElement::HTMLMediaElement");
document->registerForMediaVolumeCallbacks(this);
- document->registerForPrivateBrowsingStateChangedCallbacks(this);
if (document->settings() && document->settings()->mediaPlaybackRequiresUserGesture()) {
addBehaviorRestriction(RequireUserGestureForRateChangeRestriction);
@@ -306,7 +305,6 @@ HTMLMediaElement::~HTMLMediaElement()
document()->removeMediaCanStartListener(this);
setShouldDelayLoadEvent(false);
document()->unregisterForMediaVolumeCallbacks(this);
- document()->unregisterForPrivateBrowsingStateChangedCallbacks(this);
#if ENABLE(VIDEO_TRACK)
document()->unregisterForCaptionPreferencesChangedCallbacks(this);
if (m_textTracks)
@@ -906,10 +904,6 @@ void HTMLMediaElement::loadResource(const KURL& initialURL, ContentType& content
if (m_sendProgressEvents)
startProgressEventTimer();
- Settings* settings = document()->settings();
- bool privateMode = !settings || settings->privateBrowsingEnabled();
- m_player->setPrivateBrowsingMode(privateMode);
-
// Reset display mode to force a recalculation of what to show because we are resetting the player.
setDisplayMode(Unknown);
@@ -4067,17 +4061,6 @@ void HTMLMediaElement::resetMediaEngines()
MediaPlayer::resetMediaEngines();
}
-void HTMLMediaElement::privateBrowsingStateDidChange()
-{
- if (!m_player)
- return;
-
- Settings* settings = document()->settings();
- bool privateMode = !settings || settings->privateBrowsingEnabled();
- LOG(Media, "HTMLMediaElement::privateBrowsingStateDidChange(%s)", boolString(privateMode));
- m_player->setPrivateBrowsingMode(privateMode);
-}
-
MediaControls* HTMLMediaElement::mediaControls() const
{
return toMediaControls(userAgentShadowRoot()->firstChild());

Powered by Google App Engine
This is Rietveld 408576698