summaryrefslogtreecommitdiffstats
path: root/remoting/resources
diff options
context:
space:
mode:
authorsolb@chromium.org <solb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-08-09 03:56:44 +0000
committersolb@chromium.org <solb@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-08-09 03:56:44 +0000
commit8895ea754b03ca88286805228dd2861346320254 (patch)
tree3fec2e40c96d352e678eb02e7164f3f128be4ed4 /remoting/resources
parentec640116870dbd130b83c7739ba7691714038d55 (diff)
downloadchromium_src-8895ea754b03ca88286805228dd2861346320254.zip
chromium_src-8895ea754b03ca88286805228dd2861346320254.tar.gz
chromium_src-8895ea754b03ca88286805228dd2861346320254.tar.bz2
Resolve several Chromoting Android UI-related crashes and misbehaviors
This protects several places where the UI once allowed users to hit dangerous edge cases: - Canceling the auth prompt on the app's first run now leaves the switcher's label text unset. - Launching the app on a device with no usable accounts once again offers to add a new account. - Trying to authenticate to an account for the first time while disconnected is detected as a network problem. - The account switcher is made unavailable when the only usable account is already selected. - There's a new refresh button for updating the currently-displayed host list from the directory server. - Pressing a device's extra/exotic buttons with high keycodes no longer DCHECKS to crash the app. - Many third-party keyboards that stubbornly covered the bottom of the remote desktop image no longer do so. - All native threads are actually properly detached from the JVM. BUG=259594 Review URL: https://chromiumcodereview.appspot.com/22662003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216556 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/resources')
-rw-r--r--remoting/resources/menu/chromoting_actionbar.xml4
-rw-r--r--remoting/resources/strings.xml2
2 files changed, 6 insertions, 0 deletions
diff --git a/remoting/resources/menu/chromoting_actionbar.xml b/remoting/resources/menu/chromoting_actionbar.xml
index 1507dd4..e602a36 100644
--- a/remoting/resources/menu/chromoting_actionbar.xml
+++ b/remoting/resources/menu/chromoting_actionbar.xml
@@ -1,6 +1,10 @@
<?xml version="1.0" encoding="utf-8"?>
<!--Action bar buttons for the Android app's host listing-->
<menu xmlns:android="http://schemas.android.com/apk/res/android">
+ <item android:id="@+id/actionbar_directoryrefresh"
+ android:title="@string/actionbar_directoryrefresh"
+ android:icon="@android:drawable/ic_popup_sync"
+ android:showAsAction="ifRoom"/>
<item android:id="@+id/actionbar_accountswitcher"
android:title="@string/actionbar_accountswitcher"
android:icon="@android:drawable/ic_menu_more"
diff --git a/remoting/resources/strings.xml b/remoting/resources/strings.xml
index 949a2aa..a2c2095 100644
--- a/remoting/resources/strings.xml
+++ b/remoting/resources/strings.xml
@@ -18,6 +18,7 @@
<string name="pin_entry_cancel">Cancel</string>
<!--Action bar buttons-->
+ <string name="actionbar_directoryrefresh">Refresh</string>
<string name="actionbar_accountswitcher">Accounts</string>
<string name="actionbar_hide">Hide</string>
<string name="actionbar_keyboard">Keyboard</string>
@@ -31,6 +32,7 @@
<string name="error_auth_canceled">Authentication prompt canceled by user</string>
<string name="error_no_accounts">Device not linked to any Google accounts</string>
<string name="error_auth_failed">Authentication with specified account failed</string>
+ <string name="error_bad_connection">No network connection</string>
<string name="error_cataloging_hosts">Unable to display host list</string>
<string name="error_displaying_host">Unable to display host entry</string>
<string name="error_unexpected_response">Account has no remote desktop hosts registered</string>