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

Unified Diff: components/enhanced_bookmarks/test_image_store.cc

Issue 288573003: Change ImageStore to use SequenceChecker instead of ThreadChecker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: git log Created 6 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/enhanced_bookmarks/persistent_image_store.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/enhanced_bookmarks/test_image_store.cc
diff --git a/components/enhanced_bookmarks/test_image_store.cc b/components/enhanced_bookmarks/test_image_store.cc
index dacb7c928c2d78ca79a1dfe1d41264633426df74..c96f3cfcb75c5bb014c2c05b39eaab9edc3b299b 100644
--- a/components/enhanced_bookmarks/test_image_store.cc
+++ b/components/enhanced_bookmarks/test_image_store.cc
@@ -11,7 +11,7 @@ TestImageStore::TestImageStore() {
}
bool TestImageStore::HasKey(const GURL& page_url) {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
return store_.find(page_url) != store_.end();
}
@@ -19,7 +19,7 @@ bool TestImageStore::HasKey(const GURL& page_url) {
void TestImageStore::Insert(const GURL& page_url,
const GURL& image_url,
const gfx::Image& image) {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
Erase(page_url);
store_.insert(std::make_pair(
@@ -29,13 +29,13 @@ void TestImageStore::Insert(const GURL& page_url,
}
void TestImageStore::Erase(const GURL& page_url) {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
store_.erase(page_url);
}
std::pair<gfx::Image, GURL> TestImageStore::Get(const GURL& page_url) {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
ImageMap::const_iterator pair_enumerator = store_.find(page_url);
if (pair_enumerator == store_.end())
@@ -45,7 +45,7 @@ std::pair<gfx::Image, GURL> TestImageStore::Get(const GURL& page_url) {
}
gfx::Size TestImageStore::GetSize(const GURL& page_url) {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
ImageMap::const_iterator pair_enumerator = store_.find(page_url);
if (pair_enumerator == store_.end())
@@ -55,7 +55,7 @@ gfx::Size TestImageStore::GetSize(const GURL& page_url) {
}
void TestImageStore::GetAllPageUrls(std::set<GURL>* urls) {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
DCHECK(urls->empty());
for (ImageMap::const_iterator it = store_.begin(); it != store_.end(); ++it)
@@ -63,11 +63,11 @@ void TestImageStore::GetAllPageUrls(std::set<GURL>* urls) {
}
void TestImageStore::ClearAll() {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
store_.clear();
}
TestImageStore::~TestImageStore() {
- DCHECK(thread_checker_.CalledOnValidThread());
+ DCHECK(sequence_checker_.CalledOnValidSequencedThread());
}
« no previous file with comments | « components/enhanced_bookmarks/persistent_image_store.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698