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

Unified Diff: content/browser/host_zoom_map_unittest.cc

Issue 7058003: Take out pref watching from HostZoomMap, in preparation of removing Profile from it. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 7 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.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/host_zoom_map_unittest.cc
===================================================================
--- content/browser/host_zoom_map_unittest.cc (revision 86148)
+++ content/browser/host_zoom_map_unittest.cc (working copy)
@@ -1,120 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/memory/ref_counted.h"
-#include "base/message_loop.h"
-#include "base/utf_string_conversions.h"
-#include "base/values.h"
-#include "chrome/browser/prefs/pref_service.h"
-#include "chrome/browser/prefs/scoped_user_pref_update.h"
-#include "chrome/common/pref_names.h"
-#include "chrome/test/testing_profile.h"
-#include "content/browser/browser_thread.h"
-#include "content/browser/host_zoom_map.h"
-#include "content/common/notification_details.h"
-#include "content/common/notification_observer_mock.h"
-#include "content/common/notification_source.h"
-#include "content/common/notification_type.h"
-#include "googleurl/src/gurl.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-using testing::_;
-using testing::Pointee;
-using testing::Property;
-
-class HostZoomMapTest : public testing::Test {
- public:
- static const double kZoomLevel;
- static const double kDefaultZoomLevel;
- HostZoomMapTest()
- : ui_thread_(BrowserThread::UI, &message_loop_),
- prefs_(profile_.GetPrefs()),
- per_host_zoom_levels_pref_(prefs::kPerHostZoomLevels),
- url_("http://example.com/test"),
- host_("example.com") {}
-
- protected:
- void SetPrefObserverExpectation() {
- EXPECT_CALL(
- pref_observer_,
- Observe(NotificationType(NotificationType::PREF_CHANGED),
- _,
- Property(&Details<std::string>::ptr,
- Pointee(per_host_zoom_levels_pref_))));
- }
-
- MessageLoopForUI message_loop_;
- BrowserThread ui_thread_;
- TestingProfile profile_;
- PrefService* prefs_;
- std::string per_host_zoom_levels_pref_; // For the observe matcher.
- GURL url_;
- std::string host_;
- NotificationObserverMock pref_observer_;
-};
-const double HostZoomMapTest::kZoomLevel = 4;
-const double HostZoomMapTest::kDefaultZoomLevel = -2;
-
-TEST_F(HostZoomMapTest, LoadNoPrefs) {
- scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_));
- EXPECT_EQ(0, map->GetZoomLevel(url_));
-}
-
-TEST_F(HostZoomMapTest, Load) {
- {
- DictionaryPrefUpdate update(prefs_, prefs::kPerHostZoomLevels);
- DictionaryValue* dict = update.Get();
- dict->SetWithoutPathExpansion(host_, Value::CreateDoubleValue(kZoomLevel));
- }
- scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_));
- EXPECT_EQ(kZoomLevel, map->GetZoomLevel(url_));
-}
-
-TEST_F(HostZoomMapTest, SetZoomLevel) {
- scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_));
- PrefChangeRegistrar registrar;
- registrar.Init(prefs_);
- registrar.Add(prefs::kPerHostZoomLevels, &pref_observer_);
- SetPrefObserverExpectation();
- map->SetZoomLevel(url_, kZoomLevel);
- EXPECT_EQ(kZoomLevel, map->GetZoomLevel(url_));
- const DictionaryValue* dict =
- prefs_->GetDictionary(prefs::kPerHostZoomLevels);
- double zoom_level = 0;
- EXPECT_TRUE(dict->GetDoubleWithoutPathExpansion(host_, &zoom_level));
- EXPECT_EQ(kZoomLevel, zoom_level);
-
- SetPrefObserverExpectation();
- map->SetZoomLevel(url_, 0);
- EXPECT_EQ(0, map->GetZoomLevel(url_));
- EXPECT_FALSE(dict->HasKey(host_));
-}
-
-TEST_F(HostZoomMapTest, ReloadOnPrefChange) {
- scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_));
- map->SetZoomLevel(url_, kZoomLevel);
-
- DictionaryValue dict;
- dict.SetWithoutPathExpansion(host_, Value::CreateDoubleValue(0));
- prefs_->Set(prefs::kPerHostZoomLevels, dict);
- EXPECT_EQ(0, map->GetZoomLevel(url_));
-}
-
-TEST_F(HostZoomMapTest, NoHost) {
- scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_));
- GURL file_url1_("file:///tmp/test.html");
- GURL file_url2_("file:///tmp/other.html");
- map->SetZoomLevel(file_url1_, kZoomLevel);
-
- EXPECT_EQ(kZoomLevel, map->GetZoomLevel(file_url1_));
- EXPECT_EQ(0, map->GetZoomLevel(file_url2_));
-}
-
-TEST_F(HostZoomMapTest, ChangeDefaultZoomLevel) {
- FundamentalValue zoom_level(kDefaultZoomLevel);
- prefs_->Set(prefs::kDefaultZoomLevel, zoom_level);
- scoped_refptr<HostZoomMap> map(new HostZoomMap(&profile_));
- EXPECT_EQ(kDefaultZoomLevel, map->GetZoomLevel(url_));
-}
« no previous file with comments | « content/browser/host_zoom_map.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698