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

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: Fixed test flakiness and comments in PS 7,8. Created 4 years, 6 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 416 matching lines...) Expand 10 before | Expand all | Expand 10 after
427 427
428 // Update the frame's opener in the renderer process in response to the 428 // Update the frame's opener in the renderer process in response to the
429 // opener being modified (e.g., with window.open or being set to null) in 429 // opener being modified (e.g., with window.open or being set to null) in
430 // another renderer process. 430 // another renderer process.
431 void UpdateOpener(); 431 void UpdateOpener();
432 432
433 // Set this frame as focused in the renderer process. This supports 433 // Set this frame as focused in the renderer process. This supports
434 // cross-process window.focus() calls. 434 // cross-process window.focus() calls.
435 void SetFocusedFrame(); 435 void SetFocusedFrame();
436 436
437 // This is used to clear focus inside an inner WebContents when focus shifts
438 // to a frame in the outer WebContents or a sibling WebContents.
439 void ClearFocusedFrame();
440
437 // Deletes the current selection plus the specified number of characters 441 // Deletes the current selection plus the specified number of characters
438 // before and after the selection or caret. 442 // before and after the selection or caret.
439 void ExtendSelectionAndDelete(size_t before, size_t after); 443 void ExtendSelectionAndDelete(size_t before, size_t after);
440 444
441 // Notifies the RenderFrame that the JavaScript message that was shown was 445 // Notifies the RenderFrame that the JavaScript message that was shown was
442 // closed by the user. 446 // closed by the user.
443 void JavaScriptDialogClosed(IPC::Message* reply_msg, 447 void JavaScriptDialogClosed(IPC::Message* reply_msg,
444 bool success, 448 bool success,
445 const base::string16& user_input, 449 const base::string16& user_input,
446 bool dialog_was_suppressed); 450 bool dialog_was_suppressed);
(...skipping 549 matching lines...) Expand 10 before | Expand all | Expand 10 after
996 1000
997 // NOTE: This must be the last member. 1001 // NOTE: This must be the last member.
998 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_; 1002 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;
999 1003
1000 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl); 1004 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
1001 }; 1005 };
1002 1006
1003 } // namespace content 1007 } // namespace content
1004 1008
1005 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 1009 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698