Add initial VR interactive omnibox.

chrome/browser/android/vr_shell/vr_shell.cc

Index: chrome/browser/android/vr_shell/vr_shell.cc
diff --git a/chrome/browser/android/vr_shell/vr_shell.cc b/chrome/browser/android/vr_shell/vr_shell.cc
index a4c520d03e728c48ca7363ddab886cb614b9387c..4cb0fbbb55d4390b8bbfbfab9f41668c85714352 100644
--- a/chrome/browser/android/vr_shell/vr_shell.cc
+++ b/chrome/browser/android/vr_shell/vr_shell.cc
@@ -397,15 +397,26 @@ void VrShell::DoUiAction(const UiAction action,
     case RELOAD:
       controller.Reload(content::ReloadType::NORMAL, false);
       break;
+    case LOAD_URL: {
+      std::string url_string;
+      CHECK(arguments->GetString("url", &url_string));
+      GURL url(url_string);
+      // TODO(cjgrant): Sanitize the URL and prefix, and pass the proper
+      // transition type down from the UI.

[bshe, 2017/01/23 16:24:37]

nit: perhaps create a crbug to track this

[cjgrant, 2017/01/24 15:31:39]

Done.

+      controller.LoadURL(url, content::Referrer(),
+                         ui::PageTransition::PAGE_TRANSITION_AUTO_TOPLEVEL,
+                         std::string(""));
+      break;
+    }
+    case OMNIBOX_CONTENT:

[asimjour1, 2017/01/23 18:25:59]

nit: OMNIBOX_CONTENT doesn't sound like an action. perhaps renaming it.

[cjgrant, 2017/01/24 15:31:39]

I agree, but I'm not sure of the best way to address this.  We have a mix of two
approaches:  A set of WebUI callbacks (domLoaded, updateScene, doAction,
setUiCssSize), and a set of actions.  All of these make calls to VrShell.  The
callbacks make specific VrShell function calls, whereas the actions pass a
command type and arguments to VrShell to parse.

In a perfect world, nothing but the message handler would parse arguments, so as
to keep the dictionary structure private to the JS-native interface.  But if we
do this, we just need to create classes to take that data and pump it through to
VrShell, UI, etc.  Lets chat about this one in person.

+      html_interface_->HandleOmniboxInput(*arguments);
+      break;
     case KEY_EVENT: {
       int char_value;
       int modifiers;
-
       CHECK(arguments->GetInteger("modifiers", &modifiers));
       CHECK(arguments->GetInteger("charValue", &char_value));
-      
       ui_input_manager_->ProcessKeyboardEvent(char_value, modifiers);
-
       break;
     }
 #if defined(ENABLE_VR_SHELL_UI_DEV)