Commit Graph

3425 Commits

Author SHA1 Message Date
TreeHugger Robot
6cdbca04fb Merge "Update winscope dependencies" into sc-dev am: b184ea69d7
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/14185371

Change-Id: I0d0f7e6b258718386fd8c12decb41677bdf9319a
2021-04-15 22:20:43 +00:00
Nataniel Borges
d945fbd381 Update winscope dependencies
Current version of winscope had several dependencies with deprecation
and security warnings. To avoid issues, update to the latest compatible
versions according to `package.json` configuration using `yarn upgrade`

Bug: 162925924
Test: compile and run winscope (yarn run dev)
Change-Id: I29b387e0620e0460d9ea51dbca4039bbeb4deb39
2021-04-14 15:03:47 +02:00
Treehugger Robot
3b6adef890 Merge "Create out dir if it does not exist." am: bd491d9de7 am: 21ff43c134 am: e57b5ccd77 am: afb3add402
Original change: https://android-review.googlesource.com/c/platform/development/+/1646655

Change-Id: I3aa3f18de858dadf6777e691d998e80d870d77a3
2021-03-30 04:54:05 +00:00
Treehugger Robot
afb3add402 Merge "Create out dir if it does not exist." am: bd491d9de7 am: 21ff43c134 am: e57b5ccd77
Original change: https://android-review.googlesource.com/c/platform/development/+/1646655

Change-Id: Id75e2fcecd716996881f5776733f87889eda1b78
2021-03-30 04:17:24 +00:00
Dan Albert
49a88d3754 Create out dir if it does not exist.
Test: ./build_ndk_docs.py
Bug: None
Change-Id: I47a31b42df4bf555c95f5abde2b0c86e64a32d57
2021-03-19 13:49:27 -07:00
TreeHugger Robot
ff974aa561 Merge "Fix missing function MapNotNull" into sc-dev am: 834d2f5cea
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/13689865

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Icb3dd177da9dfb947c5865b304da83094944883a
2021-02-24 21:50:06 +00:00
Nataniel Borges
24e39c5c4c Fix missing function MapNotNull
MapNotNull is not a standard TS/JS function

