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

Unified Diff: chrome/browser/automation/testing_automation_provider_chromeos.cc

Issue 21015010: Remove all traces of deprecated Get/SetReleaseTrack calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 5 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/automation/testing_automation_provider_chromeos.cc
diff --git a/chrome/browser/automation/testing_automation_provider_chromeos.cc b/chrome/browser/automation/testing_automation_provider_chromeos.cc
index 61833959ff93b9e393cc8c1037c43afb382afe1c..be3f3076f57b76d25454492c96db50fb25617e07 100644
--- a/chrome/browser/automation/testing_automation_provider_chromeos.cc
+++ b/chrome/browser/automation/testing_automation_provider_chromeos.cc
@@ -103,33 +103,6 @@ const char* UpdateStatusToString(
}
}
-void GetReleaseTrackCallback(AutomationJSONReply* reply,
- const std::string& track) {
- if (track.empty()) {
- reply->SendError("Unable to get release track.");
- delete reply;
- return;
- }
-
- scoped_ptr<DictionaryValue> return_value(new DictionaryValue);
- return_value->SetString("release_track", track);
-
- const UpdateEngineClient::Status& status =
- DBusThreadManager::Get()->GetUpdateEngineClient()->GetLastStatus();
- UpdateEngineClient::UpdateStatusOperation update_status =
- status.status;
- return_value->SetString("status", UpdateStatusToString(update_status));
- if (update_status == UpdateEngineClient::UPDATE_STATUS_DOWNLOADING)
- return_value->SetDouble("download_progress", status.download_progress);
- if (status.last_checked_time > 0)
- return_value->SetInteger("last_checked_time", status.last_checked_time);
- if (status.new_size > 0)
- return_value->SetInteger("new_size", status.new_size);
-
- reply->SendSuccess(return_value.get());
- delete reply;
-}
-
void UpdateCheckCallback(AutomationJSONReply* reply,
UpdateEngineClient::UpdateCheckResult result) {
if (result == UpdateEngineClient::UPDATE_RESULT_SUCCESS)
@@ -1154,13 +1127,6 @@ void TestingAutomationProvider::SetTimezone(DictionaryValue* args,
reply.SendSuccess(NULL);
}
-void TestingAutomationProvider::GetUpdateInfo(DictionaryValue* args,
- IPC::Message* reply_message) {
- AutomationJSONReply* reply = new AutomationJSONReply(this, reply_message);
- DBusThreadManager::Get()->GetUpdateEngineClient()
- ->GetReleaseTrack(base::Bind(GetReleaseTrackCallback, reply));
-}
-
void TestingAutomationProvider::UpdateCheck(
DictionaryValue* args,
IPC::Message* reply_message) {
@@ -1169,19 +1135,6 @@ void TestingAutomationProvider::UpdateCheck(
->RequestUpdateCheck(base::Bind(UpdateCheckCallback, reply));
}
-void TestingAutomationProvider::SetReleaseTrack(DictionaryValue* args,
- IPC::Message* reply_message) {
- AutomationJSONReply reply(this, reply_message);
- std::string track;
- if (!args->GetString("track", &track)) {
- reply.SendError("Invalid or missing args.");
- return;
- }
-
- DBusThreadManager::Get()->GetUpdateEngineClient()->SetReleaseTrack(track);
- reply.SendSuccess(NULL);
-}
-
void TestingAutomationProvider::GetVolumeInfo(DictionaryValue* args,
IPC::Message* reply_message) {
AutomationJSONReply reply(this, reply_message);

Powered by Google App Engine
This is Rietveld 408576698