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

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: 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 =
37 components::UserPrefs::GetInstance()->Lookup(browser_context_);
36 if (!prefs) 38 if (!prefs)
37 return nodes; 39 return nodes;
38 40
39 const ListValue* value = prefs->GetList(prefs::kBookmarkEditorExpandedNodes); 41 const ListValue* value = prefs->GetList(prefs::kBookmarkEditorExpandedNodes);
40 if (!value) 42 if (!value)
41 return nodes; 43 return nodes;
42 44
43 bool changed = false; 45 bool changed = false;
44 for (ListValue::const_iterator i = value->begin(); i != value->end(); ++i) { 46 for (ListValue::const_iterator i = value->begin(); i != value->end(); ++i) {
45 std::string value; 47 std::string value;
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 int old_index, 83 int old_index,
82 const BookmarkNode* node) { 84 const BookmarkNode* node) {
83 if (!node->is_folder()) 85 if (!node->is_folder())
84 return; // Only care about folders. 86 return; // Only care about folders.
85 87
86 // Ask for the nodes again, which removes any nodes that were deleted. 88 // Ask for the nodes again, which removes any nodes that were deleted.
87 GetExpandedNodes(); 89 GetExpandedNodes();
88 } 90 }
89 91
90 void BookmarkExpandedStateTracker::UpdatePrefs(const Nodes& nodes) { 92 void BookmarkExpandedStateTracker::UpdatePrefs(const Nodes& nodes) {
91 PrefService* prefs = PrefServiceFromBrowserContext(browser_context_); 93 PrefService* prefs =
94 components::UserPrefs::GetInstance()->Lookup(browser_context_);
92 if (!prefs) 95 if (!prefs)
93 return; 96 return;
94 97
95 ListValue values; 98 ListValue values;
96 for (Nodes::const_iterator i = nodes.begin(); i != nodes.end(); ++i) { 99 for (Nodes::const_iterator i = nodes.begin(); i != nodes.end(); ++i) {
97 values.Set(values.GetSize(), 100 values.Set(values.GetSize(),
98 new StringValue(base::Int64ToString((*i)->id()))); 101 new StringValue(base::Int64ToString((*i)->id())));
99 } 102 }
100 103
101 prefs->Set(prefs::kBookmarkEditorExpandedNodes, values); 104 prefs->Set(prefs::kBookmarkEditorExpandedNodes, values);
102 } 105 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698