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

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: revert obsolete changes in search.h/cc 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 25c4e09c1cb55eb41e68681d7746a3f9d8f34710..e0c6bdd5970614f12dcce17b145e250f3fa99f11 100644
--- a/chrome/renderer/searchbox/searchbox_extension.cc
+++ b/chrome/renderer/searchbox/searchbox_extension.cc
@@ -238,6 +238,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 {
@@ -364,6 +373,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);
};
@@ -439,6 +455,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>();
}
@@ -1085,6 +1105,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);
@@ -1149,4 +1190,9 @@ void SearchBoxExtension::DispatchMostVisitedChanged(
WebKit::WebFrame* frame) {
Dispatch(frame, kDispatchMostVisitedChangedScript);
}
+
+void SearchBoxExtension::DispatchBarsHidden(WebKit::WebFrame* frame) {
+ Dispatch(frame, kDispatchBarsHiddenEventScript);
+}
+
} // namespace extensions_v8

Powered by Google App Engine
This is Rietveld 408576698