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

Unified Diff: Source/core/dom/DOMImplementation.cpp

Issue 14722011: MediaPlayer cleanup, part one of N (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
« no previous file with comments | « Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp ('k') | Source/core/html/HTMLMediaElement.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/DOMImplementation.cpp
diff --git a/Source/core/dom/DOMImplementation.cpp b/Source/core/dom/DOMImplementation.cpp
index 5481f209c0be0c273012366243e1bb69e2e1d69b..5df2b76e6ba45e4954eab7858242f41f014e7c6e 100644
--- a/Source/core/dom/DOMImplementation.cpp
+++ b/Source/core/dom/DOMImplementation.cpp
@@ -67,22 +67,6 @@ static void addString(FeatureSet& set, const char* string)
set.add(string);
}
-class DOMImplementationSupportsTypeClient : public MediaPlayerSupportsTypeClient {
-public:
- DOMImplementationSupportsTypeClient(bool needsHacks, const String& host)
- : m_needsHacks(needsHacks)
- , m_host(host)
- {
- }
-
-private:
- virtual bool mediaPlayerNeedsSiteSpecificHacks() const OVERRIDE { return m_needsHacks; }
- virtual String mediaPlayerDocumentHost() const OVERRIDE { return m_host; }
-
- bool m_needsHacks;
- String m_host;
-};
-
#if ENABLE(SVG)
static bool isSVG10Feature(const String &feature, const String &version)
@@ -433,11 +417,10 @@ PassRefPtr<Document> DOMImplementation::createDocument(const String& type, Frame
if (Image::supportsType(type))
return ImageDocument::create(frame, url);
- // Check to see if the type can be played by our MediaPlayer, if so create a MediaDocument
+ // Check to see if the type can be played by our MediaPlayer, if so create a MediaDocument
// Key system is not applicable here.
- DOMImplementationSupportsTypeClient client(frame && frame->settings() && frame->settings()->needsSiteSpecificQuirks(), url.host());
- if (MediaPlayer::supportsType(ContentType(type), String(), url, &client))
- return MediaDocument::create(frame, url);
+ if (MediaPlayer::supportsType(ContentType(type), String(), url))
+ return MediaDocument::create(frame, url);
// Everything else except text/plain can be overridden by plugins. In particular, Adobe SVG Viewer should be used for SVG, if installed.
// Disallowing plug-ins to use text/plain prevents plug-ins from hijacking a fundamental type that the browser is expected to handle,
« no previous file with comments | « Source/WebKit/chromium/src/WebMediaPlayerClientImpl.cpp ('k') | Source/core/html/HTMLMediaElement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698