Test: compile winscope and open any file
Bug: 167522376
Change-Id: I796d96eb963627bd3a872dea9dea51c7eaa97e8c
2021-02-24 19:53:31 +01:00
TreeHugger Robot
13dcea285c Merge "Add visible region to list of visibility reasons" into sc-dev am: 0c7313d8c7
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/13589847

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I11e4fe8de5c41619941cfd2281c1cf5df38bec8f
2021-02-17 09:22:13 +00:00
TreeHugger Robot
0c7313d8c7 Merge "Add visible region to list of visibility reasons" into sc-dev 2021-02-17 08:40:25 +00:00
Vishnu Nair
082f6a661f Add visible region to list of visibility reasons
Test: open trace from bug
Fixes: 157252333
Change-Id: Ifebac00a112798bbd03cf74458a1ffc7682890f6
2021-02-16 16:19:09 -08:00
Treehugger Robot
c6a3e052d5 Merge "[LSC] Add LOCAL_LICENSE_KINDS to development" am: d6b1cbf769 am: 4d108c6ffe am: 483ddf28c8 am: 73272bf578
Original change: https://android-review.googlesource.com/c/platform/development/+/1589100

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Iaf213e3192a41b538ccbf74f34dc80faa517c0d2
2021-02-16 01:35:40 +00:00
Treehugger Robot
73272bf578 Merge "[LSC] Add LOCAL_LICENSE_KINDS to development" am: d6b1cbf769 am: 4d108c6ffe am: 483ddf28c8
Original change: https://android-review.googlesource.com/c/platform/development/+/1589100

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I8bfb99db5af219de8b7c6844b99006009152bbe8
2021-02-16 01:02:14 +00:00
Hongming Jin
71bd984858 Allow capturing accessibility trace through adb connected device.
Bug: 157601519
Test: Manual test using local windscope
Change-Id: I776d38dfc22ac4577b568e1d9a67c71ce132df7d
2021-02-15 05:13:20 +00:00
Bob Badour
f7cf305656 [LSC] Add LOCAL_LICENSE_KINDS to development
Added SPDX-license-identifier-Apache-2.0 to:
  apps/BluetoothDebug/Android.bp
  apps/BuildWidget/Android.bp
  apps/CustomLocale/Android.bp
  apps/DevelopmentSettings/Android.bp
  apps/Fallback/Android.bp
  apps/GestureBuilder/Android.bp
  apps/NinePatchLab/Android.bp
  apps/OBJViewer/Android.bp
  apps/PushApiAuthenticator/Android.bp
  apps/PushApiTestAppOne/Android.bp
  apps/PushApiTestAppTwo/Android.bp
  apps/SdkSetup/Android.bp
  apps/SettingInjectorSample/Android.bp
  apps/WidgetPreview/Android.bp
  apps/launchperf/Android.bp
  cmds/monkey/Android.mk
  gsi/gsi_util/Android.bp
  gsi/repack_super_image/Android.bp
  host/windows/Android.bp
  python-packages/Android.bp
  samples/AccelerometerPlay/Android.bp
  samples/AdaptiveIconSample/Android.bp
  samples/AliasActivity/Android.bp
  samples/AndroidBeamDemo/Android.bp
  samples/AppNavigation/Android.bp
  samples/AutofillKeyboard/Android.bp
  samples/BackupRestore/Android.bp
  samples/BasicGLSurfaceView/Android.bp
  samples/BluetoothChat/Android.bp
  samples/BrokenKeyDerivation/Android.bp
  samples/BrokenKeyDerivation/tests/Android.bp
  samples/BusinessCard/Android.bp
  samples/Compass/Android.bp
  samples/ContactManager/Android.bp
  samples/CubeLiveWallpaper/Android.bp
  samples/DataWiper/Android.bp
  samples/DeviceAdminWhitelistedAccount/Android.bp
  samples/FixedGridLayout/Android.bp
  samples/HeavyWeight/Android.bp
  samples/HelloActivity/Android.bp
  samples/HelloActivity/tests/Android.bp
  samples/HelloEffects/Android.bp
  samples/Home/Android.bp
  samples/HoneycombGallery/Android.bp
  samples/InlineFillService/Android.bp
  samples/JetBoy/Android.bp
  samples/LceDemo/Android.bp
  samples/LunarLander/Android.bp
  samples/LunarLander/tests/Android.bp
  samples/MultiResolution/Android.bp
  samples/MultiWindow/Android.bp
  samples/MySampleRss/Android.bp
  samples/NotePad/Android.bp
  samples/Obb/Android.bp
  samples/RSSReader/Android.bp
  samples/ReceiveShareDemo/Android.bp
  samples/SampleSyncAdapter/Android.bp
  samples/SimpleJNI/Android.bp
  samples/SimpleJNI/jni/Android.bp
  samples/SipDemo/Android.bp
  samples/SkeletonApp/Android.bp
  samples/SkeletonApp/tests/Android.bp
  samples/Snake/Android.bp
  samples/Snake/tests/Android.bp
  samples/SoftKeyboard/Android.bp
  samples/SpellChecker/HelloSpellChecker/Android.bp
  samples/SpellChecker/SampleSpellCheckerService/Android.bp
  samples/StackWidget/Android.bp
  samples/ThemedNavBarKeyboard/Android.bp
  samples/ToyVpn/Android.bp
  samples/TtsEngine/Android.bp
  samples/USB/AdbTest/Android.bp
  samples/USB/MissileLauncher/Android.bp
  samples/UiAutomator/Android.mk
  samples/Vault/Android.bp
  samples/Vault/tests/Android.bp
  samples/VoiceRecognitionService/Android.bp
  samples/VoicemailProviderDemo/Android.bp
  samples/WeatherListWidget/Android.bp
  samples/WiFiDirectDemo/Android.bp
  samples/WiFiDirectServiceDiscovery/Android.bp
  samples/Wiktionary/Android.bp
  samples/WiktionarySimple/Android.bp
  samples/XmlAdapters/Android.bp
  samples/apkcachetest/Android.bp
  samples/training/NsdChat/Android.bp
  scripts/Android.bp
  testrunner/Android.mk
  testrunner/tests/Android_native.mk
  tools/apkcheck/Android.bp
  tools/bugreport/Android.bp
  tools/etc1tool/Android.bp
  tools/idegen/Android.bp
  tools/mkstubs/Android.bp
  tools/rmtypedefs/Android.bp
  vndk/Android.bp
  vndk/tools/Android.bp
  vndk/tools/elfcheck/Android.bp
  vndk/tools/header-checker/Android.bp
  vndk/tools/header-checker/src/repr/protobuf/proto/Android.bp
  vndk/tools/header-checker/tests/integration/ifunc/Android.bp
  vndk/tools/header-checker/tests/integration/version_script_example/Android.bp
  vndk/tools/image-diff-tool/Android.bp

Added SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-BSD
    SPDX-license-identifier-CPL-1.0 SPDX-license-identifier-MIT
    SPDX-license-identifier-Unicode-DFS SPDX-license-identifier-W3C
    legacy_restricted legacy_unencumbered
