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

Unified Diff: content/browser/host_zoom_map_impl.cc

Issue 9296041: Create Content API around HostZoomMap. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 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
« no previous file with comments | « content/browser/host_zoom_map_impl.h ('k') | content/browser/host_zoom_map_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/host_zoom_map_impl.cc
===================================================================
--- content/browser/host_zoom_map_impl.cc (revision 119530)
+++ content/browser/host_zoom_map_impl.cc (working copy)
@@ -4,7 +4,7 @@
#include <cmath>
-#include "content/browser/host_zoom_map.h"
+#include "content/browser/host_zoom_map_impl.h"
#include "base/string_piece.h"
#include "base/utf_string_conversions.h"
@@ -25,19 +25,28 @@
using content::BrowserThread;
using content::RenderProcessHost;
-HostZoomMap::HostZoomMap()
+namespace content {
+
+HostZoomMap* HostZoomMap::Create() {
+ return new HostZoomMapImpl();
+}
+
+} // namespace content
+
+HostZoomMapImpl::HostZoomMapImpl()
: default_zoom_level_(0.0) {
registrar_.Add(
this, content::NOTIFICATION_RENDER_VIEW_HOST_WILL_CLOSE_RENDER_VIEW,
content::NotificationService::AllSources());
}
-void HostZoomMap::CopyFrom(HostZoomMap* copy) {
+void HostZoomMapImpl::CopyFrom(HostZoomMap* copy_interface) {
// This can only be called on the UI thread to avoid deadlocks, otherwise
// UI: a.CopyFrom(b);
// IO: b.CopyFrom(a);
// can deadlock.
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ HostZoomMapImpl* copy = static_cast<HostZoomMapImpl*>(copy_interface);
base::AutoLock auto_lock(lock_);
base::AutoLock copy_auto_lock(copy->lock_);
for (HostZoomLevels::const_iterator i(copy->host_zoom_levels_.begin());
@@ -46,13 +55,13 @@
}
}
-double HostZoomMap::GetZoomLevel(const std::string& host) const {
+double HostZoomMapImpl::GetZoomLevel(const std::string& host) const {
base::AutoLock auto_lock(lock_);
HostZoomLevels::const_iterator i(host_zoom_levels_.find(host));
return (i == host_zoom_levels_.end()) ? default_zoom_level_ : i->second;
}
-void HostZoomMap::SetZoomLevel(std::string host, double level) {
+void HostZoomMapImpl::SetZoomLevel(std::string host, double level) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
{
@@ -80,8 +89,16 @@
content::Details<const std::string>(&host));
}
-double HostZoomMap::GetTemporaryZoomLevel(int render_process_id,
- int render_view_id) const {
+double HostZoomMapImpl::GetDefaultZoomLevel() const {
+ return default_zoom_level_;
+}
+
+void HostZoomMapImpl::SetDefaultZoomLevel(double level) {
+ default_zoom_level_ = level;
+}
+
+double HostZoomMapImpl::GetTemporaryZoomLevel(int render_process_id,
+ int render_view_id) const {
base::AutoLock auto_lock(lock_);
for (size_t i = 0; i < temporary_zoom_levels_.size(); ++i) {
if (temporary_zoom_levels_[i].render_process_id == render_process_id &&
@@ -92,9 +109,9 @@
return 0;
}
-void HostZoomMap::SetTemporaryZoomLevel(int render_process_id,
- int render_view_id,
- double level) {
+void HostZoomMapImpl::SetTemporaryZoomLevel(int render_process_id,
+ int render_view_id,
+ double level) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
{
@@ -128,7 +145,7 @@
content::Details<const std::string>(&host));
}
-void HostZoomMap::Observe(
+void HostZoomMapImpl::Observe(
int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) {
@@ -156,5 +173,5 @@
}
}
-HostZoomMap::~HostZoomMap() {
+HostZoomMapImpl::~HostZoomMapImpl() {
}
« no previous file with comments | « content/browser/host_zoom_map_impl.h ('k') | content/browser/host_zoom_map_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698