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

Unified Diff: chrome/browser/signin/easy_unlock_screenlock_state_handler.cc

Issue 868133005: [Smart Lock] Add error message strings for the "Tx Power too high" case. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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: chrome/browser/signin/easy_unlock_screenlock_state_handler.cc
diff --git a/chrome/browser/signin/easy_unlock_screenlock_state_handler.cc b/chrome/browser/signin/easy_unlock_screenlock_state_handler.cc
index 463c1bc5f4150596eca2a7c0162155d3bac0200a..3a0296f6708f3d377a24653d2a08b42f555b4ca2 100644
--- a/chrome/browser/signin/easy_unlock_screenlock_state_handler.cc
+++ b/chrome/browser/signin/easy_unlock_screenlock_state_handler.cc
@@ -25,6 +25,10 @@ ScreenlockBridge::UserPodCustomIcon GetIconForState(
case EasyUnlockScreenlockStateHandler::STATE_RSSI_TOO_LOW:
return ScreenlockBridge::USER_POD_CUSTOM_ICON_LOCKED;
case EasyUnlockScreenlockStateHandler::STATE_TX_POWER_TOO_HIGH:
+ case EasyUnlockScreenlockStateHandler::
+ STATE_PHONE_LOCKED_AND_TX_POWER_TOO_HIGH:
+ // TODO(isherman): This icon is currently identical to the regular locked
+ // icon. Perhaps we should just delete the redundant icon?
Tim Song 2015/01/24 02:38:31 Let's wait until we're ready to remove the flag.
Ilya Sherman 2015/01/27 01:20:24 Updated the TODO text.
return ScreenlockBridge::USER_POD_CUSTOM_ICON_LOCKED_WITH_PROXIMITY_HINT;
case EasyUnlockScreenlockStateHandler::STATE_BLUETOOTH_CONNECTING:
return ScreenlockBridge::USER_POD_CUSTOM_ICON_SPINNER;
@@ -55,6 +59,10 @@ size_t GetTooltipResourceId(EasyUnlockScreenlockStateHandler::State state) {
return IDS_EASY_UNLOCK_SCREENLOCK_TOOLTIP_RSSI_TOO_LOW;
case EasyUnlockScreenlockStateHandler::STATE_TX_POWER_TOO_HIGH:
return IDS_EASY_UNLOCK_SCREENLOCK_TOOLTIP_TX_POWER_TOO_HIGH;
+ case EasyUnlockScreenlockStateHandler::
+ STATE_PHONE_LOCKED_AND_TX_POWER_TOO_HIGH:
+ return
+ IDS_EASY_UNLOCK_SCREENLOCK_TOOLTIP_PHONE_LOCKED_AND_TX_POWER_TOO_HIGH;
case EasyUnlockScreenlockStateHandler::STATE_AUTHENTICATED:
return IDS_EASY_UNLOCK_SCREENLOCK_TOOLTIP_HARDLOCK_INSTRUCTIONS;
case EasyUnlockScreenlockStateHandler::STATE_PHONE_UNSUPPORTED:
@@ -69,8 +77,9 @@ bool TooltipContainsDeviceType(EasyUnlockScreenlockStateHandler::State state) {
state == EasyUnlockScreenlockStateHandler::STATE_PHONE_UNLOCKABLE ||
state == EasyUnlockScreenlockStateHandler::STATE_NO_BLUETOOTH ||
state == EasyUnlockScreenlockStateHandler::STATE_PHONE_UNSUPPORTED ||
- state == EasyUnlockScreenlockStateHandler::STATE_RSSI_TOO_LOW ||
- state == EasyUnlockScreenlockStateHandler::STATE_TX_POWER_TOO_HIGH;
+ state == EasyUnlockScreenlockStateHandler::STATE_TX_POWER_TOO_HIGH ||
+ state == EasyUnlockScreenlockStateHandler::
+ STATE_PHONE_LOCKED_AND_TX_POWER_TOO_HIGH;
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698