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

Unified Diff: chrome/renderer/chrome_render_view_observer.cc

Issue 71723003: Remove HistoryPublisher code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove ref to history_indexer.idl Created 7 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
« no previous file with comments | « chrome/common/render_messages.h ('k') | tools/memory_watcher/scripts/summary.pl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/chrome_render_view_observer.cc
diff --git a/chrome/renderer/chrome_render_view_observer.cc b/chrome/renderer/chrome_render_view_observer.cc
index 8d5f088da2250790ce11782be372a2868033bc80..fc75dad1c61ff9b83a11c62ad6bbd72658b1671a 100644
--- a/chrome/renderer/chrome_render_view_observer.cc
+++ b/chrome/renderer/chrome_render_view_observer.cc
@@ -911,6 +911,8 @@ void ChromeRenderViewObserver::CapturePageInfo(int page_id,
if (translate_helper_)
translate_helper_->PageCaptured(page_id, contents);
+ // TODO(shess): Is indexing "Full text search" indexing? In that
+ // case more of this can go.
// Skip indexing if this is not a new load. Note that the case where
// page_id == last_indexed_page_id_ is more complicated, since we need to
// reindex if the toplevel URL has changed (such as from a redirect), even
@@ -944,12 +946,6 @@ void ChromeRenderViewObserver::CapturePageInfo(int page_id,
TRACE_EVENT0("renderer", "ChromeRenderViewObserver::CapturePageInfo");
- if (contents.size()) {
- // Send the text to the browser for indexing (the browser might decide not
- // to index, if the URL is HTTPS for instance).
- Send(new ChromeViewHostMsg_PageContents(routing_id(), url, contents));
- }
-
#if defined(FULL_SAFE_BROWSING)
// Will swap out the string.
if (phishing_classifier_)
« no previous file with comments | « chrome/common/render_messages.h ('k') | tools/memory_watcher/scripts/summary.pl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698