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

Unified Diff: content/renderer/render_view_impl.cc

Issue 10836195: Remove obsolete code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index c14703332c1adcecfa0e42d21e769d213342ef8f..57941a5fc3df8b4ebec09a1ad85cf1d06de66697 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -877,7 +877,6 @@ bool RenderViewImpl::OnMessageReceived(const IPC::Message& message) {
IPC_MESSAGE_HANDLER(ViewMsg_ExecuteEditCommand, OnExecuteEditCommand)
IPC_MESSAGE_HANDLER(ViewMsg_Find, OnFind)
IPC_MESSAGE_HANDLER(ViewMsg_StopFinding, OnStopFinding)
- IPC_MESSAGE_HANDLER(ViewMsg_FindReplyACK, OnFindReplyAck)
IPC_MESSAGE_HANDLER(ViewMsg_Zoom, OnZoom)
IPC_MESSAGE_HANDLER(ViewMsg_SetZoomLevel, OnSetZoomLevel)
IPC_MESSAGE_HANDLER(ViewMsg_ZoomFactor, OnZoomFactor)
@@ -3659,23 +3658,12 @@ void RenderViewImpl::reportFindInPageMatchCount(int request_id, int count,
if (!count)
active_match_ordinal = 0;
- IPC::Message* msg = new ViewHostMsg_Find_Reply(
- routing_id_,
- request_id,
- count,
- gfx::Rect(),
- active_match_ordinal,
- final_update);
-
- // If we have a message that has been queued up, then we should just replace
- // it. The ACK from the browser will make sure it gets sent when the browser
- // wants it.
- if (queued_find_reply_message_.get()) {
- queued_find_reply_message_.reset(msg);
- } else {
- // Send the search result over to the browser process.
- Send(msg);
- }
+ Send(new ViewHostMsg_Find_Reply(routing_id_,
+ request_id,
+ count,
+ gfx::Rect(),
+ active_match_ordinal,
+ final_update));
}
void RenderViewImpl::reportFindInPageSelection(int request_id,
@@ -4440,14 +4428,6 @@ void RenderViewImpl::OnStopFinding(content::StopFindAction action) {
}
}
-void RenderViewImpl::OnFindReplyAck() {
- // Check if there is any queued up request waiting to be sent.
- if (queued_find_reply_message_.get()) {
- // Send the search result over to the browser process.
- Send(queued_find_reply_message_.release());
- }
-}
-
void RenderViewImpl::OnZoom(content::PageZoom zoom) {
if (!webview()) // Not sure if this can happen, but no harm in being safe.
return;
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698