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

Side by Side Diff: chrome/browser/extensions/api/declarative_content/declarative_content_is_bookmarked_condition_tracker.cc

Issue 2230203002: chrome: Use stl utilities from the base namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed accidental components/ change Created 4 years, 4 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/extensions/api/declarative_content/declarative_content_ is_bookmarked_condition_tracker.h" 5 #include "chrome/browser/extensions/api/declarative_content/declarative_content_ is_bookmarked_condition_tracker.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #include "base/stl_util.h" 9 #include "base/stl_util.h"
10 #include "base/strings/stringprintf.h" 10 #include "base/strings/stringprintf.h"
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 void DeclarativeContentIsBookmarkedConditionTracker::PerWebContentsTracker:: 103 void DeclarativeContentIsBookmarkedConditionTracker::PerWebContentsTracker::
104 BookmarkAddedForUrl(const GURL& url) { 104 BookmarkAddedForUrl(const GURL& url) {
105 if (web_contents()->GetVisibleURL() == url) { 105 if (web_contents()->GetVisibleURL() == url) {
106 is_url_bookmarked_ = true; 106 is_url_bookmarked_ = true;
107 request_evaluation_.Run(web_contents()); 107 request_evaluation_.Run(web_contents());
108 } 108 }
109 } 109 }
110 110
111 void DeclarativeContentIsBookmarkedConditionTracker::PerWebContentsTracker:: 111 void DeclarativeContentIsBookmarkedConditionTracker::PerWebContentsTracker::
112 BookmarkRemovedForUrls(const std::set<GURL>& urls) { 112 BookmarkRemovedForUrls(const std::set<GURL>& urls) {
113 if (ContainsKey(urls, web_contents()->GetVisibleURL())) { 113 if (base::ContainsKey(urls, web_contents()->GetVisibleURL())) {
114 is_url_bookmarked_ = false; 114 is_url_bookmarked_ = false;
115 request_evaluation_.Run(web_contents()); 115 request_evaluation_.Run(web_contents());
116 } 116 }
117 } 117 }
118 118
119 void DeclarativeContentIsBookmarkedConditionTracker::PerWebContentsTracker:: 119 void DeclarativeContentIsBookmarkedConditionTracker::PerWebContentsTracker::
120 UpdateState(bool request_evaluation_if_unchanged) { 120 UpdateState(bool request_evaluation_if_unchanged) {
121 bool state_changed = 121 bool state_changed =
122 IsCurrentUrlBookmarked() != is_url_bookmarked_; 122 IsCurrentUrlBookmarked() != is_url_bookmarked_;
123 if (state_changed) 123 if (state_changed)
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
195 base::Bind(&Delegate::RequestEvaluation, base::Unretained(delegate_)), 195 base::Bind(&Delegate::RequestEvaluation, base::Unretained(delegate_)),
196 base::Bind(&DeclarativeContentIsBookmarkedConditionTracker:: 196 base::Bind(&DeclarativeContentIsBookmarkedConditionTracker::
197 DeletePerWebContentsTracker, 197 DeletePerWebContentsTracker,
198 base::Unretained(this)))); 198 base::Unretained(this))));
199 } 199 }
200 200
201 void DeclarativeContentIsBookmarkedConditionTracker::OnWebContentsNavigation( 201 void DeclarativeContentIsBookmarkedConditionTracker::OnWebContentsNavigation(
202 content::WebContents* contents, 202 content::WebContents* contents,
203 const content::LoadCommittedDetails& details, 203 const content::LoadCommittedDetails& details,
204 const content::FrameNavigateParams& params) { 204 const content::FrameNavigateParams& params) {
205 DCHECK(ContainsKey(per_web_contents_tracker_, contents)); 205 DCHECK(base::ContainsKey(per_web_contents_tracker_, contents));
206 per_web_contents_tracker_[contents]->UpdateState(true); 206 per_web_contents_tracker_[contents]->UpdateState(true);
207 } 207 }
208 208
209 bool DeclarativeContentIsBookmarkedConditionTracker::EvaluatePredicate( 209 bool DeclarativeContentIsBookmarkedConditionTracker::EvaluatePredicate(
210 const ContentPredicate* predicate, 210 const ContentPredicate* predicate,
211 content::WebContents* tab) const { 211 content::WebContents* tab) const {
212 DCHECK_EQ(this, predicate->GetEvaluator()); 212 DCHECK_EQ(this, predicate->GetEvaluator());
213 const DeclarativeContentIsBookmarkedPredicate* typed_predicate = 213 const DeclarativeContentIsBookmarkedPredicate* typed_predicate =
214 static_cast<const DeclarativeContentIsBookmarkedPredicate*>(predicate); 214 static_cast<const DeclarativeContentIsBookmarkedPredicate*>(predicate);
215 auto loc = per_web_contents_tracker_.find(tab); 215 auto loc = per_web_contents_tracker_.find(tab);
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
267 void 267 void
268 DeclarativeContentIsBookmarkedConditionTracker::GroupedBookmarkChangesEnded( 268 DeclarativeContentIsBookmarkedConditionTracker::GroupedBookmarkChangesEnded(
269 bookmarks::BookmarkModel* model) { 269 bookmarks::BookmarkModel* model) {
270 if (--extensive_bookmark_changes_in_progress_ == 0) 270 if (--extensive_bookmark_changes_in_progress_ == 0)
271 UpdateAllPerWebContentsTrackers(); 271 UpdateAllPerWebContentsTrackers();
272 } 272 }
273 273
274 void 274 void
275 DeclarativeContentIsBookmarkedConditionTracker::DeletePerWebContentsTracker( 275 DeclarativeContentIsBookmarkedConditionTracker::DeletePerWebContentsTracker(
276 content::WebContents* contents) { 276 content::WebContents* contents) {
277 DCHECK(ContainsKey(per_web_contents_tracker_, contents)); 277 DCHECK(base::ContainsKey(per_web_contents_tracker_, contents));
278 per_web_contents_tracker_.erase(contents); 278 per_web_contents_tracker_.erase(contents);
279 } 279 }
280 280
281 void DeclarativeContentIsBookmarkedConditionTracker:: 281 void DeclarativeContentIsBookmarkedConditionTracker::
282 UpdateAllPerWebContentsTrackers() { 282 UpdateAllPerWebContentsTrackers() {
283 for (const auto& web_contents_tracker_pair : per_web_contents_tracker_) 283 for (const auto& web_contents_tracker_pair : per_web_contents_tracker_)
284 web_contents_tracker_pair.second->UpdateState(false); 284 web_contents_tracker_pair.second->UpdateState(false);
285 } 285 }
286 286
287 } // namespace extensions 287 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698