Commit Graph

23703 Commits

Author SHA1 Message Date
TreeHugger Robot
8be9e0d605 Merge "Don't crash if IMEI is empty" 2018-06-08 19:07:22 +00:00
Fan Zhang
838f6c7812 Fix memory leaks in Settings
Bug: 80507279
Test: inspected hprof before and after fix
Change-Id: I6ea2925695deb6261263649e858484e1667ec522
2018-06-08 11:56:42 -07:00
ryanywlin
884a945334 2nd attempt handle BT is not supported on emulator am: 59287d40d1
am: de3d9a0a73

Change-Id: I27c16d76f25480a36b5b73cdcb3042cee7754a5f
2018-06-08 11:33:53 -07:00
ryanywlin
de3d9a0a73 2nd attempt handle BT is not supported on emulator
am: 59287d40d1

Change-Id: I843fc58484df106af09cae87dcb886ea49aa53ea
2018-06-08 11:10:56 -07:00
Matthew Fritze
6a03db093a Don't crash if IMEI is empty
Change-Id: I5993d332dbd218c981ef5432aebb735d0000f67a
Fixes: 109787187
Test: robotests
2018-06-08 10:49:06 -07:00
TreeHugger Robot
a9b742dd53 Merge "Add help uri for prevent ringing gesture" 2018-06-08 16:29:31 +00:00
TreeHugger Robot
28e0ce5b40 Merge "Don't force lower case in ZenSettings" 2018-06-08 14:44:33 +00:00
TreeHugger Robot
fe805d7a41 Merge "Hide block pref on non-configurable channels" 2018-06-08 14:44:16 +00:00
Beverly
486d1facfa Add help uri for prevent ringing gesture
Test: build
Bug: 79270169
Change-Id: Iea5598f26649854521202363cad6ee297cef3057
2018-06-08 09:53:28 -04:00
ryanywlin
59287d40d1 2nd attempt handle BT is not supported on emulator
Error handle before using LocalBluetoothManager in the onStart
and onStop

Bug: 80491267
Test: make RunSettingsRoboTests ROBOTEST_FILTER="AudioOutputSwitchPreferenceControllerTest" -j42
Change-Id: I47f7d3b7cddc2fbbafb8fb5cf0fb6adb2d0d2d55
2018-06-08 15:05:22 +08:00
Fan Zhang
1fe94d4fd7 Fix context leak in AudioSwitchPreferenceController.
am: ee96c3a259

Change-Id: I3575074241ee60d573563edbffa214f0d705bea3
2018-06-07 18:14:01 -07:00
TreeHugger Robot
87bb929d7a Merge "Fix context leak in AudioSwitchPreferenceController." 2018-06-07 22:57:17 +00:00
Fan Zhang
d9f7fba943 Fix context leak in AudioSwitchPreferenceController.
In this controller the context is stored into a global singleton but was
not cleared, thus leaking context and associated views.

Change-Id: I4247f8ff753bc0a331c6c81a0e4b5b4bc41588de
Bug: 80507279
Test: robotests
Test: inspected hprof before/after change
2018-06-07 22:46:19 +00:00
Julia Reynolds
9e2ec5fa07 Hide block pref on non-configurable channels
Test: robotests
Bug: 109875297
Change-Id: I164ae2e07d178db9e84746e5c59f0968e6c77eae
2018-06-07 19:49:49 +00:00
Fan Zhang
ee96c3a259 Fix context leak in AudioSwitchPreferenceController.
In this controller the context is stored into a global singleton but was
not cleared, thus leaking context and associated views.

Change-Id: I4247f8ff753bc0a331c6c81a0e4b5b4bc41588de
Fixes: 80507279
Test: robotests
Test: inspected hprof before/after change
2018-06-07 18:53:59 +00:00
Joe Onorato
90c770278e Merge "Add comments and additional constants for the known bad anomaly types." into pi-dev am: 32d9f451f9
am: d4fba82998

Change-Id: I4f4985a792a337c90b2949d0015dc50343cad007
2018-06-07 10:39:20 -07:00
Joe Onorato
d4fba82998 Merge "Add comments and additional constants for the known bad anomaly types." into pi-dev
am: 32d9f451f9

Change-Id: Ib66379c8c5a5dbae415aecc701ebb0a66859b7d5
2018-06-07 10:07:51 -07:00
TreeHugger Robot
32d9f451f9 Merge "Add comments and additional constants for the known bad anomaly types." into pi-dev 2018-06-07 16:48:51 +00:00
Beverly
bb0501c2fd Don't force lower case in ZenSettings
Programatically lower casing letters is bad for translations

Change-Id: Iea39186b9716f628ed96ad457b09440bd177d821
Fixes: 77961695
Test: ZenModeSettingsTest.java
2018-06-07 11:14:11 -04:00
Nathan Harold
a927f53f3d [RadioInfo] Save PreferredNetworkMode to Settings am: daea8be9ab
am: 1701aaf186

