Chromium Code Reviews (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

Unified Diff: content/browser/renderer_host/render_view_host_delegate.h

Issue 7461059: Fullscreen JS API implementation. (Closed) Base URL: svn://
Patch Set: responded to sky's comment Created 9 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
Index: content/browser/renderer_host/render_view_host_delegate.h
diff --git a/content/browser/renderer_host/render_view_host_delegate.h b/content/browser/renderer_host/render_view_host_delegate.h
index 8239ba31a0c7408679fdf428910a95392662a756..94caa107c5b5fbcc72f7877fec69ddb1702b883e 100644
--- a/content/browser/renderer_host/render_view_host_delegate.h
+++ b/content/browser/renderer_host/render_view_host_delegate.h
@@ -358,6 +358,12 @@ class RenderViewHostDelegate : public IPC::Channel::Listener {
virtual void HandleMouseUp() {}
virtual void HandleMouseActivate() {}
+ // Puts the browser into fullscreen mode, focussing on this tab.
sky 2011/07/28 03:53:29 This description makes it sound as though you inte
koz (OOO until 15th September) 2011/07/28 05:14:30 I think so. What exactly does 'select the tab' mea
sky 2011/07/28 16:07:02 The tabstrip has a bunch of tabs, but only one act
koz (OOO until 15th September) 2011/07/29 04:12:57 Is it necessary to call ActivateContents() if this
sky 2011/07/29 15:51:46 With the IPC calls it might be possible for a user
koz (OOO until 15th September) 2011/08/01 03:43:38 Ah, I see. I'll rephrase the comment and remove th
+ virtual void EnterFullscreenMode() {}
+ // Makes the browser exit fullscreen mode.
+ virtual void ExitFullscreenMode() {}
virtual ~RenderViewHostDelegate() {}

Powered by Google App Engine
This is Rietveld 408576698