Commit Graph

3254 Commits

Author SHA1 Message Date
Chih-hung Hsieh
eb919bc560 Merge "Allow implicit-fallthrough warnings locally." am: dd68cf76de
am: 0db2c27df7

Change-Id: Iad6403f231cd7c125a57ec0913e996224ecc3b24
2018-10-24 14:23:59 -07:00
Chih-Hung Hsieh
8767a4ede6 Allow implicit-fallthrough warnings locally.
It will be a global error by default.

Test: make checkbuild
Bug: 112564944
Change-Id: Ib23d0cc8b95a3734d5bd28f8c9d967df669cfdf8
Exempt-From-Owner-Approval: do not block on new warnings
2018-10-19 12:13:48 -07:00
Chih-Hung Hsieh
0a6ca66a34 Merge "Fix implicit fallthrough warnings." am: 012692be3f
am: 3515bf0fc9

Change-Id: Ia07a0a00d1ae547bc7af968e6f8a7caeb51f3e6e
2018-10-12 18:05:24 -07:00
Chih-Hung Hsieh
6c5b82f7c5 Fix implicit fallthrough warnings.
Test: make checkbuild
Bug: 112564944
Change-Id: Ia33bf816fb6b179ac0a5a6a20bcb50c11b47431d
2018-10-12 15:05:01 -07:00
Yangster
8f842fae06 Merge "Phone service state change atom." am: 419fc89b52
am: 16e3399902

Change-Id: Ifbff53d3b113c527975b5509289600f734dcca7b
2018-10-10 15:53:35 -07:00
Yangster
4ccebeadc7 Phone service state change atom.
Background: BatteryStatsService tracks the phone scanning time metric. In Q,
we are migrating the dumpsys-based collection to the new infra. This CL is to
instrument the new logging mechanism.

Test: statsd test
FIX: b/116748990

Change-Id: I3cf5e0026bfc6f594c5f7b524f14b3a43b702afb
2018-10-10 19:45:10 +00:00
Igor Murashkin
13ad78e9e7 Merge "statsd: Add AID mappings for iorapd" am: 34bd3dfb39
am: 543c688c3b

Change-Id: I7bbdf54070318e3faa962690bbf214c51682bc37
2018-10-08 13:50:54 -07:00
Treehugger Robot
34bd3dfb39 Merge "statsd: Add AID mappings for iorapd" 2018-10-08 20:12:59 +00:00
Igor Murashkin
12bb16c955 statsd: Add AID mappings for iorapd
Bug: 117295673
Change-Id: Ie0c7a7bf823159492259ac1ec6680b45784afb84
2018-10-05 16:25:21 -07:00
Igor Murashkin
9485d2610f Merge "statsd: Add missing AIDs to UidMap" am: 4593c78168
am: df28b82042

Change-Id: I50631821a385b8c7a8b2889f392e5a1dce994517
2018-10-05 11:44:49 -07:00
Treehugger Robot
4593c78168 Merge "statsd: Add missing AIDs to UidMap" 2018-10-05 16:52:11 +00:00
Igor Murashkin
a1980ae12a statsd: Add missing AIDs to UidMap
Change-Id: I29f52727d344755a0c465105eb3a2c81b2bec700
2018-10-04 14:30:48 -07:00
Kweku Adams
bbaababf95 Merge "Handling potential SIGPIPE signals." am: 935f97ade5
am: 7790552308

Change-Id: I48058c9c2ac1bbf634ef02f68979a54dd0a13826
2018-09-29 01:27:39 -07:00
Kweku Adams
60a98f0cb7 Merge "Increasing TombstoneSection timeout to 2 minutes." am: 0119603e8e
am: c87180623d

Change-Id: I25d2ba1843ac79d1b23be64d3e19771f419e50e0
2018-09-29 01:22:16 -07:00
Treehugger Robot
935f97ade5 Merge "Handling potential SIGPIPE signals." 2018-09-29 07:39:04 +00:00
Treehugger Robot
0119603e8e Merge "Increasing TombstoneSection timeout to 2 minutes." 2018-09-29 07:38:59 +00:00
Bookatz
27e85f974b Merge "Statsd atoms: restrict OEM fields 100000-199999" am: 824f940195
am: 3854309030

