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

Unified Diff: chrome/renderer/searchbox/searchbox_extension.cc

Issue 12631008: alternate ntp: implement Show/HideBars API to reduce jank when showing/hiding bars (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 months 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/renderer/searchbox/searchbox_extension.cc
diff --git a/chrome/renderer/searchbox/searchbox_extension.cc b/chrome/renderer/searchbox/searchbox_extension.cc
index 62b8397536d177b9dbba5cf96dc8cbf9c50aec6a..ba46a3ebf37b9ad8dda1128ff08255a0b6020f0f 100644
--- a/chrome/renderer/searchbox/searchbox_extension.cc
+++ b/chrome/renderer/searchbox/searchbox_extension.cc
@@ -216,6 +216,15 @@ static const char kDispatchMostVisitedChangedScript[] =
" true;"
"}";
+static const char kDispatchBarsHiddenEventScript[] =
+ "if (window.chrome &&"
+ " window.chrome.searchBox &&"
+ " window.chrome.searchBox.onbarshidden &&"
+ " typeof window.chrome.searchBox.onbarshidden == 'function') {"
+ " window.chrome.searchBox.onbarshidden();"
+ " true;"
+ "}";
+
// ----------------------------------------------------------------------------
class SearchBoxExtensionWrapper : public v8::Extension {
@@ -336,6 +345,13 @@ class SearchBoxExtensionWrapper : public v8::Extension {
static v8::Handle<v8::Value> UndoMostVisitedDeletion(
const v8::Arguments& args);
+ // Shows any attached bars.
+ static v8::Handle<v8::Value> ShowBars(const v8::Arguments& args);
+
+ // Hides any attached bars. When the bars are hidden, the "onbarshidden"
+ // event is fired to notify the page.
+ static v8::Handle<v8::Value> HideBars(const v8::Arguments& args);
+
private:
DISALLOW_COPY_AND_ASSIGN(SearchBoxExtensionWrapper);
};
@@ -407,6 +423,10 @@ v8::Handle<v8::FunctionTemplate> SearchBoxExtensionWrapper::GetNativeFunction(
return v8::FunctionTemplate::New(UndoAllMostVisitedDeletions);
if (name->Equals(v8::String::New("UndoMostVisitedDeletion")))
return v8::FunctionTemplate::New(UndoMostVisitedDeletion);
+ if (name->Equals(v8::String::New("ShowBars")))
+ return v8::FunctionTemplate::New(ShowBars);
+ if (name->Equals(v8::String::New("HideBars")))
+ return v8::FunctionTemplate::New(HideBars);
return v8::Handle<v8::FunctionTemplate>();
}
@@ -1009,6 +1029,27 @@ bool SearchBoxExtension::PageSupportsInstant(WebKit::WebFrame* frame) {
return !v.IsEmpty() && v->BooleanValue();
}
+v8::Handle<v8::Value> SearchBoxExtensionWrapper::ShowBars(
+ const v8::Arguments& args) {
+ content::RenderView* render_view = GetRenderView();
+ if (!render_view) return v8::Undefined();
+
+ DVLOG(1) << render_view << " ShowBars";
+ SearchBox::Get(render_view)->ShowBars();
+ return v8::Undefined();
+}
+
+// static
+v8::Handle<v8::Value> SearchBoxExtensionWrapper::HideBars(
+ const v8::Arguments& args) {
+ content::RenderView* render_view = GetRenderView();
+ if (!render_view) return v8::Undefined();
+
+ DVLOG(1) << render_view << " HideBars";
+ SearchBox::Get(render_view)->HideBars();
+ return v8::Undefined();
+}
+
// static
void SearchBoxExtension::DispatchChange(WebKit::WebFrame* frame) {
Dispatch(frame, kDispatchChangeEventScript);
@@ -1073,4 +1114,9 @@ void SearchBoxExtension::DispatchMostVisitedChanged(
WebKit::WebFrame* frame) {
Dispatch(frame, kDispatchMostVisitedChangedScript);
}
+
+void SearchBoxExtension::DispatchBarsHidden(WebKit::WebFrame* frame) {
+ Dispatch(frame, kDispatchBarsHiddenEventScript);
+}
+
} // namespace extensions_v8
« chrome/renderer/searchbox/searchbox.cc ('K') | « chrome/renderer/searchbox/searchbox_extension.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698