Change-Id: Ic2e685fa8076d0ac1255d3bf61b49c70f1d1767f
2018-06-06 19:36:03 -07:00
Evan Rosky
70564e8c18 Request Focus in fingerprint rename dialog am: 7a1d52eb06
am: 679c8de583

Change-Id: I5e17fe8417ad5394bea6d9cc1a9a8f96f6fc1fa4
2018-06-06 19:25:24 -07:00
Nathan Harold
1701aaf186 [RadioInfo] Save PreferredNetworkMode to Settings
am: daea8be9ab

Change-Id: I801ae6b29a4cfe1286c4f032662191154dc911fb
2018-06-06 19:19:44 -07:00
Beverly
1717b8550e Merge "If zen settings are updated, don't show onboarding" into pi-dev
am: aaac61eb5e

Change-Id: I96929a31ab8f644f83db74ce12b201bd0852c782
2018-06-06 18:34:25 -07:00
Nathan Harold
daea8be9ab [RadioInfo] Save PreferredNetworkMode to Settings
Historically, the debug menu intentionally did not
save the preferred network mode once chosen. This
causes problems because some settings cause a phone
switch which overrides the preferred network mode,
which can cause another phone switch and again
override the preferred network mode, completely
obliterating the requested debug setting. This change
will now save the debug menu setting to the system
settings, which will prevent the circular changes and
loss of setting due to phone switching.

However, caching the debug setting to prevent the phone
switch logic from overriding the setting has a side
effect, which is why it wasn't done historically.
If a debug setting of the preferred network mode
is set, it will cause the UX of the non-debug network
preference screen to change. Thus, someone who uses
the debug menu to make changes must be careful to
re-set the setting to return to the correct UX of the
publicly displayed menu.

Bug: 95133265
Test: -manually set preferred network mode using the
       RadioInfo menu and observed that there is no
       phone switch when setting CDMA.
      -confirmed that changing the network mode in
       RadioInfo will cause UX changes to the public
       network preference menu.

Change-Id: I91f669956a6d02515530855c4617cd0a767d73fa
2018-06-06 17:27:50 -07:00
Evan Rosky
679c8de583 Request Focus in fingerprint rename dialog
am: 7a1d52eb06

Change-Id: I5add9d81a0e6814e5664fc25a3309acda29361b5
2018-06-06 17:24:22 -07:00
TreeHugger Robot
eba407663e Merge "Make icon size on Printing settings match the rest of settings" 2018-06-06 23:58:27 +00:00
Antony Sargent
5a6c3fe645 Make icon size on Printing settings match the rest of settings
The icons on this page were being displayed at 48dp because we were
using a generic Preference element, instead of the custom settings
AppPreference which we use in lots of other places in settings for
displaying app entries in a list (and has a custom layout that ends up
with 32dp icons).

Fixes: 78654919
Test: manual (go to Settings -> Connected devices -> Connection
preferences -> Printing)

Change-Id: Icf21ab6b41fc00936cd58f3342a8c5502c6dd87f
2018-06-06 16:35:52 -07:00
Joe Onorato
f4781a79b4 Add comments and additional constants for the known bad anomaly types.
They aren't being used now, but by declaring them now we can consolidate
what we are encouraging OEMs to do in the PI timeframe.

Bug: 78013987
Test: treehugger
Change-Id: I7f86491448e799081b18d71274d2629a902d4972
2018-06-06 16:22:33 -07:00
TreeHugger Robot
aaac61eb5e Merge "If zen settings are updated, don't show onboarding" into pi-dev 2018-06-06 21:40:43 +00:00
TreeHugger Robot
bef9b58f17 Merge "Change UI for wireless AP tether band selection" 2018-06-06 21:19:41 +00:00
Beverly
8aef786baa If zen settings are updated, don't show onboarding
Change-Id: I33c328e5d90f1b948c7dacb2e7f6265a1e496d35
Fixes: 109668117
Test: ZenOnboardingActivity
(cherry picked from commit da412f07e5)
2018-06-06 20:29:43 +00:00
Salvador Martinez
e83388164c Change UI for wireless AP tether band selection
We decided to use the list preference after all due to
some devices not supporting various combinations of AP
configurations. This change makes it so that there are
three different UIs depending on the configurations
which are supported.
- 5.0 GHz unsupported: disable the preference and just
set the value to 2.4 GHz
- all supported, no dual mode: allow the user to choose
EITHER 2.4 GHz or 5.0 GHz
- all supported, dual mode: allow the user to choose
2.4 GHz or BOTH 2.4 GHz & 5.0 GHz with 5.0 being
preferred

