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

Unified Diff: content/renderer/render_view_impl.cc

Issue 2374863002: Remove usage of HostZoomMap from c/b/loader via ReadyToCommitNavigation (Closed)
Patch Set: init Created 4 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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 93479bcf1bd718cd0da2355b5ca3a92d82777561..9649a90a511d3cb259b642148e7ca7eac6225f88 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -1316,8 +1316,6 @@ bool RenderViewImpl::OnMessageReceived(const IPC::Message& message) {
OnScrollFocusedEditableNodeIntoRect)
IPC_MESSAGE_HANDLER(ViewMsg_SetPageScale, OnSetPageScale)
IPC_MESSAGE_HANDLER(ViewMsg_Zoom, OnZoom)
- IPC_MESSAGE_HANDLER(ViewMsg_SetZoomLevelForLoadingURL,
- OnSetZoomLevelForLoadingURL)
IPC_MESSAGE_HANDLER(DragMsg_TargetDragEnter, OnDragTargetDragEnter)
IPC_MESSAGE_HANDLER(DragMsg_TargetDragOver, OnDragTargetDragOver)
IPC_MESSAGE_HANDLER(DragMsg_TargetDragLeave, OnDragTargetDragLeave)
@@ -2244,21 +2242,6 @@ void RenderViewImpl::OnZoom(PageZoom zoom) {
zoomLevelChanged();
}
-void RenderViewImpl::OnSetZoomLevelForLoadingURL(const GURL& url,
- double zoom_level) {
- // TODO(wjmaclean): We should see if this restriction is really necessary,
- // since it isn't enforced in other parts of the page zoom system (e.g.
- // when a users changes the zoom of a currently displayed page). Android
- // has no UI for this, so in theory the following code would normally just use
- // the default zoom anyways.
-#if !defined(OS_ANDROID)
- // On Android, page zoom isn't used, and in case of WebView, text zoom is used
- // for legacy WebView text scaling emulation. Thus, the code that resets
- // the zoom level from this map will be effectively resetting text zoom level.
- host_zoom_levels_[url] = zoom_level;
-#endif
-}
-
void RenderViewImpl::OnSetZoomLevel(
PageMsg_SetZoomLevel_Command command,
double zoom_level) {

Powered by Google App Engine
This is Rietveld 408576698