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

Side by Side Diff: chrome/browser/bookmarks/bookmark_expanded_state_tracker.cc

Issue 12340111: Introduce //components/user_prefs, use to eliminate c/b/prefs dependency in Autofill. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pure merge of LKGR Created 7 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/bookmarks/bookmark_expanded_state_tracker.h" 5 #include "chrome/browser/bookmarks/bookmark_expanded_state_tracker.h"
6 6
7 #include "base/prefs/pref_service.h" 7 #include "base/prefs/pref_service.h"
8 #include "base/strings/string_number_conversions.h" 8 #include "base/strings/string_number_conversions.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "chrome/browser/bookmarks/bookmark_model.h" 10 #include "chrome/browser/bookmarks/bookmark_model.h"
11 #include "chrome/browser/bookmarks/bookmark_model_factory.h" 11 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
12 #include "chrome/common/pref_names.h" 12 #include "chrome/common/pref_names.h"
13 #include "components/user_prefs/user_prefs.h"
13 14
14 BookmarkExpandedStateTracker::BookmarkExpandedStateTracker( 15 BookmarkExpandedStateTracker::BookmarkExpandedStateTracker(
15 content::BrowserContext* browser_context, 16 content::BrowserContext* browser_context,
16 BookmarkModel* bookmark_model) 17 BookmarkModel* bookmark_model)
17 : browser_context_(browser_context), 18 : browser_context_(browser_context),
18 bookmark_model_(bookmark_model) { 19 bookmark_model_(bookmark_model) {
19 bookmark_model->AddObserver(this); 20 bookmark_model->AddObserver(this);
20 } 21 }
21 22
22 BookmarkExpandedStateTracker::~BookmarkExpandedStateTracker() { 23 BookmarkExpandedStateTracker::~BookmarkExpandedStateTracker() {
23 } 24 }
24 25
25 void BookmarkExpandedStateTracker::SetExpandedNodes(const Nodes& nodes) { 26 void BookmarkExpandedStateTracker::SetExpandedNodes(const Nodes& nodes) {
26 UpdatePrefs(nodes); 27 UpdatePrefs(nodes);
27 } 28 }
28 29
29 BookmarkExpandedStateTracker::Nodes 30 BookmarkExpandedStateTracker::Nodes
30 BookmarkExpandedStateTracker::GetExpandedNodes() { 31 BookmarkExpandedStateTracker::GetExpandedNodes() {
31 Nodes nodes; 32 Nodes nodes;
32 if (!bookmark_model_->IsLoaded()) 33 if (!bookmark_model_->IsLoaded())
33 return nodes; 34 return nodes;
34 35
35 PrefService* prefs = PrefServiceFromBrowserContext(browser_context_); 36 PrefService* prefs = components::UserPrefs::Get(browser_context_);
36 if (!prefs) 37 if (!prefs)
37 return nodes; 38 return nodes;
38 39
39 const ListValue* value = prefs->GetList(prefs::kBookmarkEditorExpandedNodes); 40 const ListValue* value = prefs->GetList(prefs::kBookmarkEditorExpandedNodes);
40 if (!value) 41 if (!value)
41 return nodes; 42 return nodes;
42 43
43 bool changed = false; 44 bool changed = false;
44 for (ListValue::const_iterator i = value->begin(); i != value->end(); ++i) { 45 for (ListValue::const_iterator i = value->begin(); i != value->end(); ++i) {
45 std::string value; 46 std::string value;
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 int old_index, 82 int old_index,
82 const BookmarkNode* node) { 83 const BookmarkNode* node) {
83 if (!node->is_folder()) 84 if (!node->is_folder())
84 return; // Only care about folders. 85 return; // Only care about folders.
85 86
86 // Ask for the nodes again, which removes any nodes that were deleted. 87 // Ask for the nodes again, which removes any nodes that were deleted.
87 GetExpandedNodes(); 88 GetExpandedNodes();
88 } 89 }
89 90
90 void BookmarkExpandedStateTracker::UpdatePrefs(const Nodes& nodes) { 91 void BookmarkExpandedStateTracker::UpdatePrefs(const Nodes& nodes) {
91 PrefService* prefs = PrefServiceFromBrowserContext(browser_context_); 92 PrefService* prefs = components::UserPrefs::Get(browser_context_);
92 if (!prefs) 93 if (!prefs)
93 return; 94 return;
94 95
95 ListValue values; 96 ListValue values;
96 for (Nodes::const_iterator i = nodes.begin(); i != nodes.end(); ++i) { 97 for (Nodes::const_iterator i = nodes.begin(); i != nodes.end(); ++i) {
97 values.Set(values.GetSize(), 98 values.Set(values.GetSize(),
98 new StringValue(base::Int64ToString((*i)->id()))); 99 new StringValue(base::Int64ToString((*i)->id())));
99 } 100 }
100 101
101 prefs->Set(prefs::kBookmarkEditorExpandedNodes, values); 102 prefs->Set(prefs::kBookmarkEditorExpandedNodes, values);
102 } 103 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698