Change-Id: I8c7cd1a832d496b469a98a4b9509f17c809cc78a
2018-09-21 00:14:26 -07:00
Treehugger Robot
824f940195 Merge "Statsd atoms: restrict OEM fields 100000-199999" 2018-09-21 03:00:33 +00:00
Kweku Adams
4d4a23306a Handling potential SIGPIPE signals.
There are cases where dumping Java traces times out and then later tries
to write to a closed pipe, causing incidentd to crash. This introduces a
handler for the SIGPIPE signal so it doesn't crash due to a closed pipe.

Bug: 111729299
Test: lower section timeout to 5 seconds and make sure incident service
doesn't crash

Change-Id: I868aba1ae21a50ae76977e0e09e782cf94198473
(cherry picked from commit 5b763c1347)
2018-09-18 18:55:19 -07:00
Bookatz
76aafcfbea Statsd atoms: restrict OEM fields 100000-199999
The current wording seems to suggest allocating an infinite number of
fields, which is not future-proof.

Bug: 72866543
Test: N/A
Change-Id: I88732ea8695521a39f20524348547cfe50cbf776
2018-09-18 10:40:43 -07:00
Bookatz
18852e3636 Merge "Unifying source of statsd BleScan calls and cleaning it up." am: 056a095dd3
am: 7ed0147613

Change-Id: Ied0f4683311276d2c643e882f57f8a21943aebe3
2018-09-17 11:45:14 -07:00
Kweku Adams
10e38c9432 Increasing TombstoneSection timeout to 2 minutes.
Java trace dumps can take significantly longer than 30 seconds. Testing
suggests that 2 minutes should be ample time.

Bug: 111729299
Test: 'adb shell incident' and ensure the section doesn't time out.
Change-Id: I35e09fc1ca608122a1a352985ea3741f6ea0a344
(cherry picked from commit 0f7167947a)
2018-09-14 16:54:32 -07:00
Bookatz
ae6738ed2a Unifying source of statsd BleScan calls and cleaning it up.
Bug: 80308558
Test: cts-tradefed run cts-dev -m CtsStatsdHostTestCases
Change-Id: I0b4a0b56631b4354b73f0fafea2c0dc7dff248d1
2018-09-13 18:48:10 +00:00
Josh Gao
86b0140271 Merge "statsd: fix double close." am: 0b8f17b3ba
am: 6547dad5ac

Change-Id: Ida7fc9c7f7e01d6993361c0734f71d1d13d97158
2018-09-05 12:34:31 -07:00
Treehugger Robot
0b8f17b3ba Merge "statsd: fix double close." 2018-09-05 18:16:46 +00:00
Tri Vo
d3e4700c9b Merge "Update PageTypeInfoParser to be less restrictive." am: 0f34408378
am: 884b86fe77

Change-Id: I4761a3906dffc9f45aeb948dc32785bdf9528ab6
2018-09-04 14:40:36 -07:00
Josh Gao
3330d1208f statsd: fix double close.
Previously, fdopen was being called on a file descriptor that was owned
by a unique_fd without releasing. This leads to a double close, since
both fclose and the unique_fd destructor will try to close the fd.

Bug: http://b/113880863
Test: treehugger
Change-Id: I6f6f48d304861b5e4d7efee0d3ad0e30178a95a4
2018-09-04 11:12:04 -07:00
Kenneth Magic
93ca85a172 Update PageTypeInfoParser to be less restrictive.
The page block order is an independent variable from the free
page order table in the kernel, so the parser should not error
out if they do not match.

Bug: 110559361
Test: incident_helper_test
Change-Id: I08ad9c8f9b29cc15b80a89929f3a1fb6886852e6
Merged-In: I08ad9c8f9b29cc15b80a89929f3a1fb6886852e6
(cherry picked from commit 3abb439916)
2018-08-28 10:39:42 -07:00
Jorim Jaggi
2c68abc9fc Merge "BootAnimation: Fix boot animation with hidden cutout" into pi-dev
am: e387ce18d8

