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

Side by Side Diff: chrome/browser/ui/panels/panel_host.cc

Issue 10978016: Remove two functions on WebContentsDelegate which didn't belong in content. They were only called f… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: nits and fixed prerender browser tests Created 8 years, 2 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 | Annotate | Revision Log
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 "chrome/browser/ui/panels/panel_host.h" 5 #include "chrome/browser/ui/panels/panel_host.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "chrome/browser/chrome_page_zoom.h" 10 #include "chrome/browser/chrome_page_zoom.h"
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 } 164 }
165 165
166 bool PanelHost::IsPopupOrPanel(const content::WebContents* source) const { 166 bool PanelHost::IsPopupOrPanel(const content::WebContents* source) const {
167 return true; 167 return true;
168 } 168 }
169 169
170 void PanelHost::ContentsZoomChange(bool zoom_in) { 170 void PanelHost::ContentsZoomChange(bool zoom_in) {
171 Zoom(zoom_in ? content::PAGE_ZOOM_IN : content::PAGE_ZOOM_OUT); 171 Zoom(zoom_in ? content::PAGE_ZOOM_IN : content::PAGE_ZOOM_OUT);
172 } 172 }
173 173
174 bool PanelHost::IsApplication() const {
175 return true;
176 }
177
178 bool PanelHost::HandleContextMenu(const content::ContextMenuParams& params) { 174 bool PanelHost::HandleContextMenu(const content::ContextMenuParams& params) {
179 return true; // Disallow context menu. 175 return true; // Disallow context menu.
180 } 176 }
181 177
182 void PanelHost::HandleKeyboardEvent( 178 void PanelHost::HandleKeyboardEvent(
183 content::WebContents* source, 179 content::WebContents* source,
184 const content::NativeWebKeyboardEvent& event) { 180 const content::NativeWebKeyboardEvent& event) {
185 return panel_->HandleKeyboardEvent(event); 181 return panel_->HandleKeyboardEvent(event);
186 } 182 }
187 183
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
253 } 249 }
254 250
255 void PanelHost::StopLoading() { 251 void PanelHost::StopLoading() {
256 content::RecordAction(UserMetricsAction("Stop")); 252 content::RecordAction(UserMetricsAction("Stop"));
257 web_contents_->Stop(); 253 web_contents_->Stop();
258 } 254 }
259 255
260 void PanelHost::Zoom(content::PageZoom zoom) { 256 void PanelHost::Zoom(content::PageZoom zoom) {
261 chrome_page_zoom::Zoom(web_contents_.get(), zoom); 257 chrome_page_zoom::Zoom(web_contents_.get(), zoom);
262 } 258 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698