Skip to content

Commit

Permalink
Fix unreplied automation messages that trigger in Debug.
Browse files Browse the repository at this point in the history
BUG=
TEST=downloads.py runs fine in Debug

Review URL: http://codereview.chromium.org/5730005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69202 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
nirnimesh@chromium.org committed Dec 15, 2010
1 parent 1946c93 commit 56b351d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions chrome/browser/automation/testing_automation_provider.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2598,17 +2598,17 @@ void TestingAutomationProvider::WaitForDownloadsToComplete(
Browser* browser,
DictionaryValue* args,
IPC::Message* reply_message) {
AutomationJSONReply reply(this, reply_message);

// Look for a quick return.
if (!profile_->HasCreatedDownloadManager()) {
reply.SendSuccess(NULL); // No download manager.
// No download manager.
AutomationJSONReply(this, reply_message).SendSuccess(NULL);
return;
}
std::vector<DownloadItem*> downloads;
profile_->GetDownloadManager()->GetCurrentDownloads(FilePath(), &downloads);
if (downloads.empty()) {
reply.SendSuccess(NULL);
AutomationJSONReply(this, reply_message).SendSuccess(NULL);
return;
}

Expand Down

0 comments on commit 56b351d

Please sign in to comment.