Change-Id: I55c32ef69c4bb565376946ce65101f37ef0cce8a
2018-08-22 16:20:32 -07:00
Adrian Roos
9ee5dff83c BootAnimation: Fix boot animation with hidden cutout
We do this by storing the masking inset in a persistent property.
The boot animation then animates itself to where it would be if that
masking inset were applied, then changes the viewport.

For this to work, we also need to make sure the DisplayManagerService
has the right overlay right at the start.

Bug: 112876936
Test: Hide cutout, then reboot. Verify boot animation is smooth.
Change-Id: I3e988b2340b2e0d2be3939bdc6878704c234ccc8
2018-08-22 21:37:39 +01:00
Yi Kong
288f355e1a Modernize codebase by replacing NULL with nullptr
Fixes -Wzero-as-null-pointer-constant warning.

Test: m
Bug: 68236239
Change-Id: Ic440b9fcb9be97c316448b764f7110ef1f7eb525
Merged-In: Ic440b9fcb9be97c316448b764f7110ef1f7eb525
(cherry picked from commit 08a8d728db)
2018-08-08 14:08:35 -07:00
Yao Chen
39c8fe12e4 Merge "Disable statsd and make StatsLog no-op if ro.statsd.enable=false" into pi-dev
am: 38b92986df

Change-Id: Ia90bed14900a2c944165aefe00661d3979c08aed
2018-06-07 14:53:43 -07:00
Yao Chen
38b92986df Merge "Disable statsd and make StatsLog no-op if ro.statsd.enable=false" into pi-dev 2018-06-07 21:10:16 +00:00
Yao Chen
cf3829a696 Disable statsd and make StatsLog no-op if ro.statsd.enable=false
Bug: 79324611
Test: manually tested on marlin_svelte
Change-Id: Ic7fbf86d7c295cb6b2f20e7f9a957d1ed74c42c0
2018-06-06 17:25:08 -07:00
Steven Moreland
a89ae86745 screencap: return instead of _exit.
_exit was being used instead of return in order to
work around a static destructor issue that has been
fixed.

