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

Side by Side Diff: chrome/browser/external_tab_container_win.cc

Issue 5783004: Keep deinlining stuff. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Well, it all compiles locally? Created 10 years 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 "chrome/browser/external_tab_container_win.h" 5 #include "chrome/browser/external_tab_container_win.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "app/l10n_util.h" 9 #include "app/l10n_util.h"
10 #include "app/view_prop.h" 10 #include "app/view_prop.h"
(...skipping 604 matching lines...) Expand 10 before | Expand all | Expand 10 after
615 } 615 }
616 external_context_menu_.reset( 616 external_context_menu_.reset(
617 new RenderViewContextMenuViews(tab_contents(), params)); 617 new RenderViewContextMenuViews(tab_contents(), params));
618 external_context_menu_->SetExternal(); 618 external_context_menu_->SetExternal();
619 external_context_menu_->Init(); 619 external_context_menu_->Init();
620 external_context_menu_->UpdateMenuItemStates(); 620 external_context_menu_->UpdateMenuItemStates();
621 621
622 POINT screen_pt = { params.x, params.y }; 622 POINT screen_pt = { params.x, params.y };
623 MapWindowPoints(GetNativeView(), HWND_DESKTOP, &screen_pt, 1); 623 MapWindowPoints(GetNativeView(), HWND_DESKTOP, &screen_pt, 1);
624 624
625 IPC::MiniContextMenuParams ipc_params; 625 IPC::MiniContextMenuParams ipc_params(
626 ipc_params.screen_x = screen_pt.x; 626 screen_pt.x,
627 ipc_params.screen_y = screen_pt.y; 627 screen_pt.y,
628 ipc_params.link_url = params.link_url; 628 params.link_url,
629 ipc_params.unfiltered_link_url = params.unfiltered_link_url; 629 params.unfiltered_link_url,
630 ipc_params.src_url = params.src_url; 630 params.src_url,
631 ipc_params.page_url = params.page_url; 631 params.page_url,
632 ipc_params.frame_url = params.frame_url; 632 params.frame_url);
633 633
634 bool rtl = base::i18n::IsRTL(); 634 bool rtl = base::i18n::IsRTL();
635 automation_->Send( 635 automation_->Send(
636 new AutomationMsg_ForwardContextMenuToExternalHost(0, tab_handle_, 636 new AutomationMsg_ForwardContextMenuToExternalHost(0, tab_handle_,
637 external_context_menu_->GetMenuHandle(), 637 external_context_menu_->GetMenuHandle(),
638 rtl ? TPM_RIGHTALIGN : TPM_LEFTALIGN, ipc_params)); 638 rtl ? TPM_RIGHTALIGN : TPM_LEFTALIGN, ipc_params));
639 639
640 return true; 640 return true;
641 } 641 }
642 642
(...skipping 470 matching lines...) Expand 10 before | Expand all | Expand 10 after
1113 1113
1114 if (disposition == CURRENT_TAB) { 1114 if (disposition == CURRENT_TAB) {
1115 DCHECK(route_all_top_level_navigations_); 1115 DCHECK(route_all_top_level_navigations_);
1116 disposition = NEW_FOREGROUND_TAB; 1116 disposition = NEW_FOREGROUND_TAB;
1117 } 1117 }
1118 ExternalTabContainer::OpenURLFromTab(source, url, referrer, disposition, 1118 ExternalTabContainer::OpenURLFromTab(source, url, referrer, disposition,
1119 transition); 1119 transition);
1120 // support only one navigation for a dummy tab before it is killed. 1120 // support only one navigation for a dummy tab before it is killed.
1121 ::DestroyWindow(GetNativeView()); 1121 ::DestroyWindow(GetNativeView());
1122 } 1122 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698