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

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

Issue 2617443002: Implement ThreadedInputConnection.deleteSurroundingTextInCodePoints() (Closed)
Patch Set: Add more comments Created 3 years, 11 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 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 418 matching lines...) Expand 10 before | Expand all | Expand 10 after
429 429
430 // Set this frame as focused in the renderer process. This supports 430 // Set this frame as focused in the renderer process. This supports
431 // cross-process window.focus() calls. 431 // cross-process window.focus() calls.
432 void SetFocusedFrame(); 432 void SetFocusedFrame();
433 433
434 // Deletes the current selection plus the specified number of characters 434 // Deletes the current selection plus the specified number of characters
435 // before and after the selection or caret. 435 // before and after the selection or caret.
436 void ExtendSelectionAndDelete(size_t before, size_t after); 436 void ExtendSelectionAndDelete(size_t before, size_t after);
437 437
438 // Deletes text before and after the current cursor position, excluding the 438 // Deletes text before and after the current cursor position, excluding the
439 // selection. 439 // selection. The lengths are supplied in Java chars, not in code points or in
440 // glyphs.
440 void DeleteSurroundingText(size_t before, size_t after); 441 void DeleteSurroundingText(size_t before, size_t after);
441 442
443 // Deletes text before and after the current cursor position, excluding the
444 // selection. The lengths are supplied in code points, not in Java chars or in
445 // glyphs. Do nothing if there are one or more invalid surrogate pairs in the
446 // requested range
Changwan Ryu 2017/01/11 02:00:27 Period at the end?
yabinh 2017/01/24 11:39:56 Done.
447 void DeleteSurroundingTextInCodePoints(size_t before, size_t after);
448
442 // Notifies the RenderFrame that the JavaScript message that was shown was 449 // Notifies the RenderFrame that the JavaScript message that was shown was
443 // closed by the user. 450 // closed by the user.
444 void JavaScriptDialogClosed(IPC::Message* reply_msg, 451 void JavaScriptDialogClosed(IPC::Message* reply_msg,
445 bool success, 452 bool success,
446 const base::string16& user_input, 453 const base::string16& user_input,
447 bool dialog_was_suppressed); 454 bool dialog_was_suppressed);
448 455
449 // Get the accessibility mode from the delegate and Send a message to the 456 // Get the accessibility mode from the delegate and Send a message to the
450 // renderer process to change the accessibility mode. 457 // renderer process to change the accessibility mode.
451 void UpdateAccessibilityMode(); 458 void UpdateAccessibilityMode();
(...skipping 654 matching lines...) Expand 10 before | Expand all | Expand 10 after
1106 1113
1107 // NOTE: This must be the last member. 1114 // NOTE: This must be the last member.
1108 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_; 1115 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;
1109 1116
1110 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl); 1117 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
1111 }; 1118 };
1112 1119
1113 } // namespace content 1120 } // namespace content
1114 1121
1115 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 1122 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698