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

Unified Diff: chrome/browser/media_galleries/fileapi/itunes_data_provider.cc

Issue 2318133002: Remove call to IsRunningSequenceOnCurrentThread() in media_file_system_backend.cc (Closed)
Patch Set: Leaky LazyInstance Created 4 years, 3 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
Index: chrome/browser/media_galleries/fileapi/itunes_data_provider.cc
diff --git a/chrome/browser/media_galleries/fileapi/itunes_data_provider.cc b/chrome/browser/media_galleries/fileapi/itunes_data_provider.cc
index c2721112382a5b988bfc0a18f91d8ca9f68f7af7..7d7995cf55a0f2c90f4df27fa7bead05f6d9651a 100644
--- a/chrome/browser/media_galleries/fileapi/itunes_data_provider.cc
+++ b/chrome/browser/media_galleries/fileapi/itunes_data_provider.cc
@@ -199,14 +199,14 @@ const base::FilePath& ITunesDataProvider::auto_add_path() const {
}
bool ITunesDataProvider::KnownArtist(const ArtistName& artist) const {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
DCHECK(valid());
return base::ContainsKey(library_, artist);
}
bool ITunesDataProvider::KnownAlbum(const ArtistName& artist,
const AlbumName& album) const {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
DCHECK(valid());
Library::const_iterator library_it = library_.find(artist);
if (library_it == library_.end())
@@ -217,7 +217,7 @@ bool ITunesDataProvider::KnownAlbum(const ArtistName& artist,
base::FilePath ITunesDataProvider::GetTrackLocation(
const ArtistName& artist, const AlbumName& album,
const TrackName& track) const {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
DCHECK(valid());
Library::const_iterator library_it = library_.find(artist);
if (library_it == library_.end())
@@ -235,7 +235,7 @@ base::FilePath ITunesDataProvider::GetTrackLocation(
std::set<ITunesDataProvider::ArtistName>
ITunesDataProvider::GetArtistNames() const {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
DCHECK(valid());
std::set<ArtistName> result;
Library::const_iterator it;
@@ -247,7 +247,7 @@ ITunesDataProvider::GetArtistNames() const {
std::set<ITunesDataProvider::AlbumName> ITunesDataProvider::GetAlbumNames(
const ArtistName& artist) const {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
DCHECK(valid());
std::set<AlbumName> result;
Library::const_iterator artist_lookup = library_.find(artist);
@@ -264,7 +264,7 @@ std::set<ITunesDataProvider::AlbumName> ITunesDataProvider::GetAlbumNames(
ITunesDataProvider::Album ITunesDataProvider::GetAlbum(
const ArtistName& artist, const AlbumName& album) const {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
DCHECK(valid());
Album result;
Library::const_iterator artist_lookup = library_.find(artist);
@@ -279,7 +279,7 @@ ITunesDataProvider::Album ITunesDataProvider::GetAlbum(
void ITunesDataProvider::OnLibraryParsed(const ReadyCallback& ready_callback,
bool result,
const parser::Library& library) {
- DCHECK(MediaFileSystemBackend::CurrentlyOnMediaTaskRunnerThread());
+ MediaFileSystemBackend::AssertCurrentlyOnMediaSequence();
set_valid(result);
if (valid()) {
library_.clear();

Powered by Google App Engine
This is Rietveld 408576698