Add new IPC for searchbox to pipe SearchBox NavigateContentWindow calls to the browser.

chrome/browser/ui/browser_instant_controller.cc

Index: chrome/browser/ui/browser_instant_controller.cc
diff --git a/chrome/browser/ui/browser_instant_controller.cc b/chrome/browser/ui/browser_instant_controller.cc
index 15ec0efa678a1b45dad23dd1d29a556d43671567..86d234ea97b112136db8f5c2d3eafae7ac13ba3c 100644
--- a/chrome/browser/ui/browser_instant_controller.cc
+++ b/chrome/browser/ui/browser_instant_controller.cc
@@ -161,6 +161,15 @@ void BrowserInstantController::UpdateThemeInfoForPreview() {
     OnThemeChanged(NULL);
 }
 
+void BrowserInstantController::OpenURLInCurrentTab(const GURL& url) {
+  browser_->OpenURL(content::OpenURLParams(
+      url,
+      content::Referrer(),
+      CURRENT_TAB,
+      content::PAGE_TRANSITION_TYPED,

[sky, 2012/12/12 00:09:21]

Why is this typed, and should the referrer be set?

[Shishir, 2012/12/12 00:48:28]

We talked about this and said we dont want to set referers here since the
navigation happened due to a click on the area that is equivalent of the omnibox
popup .

As for the typed page transition type - because it was the default for
AutcompleteMatch. Do you think any other one fits better here?

[sky, 2012/12/12 04:11:19]

Omnibox also uses PAGE_TRANSITION_GENERATED. Which one it uses depends upon the
entry. Maybe the page needs to pass up some context when invoking this.

[samarth, 2012/12/12 16:32:27]

The searchbox code has access to InstantAutocompleteResult, so it could
determine the appropriate transition type and pass it through.

[Shishir, 2012/12/12 20:04:47]

Done.

[Shishir, 2012/12/12 20:04:47]

Done.

+      false));
+}
+
 void BrowserInstantController::ResetInstant() {
   instant_.SetInstantEnabled(IsInstantEnabled(browser_->profile()));
 }