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

Side by Side Diff: content/browser/frame_host/render_frame_host_impl.cc

Issue 2617443002: Implement ThreadedInputConnection.deleteSurroundingTextInCodePoints() (Closed)
Patch Set: Add some DCHECK Created 3 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/frame_host/render_frame_host_impl.h" 5 #include "content/browser/frame_host/render_frame_host_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 2623 matching lines...) Expand 10 before | Expand all | Expand 10 after
2634 2634
2635 void RenderFrameHostImpl::ExtendSelectionAndDelete(size_t before, 2635 void RenderFrameHostImpl::ExtendSelectionAndDelete(size_t before,
2636 size_t after) { 2636 size_t after) {
2637 Send(new InputMsg_ExtendSelectionAndDelete(routing_id_, before, after)); 2637 Send(new InputMsg_ExtendSelectionAndDelete(routing_id_, before, after));
2638 } 2638 }
2639 2639
2640 void RenderFrameHostImpl::DeleteSurroundingText(size_t before, size_t after) { 2640 void RenderFrameHostImpl::DeleteSurroundingText(size_t before, size_t after) {
2641 Send(new InputMsg_DeleteSurroundingText(routing_id_, before, after)); 2641 Send(new InputMsg_DeleteSurroundingText(routing_id_, before, after));
2642 } 2642 }
2643 2643
2644 void RenderFrameHostImpl::DeleteSurroundingTextInCodePoints(size_t before,
dcheng 2017/02/25 07:37:03 I'm wondering why the caller of this function uses
yabinh 2017/02/27 02:04:33 Done. Replaced size_t with int.
2645 size_t after) {
2646 Send(new InputMsg_DeleteSurroundingTextInCodePoints(routing_id_, before,
2647 after));
2648 }
2649
2644 void RenderFrameHostImpl::JavaScriptDialogClosed( 2650 void RenderFrameHostImpl::JavaScriptDialogClosed(
2645 IPC::Message* reply_msg, 2651 IPC::Message* reply_msg,
2646 bool success, 2652 bool success,
2647 const base::string16& user_input, 2653 const base::string16& user_input,
2648 bool dialog_was_suppressed) { 2654 bool dialog_was_suppressed) {
2649 GetProcess()->SetIgnoreInputEvents(false); 2655 GetProcess()->SetIgnoreInputEvents(false);
2650 bool is_waiting = is_waiting_for_beforeunload_ack_ || IsWaitingForUnloadACK(); 2656 bool is_waiting = is_waiting_for_beforeunload_ack_ || IsWaitingForUnloadACK();
2651 2657
2652 // If we are executing as part of (before)unload event handling, we don't 2658 // If we are executing as part of (before)unload event handling, we don't
2653 // want to use the regular hung_renderer_delay_ms_ if the user has agreed to 2659 // want to use the regular hung_renderer_delay_ms_ if the user has agreed to
(...skipping 838 matching lines...) Expand 10 before | Expand all | Expand 10 after
3492 // There is no pending NavigationEntry in these cases, so pass 0 as the 3498 // There is no pending NavigationEntry in these cases, so pass 0 as the
3493 // pending_nav_entry_id. If the previous handle was a prematurely aborted 3499 // pending_nav_entry_id. If the previous handle was a prematurely aborted
3494 // navigation loaded via LoadDataWithBaseURL, propagate the entry id. 3500 // navigation loaded via LoadDataWithBaseURL, propagate the entry id.
3495 return NavigationHandleImpl::Create( 3501 return NavigationHandleImpl::Create(
3496 params.url, params.redirects, frame_tree_node_, is_renderer_initiated, 3502 params.url, params.redirects, frame_tree_node_, is_renderer_initiated,
3497 params.was_within_same_page, base::TimeTicks::Now(), 3503 params.was_within_same_page, base::TimeTicks::Now(),
3498 entry_id_for_data_nav, false); // started_from_context_menu 3504 entry_id_for_data_nav, false); // started_from_context_menu
3499 } 3505 }
3500 3506
3501 } // namespace content 3507 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698