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

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

Issue 1934703002: Fix keyboard focus for OOPIF-<webview>. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address nasko@ comments. Created 4 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 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 432 matching lines...) Expand 10 before | Expand all | Expand 10 after
443 443
444 // Update the frame's opener in the renderer process in response to the 444 // Update the frame's opener in the renderer process in response to the
445 // opener being modified (e.g., with window.open or being set to null) in 445 // opener being modified (e.g., with window.open or being set to null) in
446 // another renderer process. 446 // another renderer process.
447 void UpdateOpener(); 447 void UpdateOpener();
448 448
449 // Set this frame as focused in the renderer process. This supports 449 // Set this frame as focused in the renderer process. This supports
450 // cross-process window.focus() calls. 450 // cross-process window.focus() calls.
451 void SetFocusedFrame(); 451 void SetFocusedFrame();
452 452
453 // This is used to clear focus inside an inner WebContents when focus shifts
454 // to a frame in the outer WebContents or a sibling WebContents.
455 void ClearFocusedFrame();
456
453 // Deletes the current selection plus the specified number of characters 457 // Deletes the current selection plus the specified number of characters
454 // before and after the selection or caret. 458 // before and after the selection or caret.
455 void ExtendSelectionAndDelete(size_t before, size_t after); 459 void ExtendSelectionAndDelete(size_t before, size_t after);
456 460
457 // Notifies the RenderFrame that the JavaScript message that was shown was 461 // Notifies the RenderFrame that the JavaScript message that was shown was
458 // closed by the user. 462 // closed by the user.
459 void JavaScriptDialogClosed(IPC::Message* reply_msg, 463 void JavaScriptDialogClosed(IPC::Message* reply_msg,
460 bool success, 464 bool success,
461 const base::string16& user_input, 465 const base::string16& user_input,
462 bool dialog_was_suppressed); 466 bool dialog_was_suppressed);
(...skipping 579 matching lines...) Expand 10 before | Expand all | Expand 10 after
1042 1046
1043 // NOTE: This must be the last member. 1047 // NOTE: This must be the last member.
1044 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_; 1048 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;
1045 1049
1046 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl); 1050 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
1047 }; 1051 };
1048 1052
1049 } // namespace content 1053 } // namespace content
1050 1054
1051 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 1055 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_frame_host_delegate.h ('k') | content/browser/frame_host/render_frame_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698