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

Unified Diff: chrome/renderer/render_view.cc

Issue 3026043: Full screen video Chromium patch (Closed)
Patch Set: Rename GetClient/SetClient to client/setClient to match WebKit style. Created 10 years, 5 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 | « chrome/renderer/render_view.h ('k') | chrome/renderer/render_widget_fullscreen.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/render_view.cc
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc
index e3b2c7e1818acf6afa29117ffddcfa93ccdc7769..59b0270f8f83a299063d31e5c3b8978e96fc80a6 100644
--- a/chrome/renderer/render_view.cc
+++ b/chrome/renderer/render_view.cc
@@ -94,9 +94,11 @@
#include "third_party/WebKit/WebKit/chromium/public/WebFormControlElement.h"
#include "third_party/WebKit/WebKit/chromium/public/WebFormElement.h"
#include "third_party/WebKit/WebKit/chromium/public/WebFrame.h"
+#include "third_party/WebKit/WebKit/chromium/public/WebFullscreenMediaPlayerClient.h"
#include "third_party/WebKit/WebKit/chromium/public/WebHistoryItem.h"
#include "third_party/WebKit/WebKit/chromium/public/WebImage.h"
#include "third_party/WebKit/WebKit/chromium/public/WebInputElement.h"
+#include "third_party/WebKit/WebKit/chromium/public/WebMediaElement.h"
#include "third_party/WebKit/WebKit/chromium/public/WebNode.h"
#include "third_party/WebKit/WebKit/chromium/public/WebNodeList.h"
#include "third_party/WebKit/WebKit/chromium/public/WebPageSerializer.h"
@@ -187,9 +189,11 @@ using WebKit::WebFindOptions;
using WebKit::WebFormControlElement;
using WebKit::WebFormElement;
using WebKit::WebFrame;
+using WebKit::WebFullscreenMediaPlayer;
using WebKit::WebHistoryItem;
using WebKit::WebImage;
using WebKit::WebInputElement;
+using WebKit::WebMediaElement;
using WebKit::WebMediaPlayer;
using WebKit::WebMediaPlayerAction;
using WebKit::WebMediaPlayerClient;
@@ -487,6 +491,8 @@ RenderView::~RenderView() {
for (ViewMap::iterator it = views->begin(); it != views->end(); ++it)
DCHECK_NE(this, it->second) << "Failed to call Close?";
#endif
+
+ DCHECK(fullscreen_widgets_.empty());
}
/*static*/
@@ -1968,14 +1974,6 @@ bool RenderView::supportsFullscreen() {
switches::kEnableVideoFullscreen);
}
-void RenderView::enterFullscreenForNode(const WebKit::WebNode& node) {
- NOTIMPLEMENTED();
-}
-
-void RenderView::exitFullscreenForNode(const WebKit::WebNode& node) {
- NOTIMPLEMENTED();
-}
-
void RenderView::setStatusText(const WebString& text) {
}
« no previous file with comments | « chrome/renderer/render_view.h ('k') | chrome/renderer/render_widget_fullscreen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698