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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/host_zoom_map.cc

Issue 8528011: Page zoom improvements (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 1 month 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
« chrome/browser/ui/webui/options/advanced_options_handler.cc ('K') | « chrome/browser/ui/webui/options/advanced_options_handler.cc ('k') | content/browser/tab_contents/tab_contents.cc » ('j') | content/browser/tab_contents/tab_contents.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/host_zoom_map.cc
===================================================================
--- content/browser/host_zoom_map.cc (revision 109526)
+++ content/browser/host_zoom_map.cc (working copy)
@@ -14,6 +14,7 @@
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/notification_types.h"
+#include "content/public/common/page_zoom.h"
#include "googleurl/src/gurl.h"
#include "net/base/net_util.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/WebView.h"
@@ -56,7 +57,8 @@
{
base::AutoLock auto_lock(lock_);
- if (level == default_zoom_level_)
+
+ if (std::fabs(level - default_zoom_level_) <= content::kPageZoomEpsilon)
James Hawkins 2011/11/14 18:09:05 I'm seeing this a lot; can we refactor this somewh
csilv 2011/11/15 02:26:49 Done.
host_zoom_levels_.erase(host);
else
host_zoom_levels_[host] = level;
« chrome/browser/ui/webui/options/advanced_options_handler.cc ('K') | « chrome/browser/ui/webui/options/advanced_options_handler.cc ('k') | content/browser/tab_contents/tab_contents.cc » ('j') | content/browser/tab_contents/tab_contents.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698