Clicks broken in NTP section of Search overlay

chrome/browser/ui/views/search_view_controller.h

 // Copyright (c) 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_SEARCH_VIEW_CONTROLLER_H_
 #define CHROME_BROWSER_UI_VIEWS_SEARCH_VIEW_CONTROLLER_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/search/search_model_observer.h"
@@ skipping 20 matching lines @@
 }
 
 // SearchViewController maintains the search overlay (native new tab page).
 // To avoid ordering dependencies this class listens directly to the
 // SearchModel of the active tab. BrowserView is responsible for telling this
 // class when the active tab changes.
 class SearchViewController
     : public chrome::search::SearchModelObserver,
       public ui::ImplicitAnimationObserver {
  public:
-  SearchViewController(content::BrowserContext* browser_context,
-                       ContentsContainer* contents_container);
+  explicit SearchViewController(ContentsContainer* contents_container);
   virtual ~SearchViewController();
 
   views::View* omnibox_popup_view_parent();
 
   void set_location_bar_container(
       LocationBarContainer* location_bar_container) {
     location_bar_container_ = location_bar_container;
   }
 
   // Sets the active tab.
@@ skipping 42 matching lines @@
   // Create the various views and installs them as an overlay on
   // |contents_container_|.
   void CreateViews();
 
   // Destroys the various views.
   void DestroyViews();
 
   // Invoked when the visibility of the omnibox popup changes.
   void PopupVisibilityChanged();
 
-  // Load the NTP from the associated |SearchTabHelper| if in NTP mode
-  // and the current |tab_contents_| has changed.
-  void MaybeLoadNTP();
-
   // Access active search model.
   chrome::search::SearchModel* search_model();
 
   // Access active web contents.
   content::WebContents* web_contents();
 
-  // The profile.  Weak.
-  content::BrowserContext* browser_context_;
-
   // Where the overlay is placed.  Weak.
   ContentsContainer* contents_container_;
 
   // Weak.
   LocationBarContainer* location_bar_container_;
 
   State state_;
 
   // The active TabContents.  Weak.  May be NULL.
   TabContents* tab_contents_;
@@ skipping 23 matching lines @@
   // of any of these views.
   //
   // NTPView and OmniboxPopupViewParent are siblings. When on the NTP the
   // OmniboxPopupViewParent is obscured by the NTPView. When on a search page
   // the NTPView is hidden.
   //
   //
   views::View* search_container_;
   views::View* ntp_view_;
   views::View* logo_view_;
+
+  // An alias to |contents_container_->active()|, but reparented within
+  // |ntp_view_| when in the NTP state.
   views::WebView* content_view_;
+
   OmniboxPopupViewParent* omnibox_popup_view_parent_;
 
   DISALLOW_COPY_AND_ASSIGN(SearchViewController);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_SEARCH_VIEW_CONTROLLER_H_