diff options
| author | brettw@google.com <brettw@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-25 22:11:06 +0000 |
|---|---|---|
| committer | brettw@google.com <brettw@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-25 22:11:06 +0000 |
| commit | 1f5af444f94761190a16ac6088ad6e6831aeb4a0 (patch) | |
| tree | 6c89513715320dacc744a49e0e2104a4ee3e9807 /chrome/browser/render_view_context_menu_controller.cc | |
| parent | de8d2667d6953abb31b7c5385ca718ad47adf6a1 (diff) | |
| download | chromium_src-1f5af444f94761190a16ac6088ad6e6831aeb4a0.zip chromium_src-1f5af444f94761190a16ac6088ad6e6831aeb4a0.tar.gz chromium_src-1f5af444f94761190a16ac6088ad6e6831aeb4a0.tar.bz2 | |
Move a bunch of stuff out of WebContents. I removed a bunch of render view host pass-throughs and just made the callers call the render view host directly. I don't think we're trying to isolate the layers to this degree, and WebContents is so big these just added noise.
I removed the RenderViewHost->WebContents->SavePackage pass-through by using a delegate that the SavePackage implements (like we already do for find in page). I also noticed some file upload stuff wasn't used at all and removed it.
Review URL: http://codereview.chromium.org/4088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2612 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/render_view_context_menu_controller.cc')
| -rw-r--r-- | chrome/browser/render_view_context_menu_controller.cc | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/chrome/browser/render_view_context_menu_controller.cc b/chrome/browser/render_view_context_menu_controller.cc index 2968c6d..98eaf85 100644 --- a/chrome/browser/render_view_context_menu_controller.cc +++ b/chrome/browser/render_view_context_menu_controller.cc @@ -51,11 +51,11 @@ void RenderViewContextMenuController::OpenURL( } void RenderViewContextMenuController::CopyImageAt(int x, int y) { - source_web_contents_->CopyImageAt(x, y); + source_web_contents_->render_view_host()->CopyImageAt(x, y); } void RenderViewContextMenuController::Inspect(int x, int y) { - source_web_contents_->InspectElementAt(x, y); + source_web_contents_->render_view_host()->InspectElementAt(x, y); } void RenderViewContextMenuController::WriteTextToClipboard( @@ -344,31 +344,31 @@ void RenderViewContextMenuController::ExecuteCommand(int id) { } case IDS_CONTENT_CONTEXT_UNDO: - source_web_contents_->Undo(); + source_web_contents_->render_view_host()->Undo(); break; case IDS_CONTENT_CONTEXT_REDO: - source_web_contents_->Redo(); + source_web_contents_->render_view_host()->Redo(); break; case IDS_CONTENT_CONTEXT_CUT: - source_web_contents_->Cut(); + source_web_contents_->render_view_host()->Cut(); break; case IDS_CONTENT_CONTEXT_COPY: - source_web_contents_->Copy(); + source_web_contents_->render_view_host()->Copy(); break; case IDS_CONTENT_CONTEXT_PASTE: - source_web_contents_->Paste(); + source_web_contents_->render_view_host()->Paste(); break; case IDS_CONTENT_CONTEXT_DELETE: - source_web_contents_->Delete(); + source_web_contents_->render_view_host()->Delete(); break; case IDS_CONTENT_CONTEXT_SELECTALL: - source_web_contents_->SelectAll(); + source_web_contents_->render_view_host()->SelectAll(); break; case IDS_CONTENT_CONTEXT_SEARCHWEBFOR: { @@ -389,12 +389,13 @@ void RenderViewContextMenuController::ExecuteCommand(int id) { case IDC_USESPELLCHECKSUGGESTION_2: case IDC_USESPELLCHECKSUGGESTION_3: case IDC_USESPELLCHECKSUGGESTION_4: - source_web_contents_->Replace(params_.dictionary_suggestions[ - id - IDC_USESPELLCHECKSUGGESTION_0]); + source_web_contents_->render_view_host()->Replace( + params_.dictionary_suggestions[id - IDC_USESPELLCHECKSUGGESTION_0]); break; case IDS_CONTENT_CONTEXT_ADD_TO_DICTIONARY: - source_web_contents_->AddToDictionary(params_.misspelled_word); + source_web_contents_->render_view_host()->AddToDictionary( + params_.misspelled_word); break; case IDS_CONTENT_CONTEXT_ADDSEARCHENGINE: // Not implemented. |