to:
  Android.bp

Added SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-CPL-1.0
to:
  samples/SearchableDictionary/Android.bp

Added SPDX-license-identifier-Apache-2.0 legacy_unencumbered
to:
  tools/Android.bp

Added SPDX-license-identifier-BSD SPDX-license-identifier-CPL-1.0
    SPDX-license-identifier-MIT SPDX-license-identifier-Unicode-DFS
    SPDX-license-identifier-W3C
to:
  sdk/Android.bp

Added legacy_restricted
to:
  samples/KeyChainDemo/Android.bp
  tools/line_endings/Android.bp
  vndk/tools/definition-tool/datasets/Android.bp

Bug: 68860345
Bug: 151177513
Bug: 151953481

Test: m all

Exempt-From-Owner-Approval: janitorial work
Change-Id: I9847e00fb4278bc083b4debbc751003c995ea45c
2021-02-12 17:13:45 -08:00
TreeHugger Robot
1a50ba43c1 Merge "Make winscope compatible with flickerlib constructors" into sc-dev am: 471c1da8dc
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/13434216

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I7ee56c7a0c30ffdb662ccfb2c07e71e4d0851e03
2021-02-05 14:57:17 +00:00
Nataniel Borges
dc836bc679 Make winscope compatible with flickerlib constructors
Recently the constructors of the flickerlib objects were
updated to reduce memory.

This CL updates winscope parser objects and fix visualization
so that it is compatible with those changes

Bug: 178572522
Test: `yarn run dev` and open a window manager trace
Change-Id: Ic331fe90d17e5c30c9062f119e26e824dbe1e5a7
2021-02-05 13:59:03 +01:00
Hongming Jin
75c9565e5e Add feature to load accessibility trace file.
Test: manual test
Bug: 157601519
Change-Id: I0383a0232beead1e363a750adccb23911fb8e38a
2021-01-25 19:03:07 -08:00
Nataniel Borges
0b1e4bbf9f Merge "Consider rounded corners in occluded by (winscope)" 2021-01-06 09:38:14 +00:00
Nataniel Borges
5f2176643f Add windowContainer to winscope
This information was removed in a previous code refactor but is still
necessary

Bug: 162925924
Test: Compile winscope. Open a WM trace. Check if the window container
property exists

Change-Id: Ia93b927945f6aafa428fe5ebc3c3aace8223e2a7
2021-01-06 09:56:52 +01:00
Nataniel Borges
8485125094 Consider rounded corners in occluded by (winscope)
The visible region calculation in the surface flinger doesn't consider that a layer with rounded corners can fully occlude another. Use the same behavior in flicker and winscope.

Bug: 174541970
Test: atest FlickerTests
Change-Id: Ie8e871a94b4413d2f02624c5738841d5d861bbbe
2021-01-05 17:18:36 +01:00
Vishnu Nair
d52381da39 Fix sf trace flag propagation in winscope
Fixes a couple of issues:
- if no flags are set, the configuration will not be send to the proxy.
Fix this by always sending the requested config
- make winscope tracing sync by default as a temp workaround for the
async tracing crashes

Bug: none
Test: toggle composition trace flags from winscope and check trace
Change-Id: I724dcc19309a616d3eb096d966dd40bf00c5d8ed
2020-12-17 23:05:17 +00:00
TreeHugger Robot
6a7886ae39 Merge "Revert "Winscope: add more sf trace flags and visibility reasons"" 2020-12-07 19:14:56 +00:00
Vishnu Nair
8e5aa26704 Revert "Winscope: add more sf trace flags and visibility reasons"
This reverts commit 78574c94dc.

Reason for revert: breaks winscope

Change-Id: I5a2200ce49277de67213c7302fed7f5d3d9a7ac1
Fixes: 174993871
2020-12-07 17:25:09 +00:00
TreeHugger Robot
858dbc28bf Merge "winscope: handle missing child layers gracefully" 2020-12-04 18:51:49 +00:00
Vishnu Nair
78574c94dc Winscope: add more sf trace flags and visibility reasons
- Detect if a layer is not drawn if its corner radius crop is incorrectly set
- Add more sf trace flags with usage info

Test: manual testing
Change-Id: I4dbf93c1cc1a51bd713414451832f4f873054afa
2020-12-03 18:40:38 -08:00
Nataniel Borges
c4d960d343 Allow null stableBounds on DisplayContent
Sometimes this information is null on the proto, this would crash
Winscope. Instead use an empty Rect when the information is unavailable

