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

Side by Side Diff: content/browser/host_zoom_map_observer.cc

Issue 2595603002: Revert of Maintain HostZoom connection per-frame on browser side (Closed)
Patch Set: Created 3 years, 12 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
« no previous file with comments | « content/browser/host_zoom_map_observer.h ('k') | content/browser/iframe_zoom_browsertest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/browser/host_zoom_map_observer.h" 5 #include "content/browser/host_zoom_map_observer.h"
6 6
7 #include "content/browser/frame_host/render_frame_host_impl.h" 7 #include "content/browser/frame_host/render_frame_host_impl.h"
8 #include "content/browser/host_zoom_map_impl.h" 8 #include "content/browser/host_zoom_map_impl.h"
9 #include "content/public/browser/navigation_handle.h" 9 #include "content/public/browser/navigation_handle.h"
10 #include "content/public/browser/render_view_host.h" 10 #include "content/public/browser/render_view_host.h"
11 #include "content/public/browser/storage_partition.h" 11 #include "content/public/browser/storage_partition.h"
12 #include "content/public/common/associated_interface_provider.h" 12 #include "content/public/common/associated_interface_provider.h"
13 13
14 namespace content { 14 namespace content {
15 15
16 HostZoomMapObserver::HostZoomMapObserver(WebContents* web_contents) 16 HostZoomMapObserver::HostZoomMapObserver(WebContents* web_contents)
17 : WebContentsObserver(web_contents) {} 17 : WebContentsObserver(web_contents) {}
18 18
19 HostZoomMapObserver::~HostZoomMapObserver() {} 19 HostZoomMapObserver::~HostZoomMapObserver() {}
20 20
21 void HostZoomMapObserver::ReadyToCommitNavigation( 21 void HostZoomMapObserver::ReadyToCommitNavigation(
22 NavigationHandle* navigation_handle) { 22 NavigationHandle* navigation_handle) {
23 if (!navigation_handle->IsInMainFrame()) 23 if (!navigation_handle->IsInMainFrame())
24 return; 24 return;
25 25
26 DCHECK(host_zoom_.is_bound());
27 if (!host_zoom_.is_bound())
28 return;
29
26 RenderFrameHost* render_frame_host = 30 RenderFrameHost* render_frame_host =
27 navigation_handle->GetRenderFrameHost(); 31 navigation_handle->GetRenderFrameHost();
28 const auto& entry = host_zoom_ptrs_.find(render_frame_host);
29 if (entry == host_zoom_ptrs_.end())
30 return;
31
32 const mojom::HostZoomAssociatedPtr& host_zoom = entry->second;
33 DCHECK(host_zoom.is_bound());
34 if (host_zoom.encountered_error())
35 return;
36
37 RenderProcessHost* render_process_host = render_frame_host->GetProcess(); 32 RenderProcessHost* render_process_host = render_frame_host->GetProcess();
38 HostZoomMapImpl* host_zoom_map = static_cast<HostZoomMapImpl*>( 33 HostZoomMapImpl* host_zoom_map = static_cast<HostZoomMapImpl*>(
39 render_process_host->GetStoragePartition()->GetHostZoomMap()); 34 render_process_host->GetStoragePartition()->GetHostZoomMap());
40 double zoom_level = host_zoom_map->GetZoomLevelForView( 35 double zoom_level = host_zoom_map->GetZoomLevelForView(
41 navigation_handle->GetURL(), render_process_host->GetID(), 36 navigation_handle->GetURL(), render_process_host->GetID(),
42 render_frame_host->GetRenderViewHost()->GetRoutingID()); 37 render_frame_host->GetRenderViewHost()->GetRoutingID());
43 host_zoom->SetHostZoomLevel(navigation_handle->GetURL(), zoom_level); 38 host_zoom_->SetHostZoomLevel(navigation_handle->GetURL(), zoom_level);
44 } 39 }
45 40
46 void HostZoomMapObserver::RenderFrameCreated(RenderFrameHost* rfh) { 41 void HostZoomMapObserver::RenderFrameCreated(RenderFrameHost* rfh) {
47 mojom::HostZoomAssociatedPtr host_zoom; 42 rfh->GetRemoteAssociatedInterfaces()->GetInterface(&host_zoom_);
48 rfh->GetRemoteAssociatedInterfaces()->GetInterface(&host_zoom);
49 host_zoom_ptrs_[rfh] = std::move(host_zoom);
50 }
51
52 void HostZoomMapObserver::RenderFrameDeleted(RenderFrameHost* rfh) {
53 const auto& entry = host_zoom_ptrs_.find(rfh);
54 DCHECK(entry != host_zoom_ptrs_.end());
55 host_zoom_ptrs_.erase(entry);
56 } 43 }
57 44
58 } // namespace content 45 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/host_zoom_map_observer.h ('k') | content/browser/iframe_zoom_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698