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

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

Issue 1423053002: Make document.activeElement work with OOPIF (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@focus-preserve-page-focus-on-subframe-navigations
Patch Set: Remove some plumbing that should instead be introduced in the window.focus() CL. Created 5 years, 1 month 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 <map> 8 #include <map>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 369 matching lines...) Expand 10 before | Expand all | Expand 10 after
380 // Returns true if a call to DispatchBeforeUnload will actually send the 380 // Returns true if a call to DispatchBeforeUnload will actually send the
381 // BeforeUnload IPC. This is the case if the current renderer is live and this 381 // BeforeUnload IPC. This is the case if the current renderer is live and this
382 // frame is the main frame. 382 // frame is the main frame.
383 bool ShouldDispatchBeforeUnload(); 383 bool ShouldDispatchBeforeUnload();
384 384
385 // Update the frame's opener in the renderer process in response to the 385 // Update the frame's opener in the renderer process in response to the
386 // opener being modified (e.g., with window.open or being set to null) in 386 // opener being modified (e.g., with window.open or being set to null) in
387 // another renderer process. 387 // another renderer process.
388 void UpdateOpener(); 388 void UpdateOpener();
389 389
390 // Clear focus from this frame in the renderer process.
391 void ClearFocus();
392
393 // Deletes the current selection plus the specified number of characters 390 // Deletes the current selection plus the specified number of characters
394 // before and after the selection or caret. 391 // before and after the selection or caret.
395 void ExtendSelectionAndDelete(size_t before, size_t after); 392 void ExtendSelectionAndDelete(size_t before, size_t after);
396 393
397 // Notifies the RenderFrame that the JavaScript message that was shown was 394 // Notifies the RenderFrame that the JavaScript message that was shown was
398 // closed by the user. 395 // closed by the user.
399 void JavaScriptDialogClosed(IPC::Message* reply_msg, 396 void JavaScriptDialogClosed(IPC::Message* reply_msg,
400 bool success, 397 bool success,
401 const base::string16& user_input, 398 const base::string16& user_input,
402 bool dialog_was_suppressed); 399 bool dialog_was_suppressed);
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
827 824
828 // NOTE: This must be the last member. 825 // NOTE: This must be the last member.
829 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_; 826 base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;
830 827
831 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl); 828 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
832 }; 829 };
833 830
834 } // namespace content 831 } // namespace content
835 832
836 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_ 833 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698