Bug: 77934844
Test: screencap (and it doesn't crash)
Change-Id: I5dc25b0af5099993a94705ac9c7b439e68432824
2018-05-25 00:55:51 +00:00
Makoto Onuki
d8cf91c5f8 Merge "Tweak exemption for sync requests made by FG apps" into pi-dev
am: 621fd20f55

Change-Id: I31ae3e9e11b9defb7e6352830439484d7ec7ac01
2018-05-24 17:54:37 -07:00
Makoto Onuki
d5f25d2cd8 Tweak exemption for sync requests made by FG apps
Exemption given to a sync request made by a foreground app (including
PROCESS_STATE_IMPORTANT_FOREGROUND).
At the schedule time, we promote the sync adapter app for a higher bucket:
- If the device is not dozing (so the sync will start right away)
  promote to ACTIVE for 1 hour.
- If the device is dozing (so the sync *won't* start right away),
promote to WORKING_SET for 4 hours, so it'll get a higher chance to be started once the
device comes out of doze.
- When the sync actually starts, we promote the sync adapter app to ACTIVE for 10 minutes,
so it can schedule and start more syncs without getting throttled, even when the first
operation was canceled and now we're retrying.

Test: atest cts/tests/tests/syncmanager/
Test: Manual test with "requestsync -f" and "am set-standby-bucket", while checking
"dumpsys usagestats"
Test: settings put global app_idle_constants \
  exempted_sync_scheduled_nd_duration=1,exempted_sync_scheduled_d_duration=2,exempted_sync_start_duration=3
  and check "dumpsys usagestats" and make sure the constants are properly updated.
Fixes: 72443754

Change-Id: I233d8e4be85769150830bac798abc04810f4cc7b
2018-05-23 14:50:05 -07:00
Jaewan Kim
a31254bde1 Merge "Differentiate MediaController in a process for MediaSession" into pi-dev
am: 43688e8f2f

Change-Id: I8deb492fa82dcb2bec04d85a65a0aa267be830a4
2018-05-21 17:29:35 -07:00
TreeHugger Robot
43688e8f2f Merge "Differentiate MediaController in a process for MediaSession" into pi-dev 2018-05-22 00:11:49 +00:00
Yangster-mac
02b40158ab Merge "Add a field in config to disable/enable the string hashing in metric report." into pi-dev
am: 76cb7fe2f6

Change-Id: I4bb331330c505396427bb72ff7b69c2b6653ba97
2018-05-21 12:01:32 -07:00
Jaewan Kim
21c23e30b4 Differentiate MediaController in a process for MediaSession
MediaSession#getCurrentControllerInfo() can have more fine grained
differentiation for multiple MediaController in a package/process

Bug: 79728675
Test: Run CtsMediaTestCases
Change-Id: I81e94736e795ee5e7fbd28cb6cd834cc3a3b8f98
2018-05-19 08:36:51 +09:00
Yangster-mac
1c58f04cd3 Add a field in config to disable/enable the string hashing in metric report.
Statsd hashes (using its own hashing function) raw strings to reduce the
upload data size when there are duplicate strings in the report. And in cloud,
the clearcut translator would backfill the strings.

In a few droidfood users, we find the translator was unable to do that. While
debugging the root cause, we first decided to provide an option to disable
the hashing from the cloud.

Test: statsd unit test, CTS test, tested manually

BUG: b/79943763
Change-Id: If0359c8cf3f3cf83a2938db9ebf95ea7906f0b0c
2018-05-18 10:39:50 -07:00
Chenjie Yu
dcc56f980a Merge "ValueMetric pushed events should check condition" into pi-dev
am: 644c94f125

Change-Id: Ib8555c0664bceee108dc4f93bbeaaa9a146ef00a
2018-05-17 15:53:17 -07:00
Chenjie Yu
021e25307d ValueMetric pushed events should check condition
+ fix unit test flakiness

Bug: 79873404
Change-Id: I15b52a79b18c05603640781e4450e7b62fac24ba
Fix: 79873404
Test: unit test
2018-05-16 14:50:11 -07:00
David Chen
fcac66945b Merge "Fixes Value metrics in statsd and app upgrades." into pi-dev
am: c9076fac9b

Change-Id: Ib54ea85970d57c70cba12157f8d0ef1fcf602a6d
2018-05-15 19:41:05 -07:00
TreeHugger Robot
c9076fac9b Merge "Fixes Value metrics in statsd and app upgrades." into pi-dev 2018-05-16 02:27:37 +00:00
David Chen
092a5a9b85 Fixes Value metrics in statsd and app upgrades.
Pulled value metrics with conditions had a subtle bug that caused
us to leave the condition on even if it should've been false.

Bug: 79778783
Test: Added unit-test and verified on marlin-eng.
Change-Id: I31f34791118319b3471f7a6ea8a024e2d511cfe7
2018-05-15 17:51:47 -07:00
Dan Willemsen
d00c851e55 Merge "Convert to Soong" am: acbd94d76f
am: b1005a8a2a

Change-Id: If5cb26e0669f79240d8fee1d40aaa2162d8e611f
2018-05-15 15:06:19 -07:00
Dan Willemsen
b1005a8a2a Merge "Convert to Soong"
am: acbd94d76f

Change-Id: Ib04e6936303b53cf4d919ad07a27a9b8dda86168
2018-05-15 15:00:18 -07:00
Dan Willemsen
4888b1f6f0 Convert to Soong
See build/soong/README.md for more information.

Test: m libframeworks_coretests_jni
Test: m FrameworkCoreTests_install
Test: m libshim_jni
Test: m CtsShimPrivUpgrade
Test: m libfilterfw
Test: m PMTest_Java_dual
Test: m libdefcontainer_jni
Test: m libperftestscore_jni
Test: m libpmtest32 libpmtest64
Test: m libprintspooler_jni
Test: m libsmartcamera_jni
Test: m idmap
Test: m libdrmframework_jni
Test: m libdvr_loader com.google.vr.platform com.google.vr.platform.xml
Test: m libfilterpack_imageproc libfilterpack_base
Test: m libwebviewchromium_loader
Test: m shared_mem_test
Test: m test-touchlag
Change-Id: I868561dd237fa28647896d59049ab9260373ada1
2018-05-14 16:47:02 -07:00