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

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

Issue 312093007: Revert of Added BookmarkClient::CanBeEditedByUser. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 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 "chrome/browser/bookmarks/chrome_bookmark_client.h" 5 #include "chrome/browser/bookmarks/chrome_bookmark_client.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/values.h" 10 #include "base/values.h"
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 // callback. 153 // callback.
154 managed_node_ = new BookmarkPermanentNode(0); 154 managed_node_ = new BookmarkPermanentNode(0);
155 return base::Bind( 155 return base::Bind(
156 &ChromeBookmarkClient::LoadExtraNodes, 156 &ChromeBookmarkClient::LoadExtraNodes,
157 StartupTaskRunnerServiceFactory::GetForProfile(profile_) 157 StartupTaskRunnerServiceFactory::GetForProfile(profile_)
158 ->GetBookmarkTaskRunner(), 158 ->GetBookmarkTaskRunner(),
159 managed_node_, 159 managed_node_,
160 base::Passed(managed_bookmarks_tracker_.GetInitialManagedBookmarks())); 160 base::Passed(managed_bookmarks_tracker_.GetInitialManagedBookmarks()));
161 } 161 }
162 162
163 bool ChromeBookmarkClient::CanRemovePermanentNodeChildren(
164 const BookmarkNode* node) {
165 return !IsDescendantOfManagedNode(node);
166 }
167
163 bool ChromeBookmarkClient::CanSetPermanentNodeTitle( 168 bool ChromeBookmarkClient::CanSetPermanentNodeTitle(
164 const BookmarkNode* permanent_node) { 169 const BookmarkNode* permanent_node) {
165 // The |managed_node_| can have its title updated if the user signs in or 170 // The |managed_node_| can have its title updated if the user signs in or
166 // out. 171 // out.
167 return !IsDescendantOfManagedNode(permanent_node) || 172 return !IsDescendantOfManagedNode(permanent_node) ||
168 permanent_node == managed_node_; 173 permanent_node == managed_node_;
169 } 174 }
170 175
171 bool ChromeBookmarkClient::CanSyncNode(const BookmarkNode* node) { 176 bool ChromeBookmarkClient::CanSyncNode(const BookmarkNode* node) {
172 return !IsDescendantOfManagedNode(node); 177 return !IsDescendantOfManagedNode(node);
173 } 178 }
174 179
175 bool ChromeBookmarkClient::CanBeEditedByUser(const BookmarkNode* node) { 180 bool ChromeBookmarkClient::CanReorderChildren(const BookmarkNode* parent) {
176 return !IsDescendantOfManagedNode(node); 181 return !IsDescendantOfManagedNode(parent);
177 } 182 }
178 183
179 void ChromeBookmarkClient::Observe( 184 void ChromeBookmarkClient::Observe(
180 int type, 185 int type,
181 const content::NotificationSource& source, 186 const content::NotificationSource& source,
182 const content::NotificationDetails& details) { 187 const content::NotificationDetails& details) {
183 switch (type) { 188 switch (type) {
184 case chrome::NOTIFICATION_FAVICON_CHANGED: { 189 case chrome::NOTIFICATION_FAVICON_CHANGED: {
185 content::Details<FaviconChangedDetails> favicon_details(details); 190 content::Details<FaviconChangedDetails> favicon_details(details);
186 model_->OnFaviconChanged(favicon_details->urls); 191 model_->OnFaviconChanged(favicon_details->urls);
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
245 return extra_nodes.Pass(); 250 return extra_nodes.Pass();
246 } 251 }
247 252
248 std::string ChromeBookmarkClient::GetManagedBookmarksDomain() { 253 std::string ChromeBookmarkClient::GetManagedBookmarksDomain() {
249 policy::ProfilePolicyConnector* connector = 254 policy::ProfilePolicyConnector* connector =
250 policy::ProfilePolicyConnectorFactory::GetForProfile(profile_); 255 policy::ProfilePolicyConnectorFactory::GetForProfile(profile_);
251 if (connector->IsPolicyFromCloudPolicy(policy::key::kManagedBookmarks)) 256 if (connector->IsPolicyFromCloudPolicy(policy::key::kManagedBookmarks))
252 return connector->GetManagementDomain(); 257 return connector->GetManagementDomain();
253 return std::string(); 258 return std::string();
254 } 259 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698