Adds paste function to searchbox api and handles paste event on fakebox

chrome/browser/ui/search/instant_controller.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/search/instant_controller.h"
 
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 318 matching lines @@
   switch (state) {
     case OMNIBOX_FOCUS_VISIBLE:
       // TODO(samarth): re-enable this once setValue() correctly handles
       // URL-shaped queries.
       // browser_->FocusOmnibox(true);
       break;
     case OMNIBOX_FOCUS_INVISIBLE:
       browser_->FocusOmnibox(false);
       break;
     case OMNIBOX_FOCUS_NONE:
       contents->GetView()->Focus();

[samarth, 2013/07/26 22:51:41]

As we discussed before, this is troublesome because this lets the page close the
popup right after pasting. Instead of handling this call here, let's call
BrowserInstantController::FocusOmnibox() in this branch as well. (Well, at that
point, you can get rid of the switch and just call it directly.)

Then in BrowserInstantController, you should only handle this case if the popup
is not open. You can check using OmniboxEditModel::popup_model::IsOpen I think.

[jfweitz, 2013/07/29 18:48:25]

Done.

       break;
   }
 }
 
 void InstantController::NavigateToURL(const content::WebContents* contents,
                                       const GURL& url,
                                       content::PageTransition transition,
                                       WindowOpenDisposition disposition,
                                       bool is_search_type) {
   LOG_INSTANT_DEBUG_EVENT(this, base::StringPrintf(
       "NavigateToURL: url='%s'", url.spec().c_str()));
 
   // TODO(samarth): handle case where contents are no longer "active" (e.g. user
   // has switched tabs).
   if (!extended_enabled())
     return;
 
   if (transition == content::PAGE_TRANSITION_AUTO_BOOKMARK) {
     content::RecordAction(
         content::UserMetricsAction("InstantExtended.MostVisitedClicked"));
   } else {
     // Exclude navigation by Most Visited click and searches.
     if (!is_search_type)
       RecordNavigationHistogram(UsingLocalPage(), true, true);
   }
   browser_->OpenURL(url, transition, disposition);
 }
 
+void InstantController::PasteIntoOmnibox(const content::WebContents* contents) {
+  if (!extended_enabled() || search_mode_.is_origin_default())
+    return;
+
+  DCHECK(IsContentsFrom(instant_tab(), contents));
+
+  browser_->PasteIntoOmnibox();
+}
+
 bool InstantController::extended_enabled() const {
   return extended_enabled_;
 }
 
 void InstantController::ResetInstantTab() {
   if (!search_mode_.is_origin_default()) {
     content::WebContents* active_tab = browser_->GetActiveWebContents();
     if (!instant_tab_ || active_tab != instant_tab_->contents()) {
       instant_tab_.reset(new InstantTab(this, browser_->profile()));
       instant_tab_->Init(active_tab);
@@ skipping 39 matching lines @@
       content::PAGE_TRANSITION_SERVER_REDIRECT,
       std::string());  // No extra headers.
   // TODO(dcblack): Remove extraneous history entry caused by 404s.
   // Note that the base case of a 204 being returned doesn't push a history
   // entry.
 }
 
 InstantService* InstantController::GetInstantService() const {
   return InstantServiceFactory::GetForProfile(profile());
 }