Allows fullscreen to be triggered with the key events used for scrolling.

content/renderer/render_view_impl.h

Index: content/renderer/render_view_impl.h
diff --git a/content/renderer/render_view_impl.h b/content/renderer/render_view_impl.h
index d2fdd8f259ae2708940c684401013cf1c5f620d1..66669e5cb4eaecc6a4e60636e8e2dd3b6ef1d7c6 100644
--- a/content/renderer/render_view_impl.h
+++ b/content/renderer/render_view_impl.h
@@ -444,6 +444,7 @@ class CONTENT_EXPORT RenderViewImpl
   virtual void didCancelCompositionOnSelectionChange();
   virtual void didChangeSelection(bool is_selection_empty);
   virtual void didExecuteCommand(const WebKit::WebString& command_name);
+  virtual void didScrollWithKeyboard(const WebKit::WebSize& delta);

[jamesr, 2013/06/13 05:25:12]

again, you've declared a function here as a member of RenderViewImpl but only
provide an implementation in an android-specific location. not good! in
particular, if anyone attempts to reference this function on a non-android
platform (a perfectly reasonable thing to do looking at the header) they'll get
a link error.

either guard the declaration or provide an empty definition for non-android
builds

[Jinsuk Kim, 2013/06/13 06:00:25]

Guarded the declaration.

   virtual bool handleCurrentKeyboardEvent();
   virtual WebKit::WebColorChooser* createColorChooser(
       WebKit::WebColorChooserClient*, const WebKit::WebColor& initial_color);
@@ -1358,6 +1359,12 @@ class CONTENT_EXPORT RenderViewImpl
   // much about leaks.
   IDMap<ContextMenuClient, IDMapExternalPointer> pending_context_menus_;
 
+#if defined(OS_ANDROID)
+  // Cache the old top controls state constraints. Used when updating
+  // current value only without altering the constraints.
+  cc::TopControlsState top_controls_constraints_;
+#endif
+
   // View ----------------------------------------------------------------------
 
   // Cache the preferred size of the page in order to prevent sending the IPC