Index: chrome/browser/extensions/api/downloads/downloads_api.cc |
diff --git a/chrome/browser/extensions/api/downloads/downloads_api.cc b/chrome/browser/extensions/api/downloads/downloads_api.cc |
index a96091c1929fe5d7eb6268269e7726f52fdcbe03..209ffb34bae8d5df3e21b52b7eb0361f06185146 100644 |
--- a/chrome/browser/extensions/api/downloads/downloads_api.cc |
+++ b/chrome/browser/extensions/api/downloads/downloads_api.cc |
@@ -334,7 +334,7 @@ bool DownloadFileIconExtractorImpl::ExtractIconURLForPath( |
void DownloadFileIconExtractorImpl::OnIconLoadComplete( |
float scale, const IconURLCallback& callback, gfx::Image* icon) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
callback.Run(!icon ? std::string() : webui::GetBitmapDataUrl( |
icon->ToImageSkia()->GetRepresentation(scale).sk_bitmap())); |
} |
@@ -600,7 +600,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
downloads::FILENAME_CONFLICT_ACTION_UNIQUIFY), |
determined_conflict_action_( |
downloads::FILENAME_CONFLICT_ACTION_UNIQUIFY) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
download_item->SetUserData(kKey, this); |
} |
@@ -622,7 +622,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
void set_filename_change_callbacks( |
const base::Closure& no_change, |
const ExtensionDownloadsEventRouter::FilenameChangedCallback& change) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
filename_no_change_ = no_change; |
filename_change_ = change; |
determined_filename_ = creator_suggested_filename_; |
@@ -632,7 +632,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
} |
void ClearPendingDeterminers() { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
determined_filename_.clear(); |
determined_conflict_action_ = |
downloads::FILENAME_CONFLICT_ACTION_UNIQUIFY; |
@@ -644,7 +644,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
} |
void DeterminerRemoved(const std::string& extension_id) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
for (DeterminerInfoVector::iterator iter = determiners_.begin(); |
iter != determiners_.end();) { |
if (iter->extension_id == extension_id) { |
@@ -660,7 +660,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
void AddPendingDeterminer(const std::string& extension_id, |
const base::Time& installed) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
for (size_t index = 0; index < determiners_.size(); ++index) { |
if (determiners_[index].extension_id == extension_id) { |
DCHECK(false) << extension_id; |
@@ -671,7 +671,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
} |
bool DeterminerAlreadyReported(const std::string& extension_id) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
for (size_t index = 0; index < determiners_.size(); ++index) { |
if (determiners_[index].extension_id == extension_id) { |
return determiners_[index].reported; |
@@ -683,7 +683,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
void CreatorSuggestedFilename( |
const base::FilePath& filename, |
downloads::FilenameConflictAction conflict_action) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
creator_suggested_filename_ = filename; |
creator_conflict_action_ = conflict_action; |
} |
@@ -698,7 +698,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
} |
void ResetCreatorSuggestion() { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
creator_suggested_filename_.clear(); |
creator_conflict_action_ = |
downloads::FILENAME_CONFLICT_ACTION_UNIQUIFY; |
@@ -712,7 +712,7 @@ class ExtensionDownloadsEventRouterData : public base::SupportsUserData::Data { |
const std::string& extension_id, |
const base::FilePath& filename, |
downloads::FilenameConflictAction conflict_action) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
bool found_info = false; |
for (size_t index = 0; index < determiners_.size(); ++index) { |
if (determiners_[index].extension_id == extension_id) { |
@@ -1043,7 +1043,7 @@ void DownloadsDownloadFunction::OnStarted( |
downloads::FilenameConflictAction creator_conflict_action, |
DownloadItem* item, |
content::DownloadInterruptReason interrupt_reason) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
VLOG(1) << __FUNCTION__ << " " << item << " " << interrupt_reason; |
if (item) { |
DCHECK_EQ(content::DOWNLOAD_INTERRUPT_REASON_NONE, interrupt_reason); |
@@ -1226,7 +1226,7 @@ bool DownloadsRemoveFileFunction::RunImpl() { |
} |
void DownloadsRemoveFileFunction::Done(bool success) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
if (!success) { |
error_ = errors::kFileNotRemoved; |
} |
@@ -1293,7 +1293,7 @@ void DownloadsAcceptDangerFunction::PromptOrWait(int download_id, int retries) { |
void DownloadsAcceptDangerFunction::DangerPromptCallback( |
int download_id, DownloadDangerPrompt::Action action) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
DownloadItem* download_item = |
GetDownload(GetProfile(), include_incognito(), download_id); |
if (InvalidId(download_item, &error_) || |
@@ -1502,7 +1502,7 @@ bool DownloadsGetFileIconFunction::RunImpl() { |
} |
void DownloadsGetFileIconFunction::OnIconURLExtracted(const std::string& url) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
if (Fault(url.empty(), errors::kIconNotFound, &error_)) { |
SendResponse(false); |
return; |
@@ -1517,7 +1517,7 @@ ExtensionDownloadsEventRouter::ExtensionDownloadsEventRouter( |
DownloadManager* manager) |
: profile_(profile), |
notifier_(manager, this) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
DCHECK(profile_); |
registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED, |
content::Source<Profile>(profile_)); |
@@ -1529,7 +1529,7 @@ ExtensionDownloadsEventRouter::ExtensionDownloadsEventRouter( |
} |
ExtensionDownloadsEventRouter::~ExtensionDownloadsEventRouter() { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
extensions::EventRouter* router = extensions::ExtensionSystem::Get(profile_)-> |
event_router(); |
if (router) |
@@ -1582,7 +1582,7 @@ void ExtensionDownloadsEventRouter::OnDeterminingFilename( |
const base::FilePath& suggested_path, |
const base::Closure& no_change, |
const FilenameChangedCallback& change) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
ExtensionDownloadsEventRouterData* data = |
ExtensionDownloadsEventRouterData::Get(item); |
if (!data) { |
@@ -1666,7 +1666,7 @@ bool ExtensionDownloadsEventRouter::DetermineFilename( |
const base::FilePath& const_filename, |
downloads::FilenameConflictAction conflict_action, |
std::string* error) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
DownloadItem* item = GetDownload(profile, include_incognito, download_id); |
ExtensionDownloadsEventRouterData* data = |
item ? ExtensionDownloadsEventRouterData::Get(item) : NULL; |
@@ -1702,7 +1702,7 @@ bool ExtensionDownloadsEventRouter::DetermineFilename( |
void ExtensionDownloadsEventRouter::OnListenerRemoved( |
const extensions::EventListenerInfo& details) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
DownloadManager* manager = notifier_.GetManager(); |
if (!manager) |
return; |
@@ -1744,7 +1744,7 @@ void ExtensionDownloadsEventRouter::OnListenerRemoved( |
void ExtensionDownloadsEventRouter::OnDownloadCreated( |
DownloadManager* manager, DownloadItem* download_item) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
if (download_item->IsTemporary()) |
return; |
@@ -1775,7 +1775,7 @@ void ExtensionDownloadsEventRouter::OnDownloadCreated( |
void ExtensionDownloadsEventRouter::OnDownloadUpdated( |
DownloadManager* manager, DownloadItem* download_item) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
extensions::EventRouter* router = extensions::ExtensionSystem::Get(profile_)-> |
event_router(); |
ExtensionDownloadsEventRouterData* data = |
@@ -1845,7 +1845,7 @@ void ExtensionDownloadsEventRouter::OnDownloadUpdated( |
void ExtensionDownloadsEventRouter::OnDownloadRemoved( |
DownloadManager* manager, DownloadItem* download_item) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
if (download_item->IsTemporary()) |
return; |
DispatchEvent(downloads::OnErased::kEventName, |
@@ -1860,7 +1860,7 @@ void ExtensionDownloadsEventRouter::DispatchEvent( |
bool include_incognito, |
const extensions::Event::WillDispatchCallback& will_dispatch_callback, |
base::Value* arg) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
if (!extensions::ExtensionSystem::Get(profile_)->event_router()) |
return; |
scoped_ptr<base::ListValue> args(new base::ListValue()); |
@@ -1894,7 +1894,7 @@ void ExtensionDownloadsEventRouter::Observe( |
int type, |
const content::NotificationSource& source, |
const content::NotificationDetails& details) { |
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
+ DCHECK_CURRENTLY_ON(BrowserThread::UI); |
switch (type) { |
case chrome::NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED: { |
extensions::UnloadedExtensionInfo* unloaded = |