Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(912)

Unified Diff: chrome/browser/instant/instant_controller.cc

Issue 11413018: alternate ntp: implement searchbox api for instant overlay to adopt themes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressed sreeram's and chris's comments Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/instant/instant_controller.cc
diff --git a/chrome/browser/instant/instant_controller.cc b/chrome/browser/instant/instant_controller.cc
index 4f8ba0185e950b99b5384f76fc52e08bb3158bb1..457b4ae01dc0b753454a8308a2a1f3d0386416e1 100644
--- a/chrome/browser/instant/instant_controller.cc
+++ b/chrome/browser/instant/instant_controller.cc
@@ -561,6 +561,16 @@ void InstantController::OnActiveTabModeChanged(chrome::search::Mode new_mode) {
}
}
+void InstantController::ThemeChanged(const ThemeBackgroundInfo& theme_info) {
+ if (GetPreviewContents())
+ loader_->SendThemeBackgroundInfo(theme_info);
+}
+
+void InstantController::ThemeAreaHeightChanged(int height) {
+ if (GetPreviewContents())
+ loader_->SendThemeAreaHeight(height);
+}
+
bool InstantController::commit_on_pointer_release() const {
return GetPreviewContents() && loader_->IsPointerDownFromActivate();
}
@@ -692,8 +702,9 @@ void InstantController::ResetLoader(const std::string& instant_url,
loader_.reset(new InstantLoader(this, instant_url, active_tab));
loader_->Init();
- // Ensure the searchbox API has the correct context.
+ // Ensure the searchbox API has the correct context and theme-related info.
loader_->OnActiveTabModeChanged(active_tab_mode_.is_ntp());
+ browser_->UpdateThemeInfoForPreview();
// Reset the loader timer.
stale_loader_timer_.Stop();

Powered by Google App Engine
This is Rietveld 408576698