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

Unified Diff: extensions/browser/api/system_display/system_display_api.cc

Issue 2638903003: Updates touch calibration API and plumbs through the native touch calibration method (Closed)
Patch Set: Resolving Comments Created 3 years, 11 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: extensions/browser/api/system_display/system_display_api.cc
diff --git a/extensions/browser/api/system_display/system_display_api.cc b/extensions/browser/api/system_display/system_display_api.cc
index 1b7596586587675fd2bea9c29f4e3bb3777ed9a7..9ce568714a3e6f4c52aae5b3a807d0a6b868cb14 100644
--- a/extensions/browser/api/system_display/system_display_api.cc
+++ b/extensions/browser/api/system_display/system_display_api.cc
@@ -267,45 +267,70 @@ SystemDisplayOverscanCalibrationCompleteFunction::Run() {
}
ExtensionFunction::ResponseAction
-SystemDisplayTouchCalibrationStartFunction::Run() {
- std::unique_ptr<display::TouchCalibrationStart::Params> params(
- display::TouchCalibrationStart::Params::Create(*args_));
+SystemDisplayShowNativeTouchCalibrationFunction::Run() {
+ std::unique_ptr<display::ShowNativeTouchCalibration::Params> params(
+ display::ShowNativeTouchCalibration::Params::Create(*args_));
std::string error;
- if (DisplayInfoProvider::Get()->IsTouchCalibrationActive(&error))
+ if (DisplayInfoProvider::Get()->IsNativeTouchCalibrationActive(&error))
return RespondNow(Error(error));
- if (!DisplayInfoProvider::Get()->TouchCalibrationStart(params->id))
- return RespondNow(Error("Invalid display ID: " + params->id));
+ if (!DisplayInfoProvider::Get()->ShowNativeTouchCalibration(
+ params->id, &error,
+ base::Bind(&SystemDisplayShowNativeTouchCalibrationFunction::
+ OnCalibrationComplete,
+ this))) {
+ return RespondNow(Error(error));
+ }
+ return RespondLater();
+}
+
+void SystemDisplayShowNativeTouchCalibrationFunction::OnCalibrationComplete(
+ bool success) {
+ Respond(OneArgument(base::MakeUnique<base::FundamentalValue>(success)));
+}
+
+ExtensionFunction::ResponseAction
+SystemDisplayStartCustomTouchCalibrationFunction::Run() {
+ std::unique_ptr<display::StartCustomTouchCalibration::Params> params(
+ display::StartCustomTouchCalibration::Params::Create(*args_));
+
+ std::string error;
+ if (DisplayInfoProvider::Get()->IsNativeTouchCalibrationActive(&error))
+ return RespondNow(Error(error));
+
+ if (!DisplayInfoProvider::Get()->StartCustomTouchCalibration(params->id,
+ &error))
+ return RespondNow(Error(error));
return RespondNow(NoArguments());
}
ExtensionFunction::ResponseAction
-SystemDisplayTouchCalibrationSetFunction::Run() {
- std::unique_ptr<display::TouchCalibrationSet::Params> params(
- display::TouchCalibrationSet::Params::Create(*args_));
+SystemDisplayCompleteCustomTouchCalibrationFunction::Run() {
+ std::unique_ptr<display::CompleteCustomTouchCalibration::Params> params(
+ display::CompleteCustomTouchCalibration::Params::Create(*args_));
std::string error;
- if (DisplayInfoProvider::Get()->IsTouchCalibrationActive(&error))
+ if (DisplayInfoProvider::Get()->IsNativeTouchCalibrationActive(&error))
return RespondNow(Error(error));
- if (!DisplayInfoProvider::Get()->TouchCalibrationSet(
- params->id, params->pairs, params->bounds, &error)) {
+ if (!DisplayInfoProvider::Get()->CompleteCustomTouchCalibration(
+ params->pairs, params->bounds, &error)) {
return RespondNow(Error(error));
}
return RespondNow(NoArguments());
}
ExtensionFunction::ResponseAction
-SystemDisplayTouchCalibrationResetFunction::Run() {
- std::unique_ptr<display::TouchCalibrationReset::Params> params(
- display::TouchCalibrationReset::Params::Create(*args_));
+SystemDisplayResetTouchCalibrationFunction::Run() {
+ std::unique_ptr<display::ResetTouchCalibration::Params> params(
+ display::ResetTouchCalibration::Params::Create(*args_));
std::string error;
- if (DisplayInfoProvider::Get()->IsTouchCalibrationActive(&error))
+ if (DisplayInfoProvider::Get()->IsNativeTouchCalibrationActive(&error))
return RespondNow(Error(error));
- if (!DisplayInfoProvider::Get()->TouchCalibrationReset(params->id, &error))
+ if (!DisplayInfoProvider::Get()->ResetTouchCalibration(params->id, &error))
return RespondNow(Error(error));
return RespondNow(NoArguments());
}

Powered by Google App Engine
This is Rietveld 408576698