Bug: 167521440
Test: open a WM trace on Winscope
Change-Id: Ia6ceddfbbf8e083d02932925536ac64edafa6a2a
2020-12-03 19:03:18 +01:00
Nataniel Borges
d4a36f8cdc Merge changes from topic "flicker-cts"
* changes:
  Determine if the WM entry is complete or not
  Make winscope compatible with flicker/CTS infra
2020-12-03 08:59:15 +00:00
Ioana Stefan
0c2f1368ca Merge "Change parse logic for complex flags" 2020-12-01 13:12:38 +00:00
Nataniel Borges
9534747390 Determine if the WM entry is complete or not
Use the same logic from CTS tests

Read the information from flickerlib and display in winscope

Test: yarn run dev and open a few WM traces
Bug: 167521440
Change-Id: I96cb956968066afaabb3acf53615db53bdfabe37
2020-12-01 12:25:40 +01:00
Nataniel Borges
5e4ca356ae Make winscope compatible with flicker/CTS infra
Flickerlib is migrating to the same structure as the CTS tests. This cl
makes winscope compatible with this changes. It now parses traces with
the new information and add a "flickerObj" entry to each node with the
calculated vlaues from flickerlib

Test: yarn run dev and open a few WM traces
Bug: 167521440
Bug: 170372278
Change-Id: I4a116ccabe392c75e5b5049808d4837f865cb2c7
2020-11-30 15:29:49 +01:00
Ioana Stefan
e30653b690 Update Winscope README
This change updates the Winscope README with the new way of generating
the IntDefs mapping and copying it to the right location.

Test: N/A
Change-Id: Ifec40c8f0c878df37142899bac87413b46bb9e40
2020-11-30 11:51:57 +00:00
Ioana Stefan
0a8e0fb003 Change parse logic for complex flags
This change modifies the order in which flags are considered for IntDef
matching in Winscope. The most inclusive flags will be considered and
displayed first. Because of this, their composing values will not be
displayed anymore if a more inclusive flag is found first.

Bug: 173682107
Test: do a WM trace through ADB Connect in Winscope and see Gravity
values

Change-Id: I4fbba4d9bd11ef6acfec50acc3147f5994e4bd1f
2020-11-30 09:48:59 +00:00
Vishnu Nair
e8ed906987 Merge "winscope: fix incorrect bounds offset" 2020-11-25 00:08:01 +00:00
Vishnu Nair
632c18f34d winscope: handle missing child layers gracefully
log a warning instead of failing to open the trace

Test: open traces in b/169708770
Change-Id: Iff327b2a6f41c231a379a698345e39b1d89e74e1
2020-11-24 13:07:49 -08:00
Vishnu Nair
75c0e70c00 winscope: fix incorrect bounds offset
Fixes incorrect visualization of sf layers if layer bounds are
not at 0,0. This can happen if the parent layer ends up cropping
the child layer.

Test: manual test with problematic winscope trace
Change-Id: I1e0c4ec0e632b015f6fae26ee20985cd6c56aa16
2020-11-23 12:49:25 -08:00
Ioana Stefan
e49c271314 Display only current InputConnection call data in Winscope
With this change Winscope will display only the current InputConnection
call information, compared to before, when all other methods were
displayed with null information.

This change affects both the client and service views.

Bug: 154348613
Test: upload a cleint or service trace in Winscope and visualise
inputConnectionCall information

Change-Id: I99e254715f7252e9349e27f56dec603d2e5f31bd
2020-11-20 17:33:44 +00:00
Ioana Stefan
d15dbbc63c Add more information on the Hierarchy side in Winscope
This change adds more contextual information on the Hierarchy side of
Winscope for IME traces. Some of the important attributes for
identification are added to the name of each entry.

Example views: https://screenshot.googleplex.com/8oPMKoGsu9nRcYG,
https://screenshot.googleplex.com/ANwZ3HshmTLteQb,
https://screenshot.googleplex.com/AaNYWJPUoVNGbM3

Bug: 154348613
Test: upload IME traces in Winscope or use ADB Connect
      check the 3 views
Change-Id: I1a4f887674ddab9cea20693a34b62af52a47a151
2020-11-20 11:38:26 +00:00
Ioana Stefan
0079eaa440 Fix ADB connect InputMethod naming
This changes fixes the InputMethodManagerSerice file naming for starting
tracing through ADB connect.

Bug: 154348613
Test: start Winscope and do an InputMethodManagerService trace through
      ADB connect
Change-Id: Ia627250f523922ef8f73bb277a95f44aacd94f0e
2020-11-17 17:47:28 +00:00
Ioana Stefan
38a47cfed7 Update clients view for IME tracing
This change updates the tarnsform operation of the clients view in
WInscope. This is done because of the new structure of dumped data on
the clients side: one trigger dumps a single client, instead of the
entire list of active clients.

