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

Side by Side Diff: content/renderer/render_view_impl.cc

Issue 1977633003: Remove RenderFrameImpl::GetFocusedElement. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes the right click + blur + copy case. Created 4 years, 7 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/renderer/render_view_impl.h" 5 #include "content/renderer/render_view_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <memory> 9 #include <memory>
10 10
(...skipping 1663 matching lines...) Expand 10 before | Expand all | Expand 10 after
1674 1674
1675 EditCommands::iterator it = edit_commands_.begin(); 1675 EditCommands::iterator it = edit_commands_.begin();
1676 EditCommands::iterator end = edit_commands_.end(); 1676 EditCommands::iterator end = edit_commands_.end();
1677 1677
1678 bool did_execute_command = false; 1678 bool did_execute_command = false;
1679 for (; it != end; ++it) { 1679 for (; it != end; ++it) {
1680 // In gtk and cocoa, it's possible to bind multiple edit commands to one 1680 // In gtk and cocoa, it's possible to bind multiple edit commands to one
1681 // key (but it's the exception). Once one edit command is not executed, it 1681 // key (but it's the exception). Once one edit command is not executed, it
1682 // seems safest to not execute the rest. 1682 // seems safest to not execute the rest.
1683 if (!frame->executeCommand(WebString::fromUTF8(it->name), 1683 if (!frame->executeCommand(WebString::fromUTF8(it->name),
1684 WebString::fromUTF8(it->value), 1684 WebString::fromUTF8(it->value)))
1685 GetFocusedElement()))
1686 break; 1685 break;
1687 did_execute_command = true; 1686 did_execute_command = true;
1688 } 1687 }
1689 1688
1690 return did_execute_command; 1689 return did_execute_command;
1691 } 1690 }
1692 1691
1693 bool RenderViewImpl::runFileChooser( 1692 bool RenderViewImpl::runFileChooser(
1694 const blink::WebFileChooserParams& params, 1693 const blink::WebFileChooserParams& params,
1695 WebFileChooserCompletion* chooser_completion) { 1694 WebFileChooserCompletion* chooser_completion) {
(...skipping 1694 matching lines...) Expand 10 before | Expand all | Expand 10 after
3390 return render_frame->focused_pepper_plugin(); 3389 return render_frame->focused_pepper_plugin();
3391 } 3390 }
3392 frame = frame->traverseNext(false); 3391 frame = frame->traverseNext(false);
3393 } 3392 }
3394 3393
3395 return nullptr; 3394 return nullptr;
3396 } 3395 }
3397 #endif 3396 #endif
3398 3397
3399 } // namespace content 3398 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | third_party/WebKit/Source/web/WebLocalFrameImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698