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

Side by Side Diff: components/policy/core/browser/managed_bookmarks_tracker_unittest.cc

Issue 684513002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "components/policy/core/browser/managed_bookmarks_tracker.h" 5 #include "components/policy/core/browser/managed_bookmarks_tracker.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
(...skipping 14 matching lines...) Expand all
25 #include "url/gurl.h" 25 #include "url/gurl.h"
26 26
27 using testing::Mock; 27 using testing::Mock;
28 using testing::_; 28 using testing::_;
29 29
30 namespace policy { 30 namespace policy {
31 31
32 class ManagedBookmarksTrackerTest : public testing::Test { 32 class ManagedBookmarksTrackerTest : public testing::Test {
33 public: 33 public:
34 ManagedBookmarksTrackerTest() : managed_node_(NULL) {} 34 ManagedBookmarksTrackerTest() : managed_node_(NULL) {}
35 virtual ~ManagedBookmarksTrackerTest() {} 35 ~ManagedBookmarksTrackerTest() override {}
36 36
37 virtual void SetUp() override { 37 void SetUp() override {
38 prefs_.registry()->RegisterListPref(bookmarks::prefs::kManagedBookmarks); 38 prefs_.registry()->RegisterListPref(bookmarks::prefs::kManagedBookmarks);
39 prefs_.registry()->RegisterListPref( 39 prefs_.registry()->RegisterListPref(
40 bookmarks::prefs::kBookmarkEditorExpandedNodes); 40 bookmarks::prefs::kBookmarkEditorExpandedNodes);
41 } 41 }
42 42
43 virtual void TearDown() override { 43 void TearDown() override {
44 if (model_) 44 if (model_)
45 model_->RemoveObserver(&observer_); 45 model_->RemoveObserver(&observer_);
46 } 46 }
47 47
48 void CreateModel() { 48 void CreateModel() {
49 // Simulate the creation of the managed node by the BookmarkClient. 49 // Simulate the creation of the managed node by the BookmarkClient.
50 BookmarkPermanentNode* managed_node = new BookmarkPermanentNode(100); 50 BookmarkPermanentNode* managed_node = new BookmarkPermanentNode(100);
51 policy::ManagedBookmarksTracker::LoadInitial( 51 policy::ManagedBookmarksTracker::LoadInitial(
52 managed_node, prefs_.GetList(bookmarks::prefs::kManagedBookmarks), 101); 52 managed_node, prefs_.GetList(bookmarks::prefs::kManagedBookmarks), 101);
53 managed_node->set_visible(!managed_node->empty()); 53 managed_node->set_visible(!managed_node->empty());
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
313 Mock::VerifyAndClearExpectations(&observer_); 313 Mock::VerifyAndClearExpectations(&observer_);
314 314
315 EXPECT_CALL(observer_, BookmarkAllUserNodesRemoved(model_.get(), _)); 315 EXPECT_CALL(observer_, BookmarkAllUserNodesRemoved(model_.get(), _));
316 model_->RemoveAllUserBookmarks(); 316 model_->RemoveAllUserBookmarks();
317 EXPECT_EQ(2, managed_node()->child_count()); 317 EXPECT_EQ(2, managed_node()->child_count());
318 EXPECT_EQ(0, model_->bookmark_bar_node()->child_count()); 318 EXPECT_EQ(0, model_->bookmark_bar_node()->child_count());
319 Mock::VerifyAndClearExpectations(&observer_); 319 Mock::VerifyAndClearExpectations(&observer_);
320 } 320 }
321 321
322 } // namespace policy 322 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698