Bug: 154348613
Test: flash a device
      start IME tracing by calling "adb shell ime tracing start"
      end IME tracing by calling "adb shell ime tracing stop"
      pull generated trace files and visualize in Winscope
      or start tracing directly through ADB Connect and visualize traces
Change-Id: I50c9a27a01a2f026acf4ee0f9dc394a6f74c06f2
2020-11-12 14:06:57 +00:00
Ioana Stefan
fdd20c9e95 Updated Winscope view for IME tracing InputMethodManagerService information
Updated the IME tracing view in Winscope for the new optimized proto
dump format. This change adds the IMMS view separately from the
previously added clients and IMS views.

Bug: 154348613
Test: build and run the code using "yarn run dev"
      build and flash the android code to a device
      start and stop IME trace from the command line and then upload the files into Winscope or
      start IME trace from the web interface, open an IME on the android device to generate logs, end trace from the web interface to see the logs
Change-Id: I0e3a2089c3d7b29c57477adc882047bb0829d2fd
2020-11-09 12:24:54 +00:00
Ioana Stefan
dedfadb349 Updated Winscope view for IME tracing InputMethodService information
Updated the IME tracing view in Winscope for the new optimized proto
dump format. This change adds the IMS view separately from the
previously added clients view.

Bug: 154348613
Test: build and run the code using "yarn run dev"
      build and flash the android code to a device
      start and stop IME trace from the command line and then upload the files into Winscope or
      start IME trace from the web interface, open an IME on the android device to generate logs, end trace from the web interface to see the logs
Change-Id: Ia1ae9dc55f20c776c6fccd3399a57e6747737a66
2020-11-09 12:24:36 +00:00
Ioana Stefan
724f11355e Updated Winscope view for IME tracing clients information
Updated the IME tracing view in Winscope for the new optimized proto
dump format. This change included only the clients view since only
this information is currently dumped.

Bug: 154348613
Test: build and run the code using "yarn run dev"
      build and flash the android code to a device
      start and stop IME trace from the command line and then upload the file into Winscope or
      start IME trace from the web interface, open an IME on the android device to generate logs, end trace from the web interface to see the logs
Change-Id: I3dc01d602147d41af3c5b99b848cb428e402c219
2020-11-09 12:24:21 +00:00
Ioana Stefan
caafebcfdc Add fallback mechanism for identifier of older window traces
Older WM traces might not have an identifier set for
WindowContainer since this was added in ag/11659380.

This change makes Winscope again backwards compatible with older
WM traces. If it exists, the identifier of a parent holds the
place of a missing WindowContainer identifier. It is possible
that both identifiers are null, in which case the title and the
hashCode of WindowContainer are set as empty strings.

Bug: 170372278
Test: build Winscope locally, upload a WM trace and visualise it
Change-Id: Ieb5adc1d6a94d9a9215304b92114b807f1d812b4
2020-10-29 15:03:12 +00:00
Julien Desprez
118e177e01 Merge "Delete hosttestlib as it's a legacy approach" am: 2f18dab43e am: 3ec8c61452 am: 4f5b4d7d57 am: 8d2f1078e8
Original change: https://android-review.googlesource.com/c/platform/development/+/1359842

Change-Id: Ic007a301c30a49066e6957c5cf18593c76258719
2020-10-23 04:41:28 +00:00
Julien Desprez
2f18dab43e Merge "Delete hosttestlib as it's a legacy approach" 2020-10-22 23:00:08 +00:00
TreeHugger Robot
a5b6561f83 Merge "Fix z-ordering in winscope sf traces" 2020-10-21 17:26:12 +00:00
Vishnu Nair
3f1c28f9b8 Fix z-ordering in winscope sf traces
Fallout from previous refactors.

Test: Open sf trace in winscope. Confirm layers are displayed in correct z-order
Change-Id: I256d03fe4fdd475f0172c79b524bbe99f4c2dde4
2020-10-19 13:33:35 -07:00
Vishnu Nair
9a4753ac5f Winscope: fix layer traversal
Test: open sf trace of chrome in winscope
Change-Id: I4e0e9fad427e920021e00a2d6330027b43543502
2020-10-15 14:49:28 -07:00
Ioana Stefan
25b14c8e4f Merge "Include IME trace in Winscope" 2020-10-05 13:33:05 +00:00
Pablo Gamito
6285321a36 Merge "Update testing build configuration to be compatible with new changes" 2020-10-01 17:22:48 +00:00