Disable browser key reservation in fullscreen mode

chrome/browser/ui/browser_command_controller.cc

Index: chrome/browser/ui/browser_command_controller.cc
diff --git a/chrome/browser/ui/browser_command_controller.cc b/chrome/browser/ui/browser_command_controller.cc
index 0d947e9a4f24cf96a056d565a89e227d60a8f26f..38a7e8a12c1d3875bbc7393f403c464d7c079e4b 100644
--- a/chrome/browser/ui/browser_command_controller.cc
+++ b/chrome/browser/ui/browser_command_controller.cc
@@ -199,8 +199,12 @@ bool BrowserCommandController::IsReservedCommandOrKey(
   }
 #endif
 
-  if (window()->IsFullscreen() && command_id == IDC_FULLSCREEN)
-    return true;
+  if (window()->IsFullscreen()) {
+    // In full-screen mode, all the keys, except for F11, should be delivered to
+    // the web page. This has been discussed in the document Allowing sites to

[dominickn, 2017/01/16 22:13:39]

Since this is a web-facing change, this cannot be landed until an intent to
implement and ship is posted to blink-dev and approved. I think garykac@ is in
charge of that.

When it's been posted and approved, this CL should link to the blink-dev
discussion, not to an internal doc.

A test should also be added for the changes in this method, probably in
browser_command_controller_unittest.cc

[Hzj_jie, 2017/01/17 23:19:10]

I have received the intent of implementation from Gary, since it's not still in
discussion stage, I will update it once it is approved.

+    // intercept keyboard shortcuts at https://goo.gl/r6XmNt.
+    return command_id == IDC_FULLSCREEN;
+  }
 
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
   // If this key was registered by the user as a content editing hotkey, then