Test: atest SettingsRoboTests
Bug: 80315296
Change-Id: I888d35811a98b8cf0155a3cb96c42ff762763378
2018-06-06 13:09:41 -07:00
Evan Rosky
7a1d52eb06 Request Focus in fingerprint rename dialog
Bug: 68841055
Test: Manual. dialog comes up with focused entry
Change-Id: I5bb1b7227c1323595bf7f483e11e87e2c3550093
2018-06-06 12:52:05 -07:00
TreeHugger Robot
bc8f79c700 Merge "If zen settings are updated, don't show onboarding" 2018-06-06 19:51:37 +00:00
Doris Ling
cd885f057a Merge "Fix monkey crash on selecting account sync prefernece." 2018-06-06 18:44:56 +00:00
Beverly
da412f07e5 If zen settings are updated, don't show onboarding
Change-Id: I33c328e5d90f1b948c7dacb2e7f6265a1e496d35
Bug: 109668117
Test: ZenOnboardingActivity
2018-06-06 14:24:12 -04:00
yuemingw
a2a593dd3b Check EnforcedAdmin to see if location is locked down by device policy.
Bug: 78563793
Test: m -j RunSettingsRoboTests
and manually via TestDPC

Change-Id: Iad2f11bde51bf6c321beb3c4709d53971a8e40c7
2018-06-06 13:46:40 +01:00
TreeHugger Robot
a809c439a9 Merge "Fix gray out previously connected device preference issue" 2018-06-06 05:26:29 +00:00
TreeHugger Robot
ddfb21113e Merge "Add a flashlight slice in settings." 2018-06-06 04:07:35 +00:00
Steven Wu
97915b15d9 Add a flashlight slice in settings.
- It retrieves the flashlight status from Settings.Secure.
- It uses the broadcast relay to update flashlight status
without action on the slice.

Test: robotests
Change-Id: Ib4d636541f5166b8634326cce76aed5665989b76
Fixes: 74913192
2018-06-06 02:59:37 +00:00
Doris Ling
6e6ed32a03 Only add entry with unique package name to default browser list.
am: 688d1d81cb

Change-Id: I36e4ececdbe333a25f2f7cba6e4a3e4e2f229b71
2018-06-05 19:42:15 -07:00
TreeHugger Robot
66f666d70b Merge "No-op clean up on UserSettings to make it more readable" 2018-06-06 01:28:42 +00:00
Doris Ling
688d1d81cb Only add entry with unique package name to default browser list.
When we query the package manager for activities that can handle the
web data uri, different capable activities within the same package will
be returned as separate entries. However, when we show the default
browser apps to the user, the entries are simply base on package name.
So, if there are multiple activities within the same package that can
handle the web data, they will be shown as duplicate entries.

When we process the resolved activities, check the corresponding package
name for duplicate entries before adding it to the default browser list.

Change-Id: I4e1f1e1ea22781efe24d791b367246423ca7a3c4
Merged-In: I70c88866eb3d5fe6466554749e23c85f429dd30c
Fixes: 84207432
Test: make RunSettingsRoboTests
2018-06-05 17:48:30 -07:00
Doris Ling
366418dab7 Fix monkey crash on selecting account sync prefernece.
When running monkey, the authority in the preference might not have been
properly set. Add a check for valid authority before trying to update
the sync automatically setting.

Change-Id: I59f910565fc9f128e86bd92337135fe46fed12e1
Fixes: 80551551
Test: make RunSettingsRoboTests
2018-06-05 17:36:12 -07:00
TreeHugger Robot
c42b1d1c09 Merge "Only add entry with unique package name to default browser list." 2018-06-06 00:32:39 +00:00
Fan Zhang
d63c8ef3b9 No-op clean up on UserSettings to make it more readable
Bug: 72319180
Test: existing tests
Change-Id: Ie7a327658d1fe16055aad7e90ecffc1e56eb6657
2018-06-05 16:29:32 -07:00
Doris Ling
1503b0273b Only add entry with unique package name to default browser list.
When we query the package manager for activities that can handle the
web data uri, different capable activities within the same package will
be returned as separate entries. However, when we show the default
browser apps to the user, the entries are simply base on package name.
So, if there are multiple activities within the same package that can
handle the web data, they will be shown as duplicate entries.

When we process the resolved activities, check the corresponding package
name for duplicate entries before adding it to the default browser list.

Change-Id: I70c88866eb3d5fe6466554749e23c85f429dd30c
Fixes: 84207432
Test: make RunSettingsRoboTests
2018-06-05 15:28:27 -07:00
Fan Zhang
44931d1e96 Properly suppress zen mode entry from soundsetting page.
Change-Id: If2c033232b05753f3ba52bb986dd6c85cab6e452
Fixes: 72748524
Test: robotests
2018-06-05 15:10:45 -07:00
Fan Zhang
752350c8d8 Suppress prevent ringer gesture parent page in search.
The full page verion of prevent ringing is still searchable.

- Created a new controller for parent entry.
- On the child controller, inherit the same getAvailabilityStatus() but
  override AVILABLE_UNSEARCHABLE to AVAILABLE so it becomes searchable.

Fixes: 72748524
Test: existing robotests for PreventRingingPreferenceController

Change-Id: Id2454db110c81b59fa4a98338602b03de0812f8a
2018-06-05 15:10:45 -07:00
TreeHugger Robot
60f2f06855 Merge "Don't crash on null Intent in FingerprintEnrollFindSensor" 2018-06-05 19:52:53 +00:00