Commit Graph

4359 Commits

Author SHA1 Message Date
Kean Mariotti
2b30f33150 Merge changes Ieddaf47c,Ia96c58d5,I89e03fff into udc-dev am: 09a1c31019
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23489996

Change-Id: I525c14df638a20580fd2d2f0afb1c7192ef2430a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-06-02 19:13:46 +00:00
Kean Mariotti
09a1c31019 Merge changes Ieddaf47c,Ia96c58d5,I89e03fff into udc-dev
* changes:
  Add 'npm run test:presubmit' script
  Change displayed trace descriptors
  Optimize bugreport loading
2023-06-02 18:00:48 +00:00
Treehugger Robot
d189471c78 Merge "Add CUJ trace parser to Winscope" into udc-dev am: aaa8dcd1f7
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23515819

Change-Id: I1b4bf62a75fa97a704bcb5658f859b20e63549a9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-06-02 13:11:38 +00:00
Treehugger Robot
aaa8dcd1f7 Merge "Add CUJ trace parser to Winscope" into udc-dev 2023-06-02 12:26:57 +00:00
Kean Mariotti
24965f613d Add 'npm run test:presubmit' script
Fix: b/285562786
Test: npm run test:presubmit
Change-Id: Ieddaf47c6737fe49d485bf4d81d2b711e4334cd2
2023-06-02 12:25:23 +00:00
Kean Mariotti
d1c80891d9 Change displayed trace descriptors
Descriptor before:
'Transitions (wm_transition_trace.winscope (bugreport.zip), shell_transition_trace.winscope (bugreport.zip))'

Descriptor after:
'wm_transition_trace.winscope (bugreport.zip), shell_transition_trace.winscope (bugreport.zip)'

Fix: b/285561971
Test: load trace and check the displayed descriptor
Change-Id: Ia96c58d563036c68e2d9ab428a94051a23a6a5b9
2023-06-02 12:25:09 +00:00
Kean Mariotti
bc966c455c Optimize bugreport loading
Fix: b/269342535
Test: npm run test:all
Change-Id: I89e03fff08fe1c7e7169a6eeb08efeda04d098d0
2023-06-02 12:08:00 +00:00
TreeHugger Robot
16752f9c9d Merge "Add pre-upload hook to run Winscope tests" into udc-dev am: c4808fc1d2
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23489997

Change-Id: I85e5799120163bba891975e53d02587e0bba00af
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-06-02 11:00:56 +00:00
TreeHugger Robot
c4808fc1d2 Merge "Add pre-upload hook to run Winscope tests" into udc-dev 2023-06-02 10:31:48 +00:00
Pablo Gamito
9336f610f0 Add CUJ trace parser to Winscope
Bug: 265791272
Test: npm run test:all
Change-Id: Ibb62236c405ab44caf94c9ac66ef34cbef4a0111
2023-06-02 09:31:57 +00:00
Pablo Gamito
70c6fb8c88 Add pre-upload hook to run Winscope tests
Test: make a change and repo upload -c .
Fixes: 282179210
Change-Id: I70fdf3f17f8d539d00e42e528d1c7d4091696372
2023-06-02 09:31:57 +00:00
Kean Mariotti
b76f9c2d53 Fix broken tests am: 35e2e69f95
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23478816

Change-Id: Ibfba76baa11411d52c0519f534c8d11c8b9f3c81
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-31 14:33:41 +00:00
Kean Mariotti
35e2e69f95 Fix broken tests
Test: npm run test:all
Change-Id: I6f064dc8846b2e65dfb002d002aee80d8422c537
2023-05-31 09:37:16 +00:00
Pablo Gamito
565bda6139 Fix winscope test am: daa40dbd3f
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23474476

Change-Id: I00d954ae3a5b433abd1d2d1b4959f02d9b9bd25c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-30 17:54:27 +00:00
Pablo Gamito
496ba10ece Add tooltips to icons am: 115ed2f243
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23466728

Change-Id: I5b636897e3dfc3db94a8095586190dc4679041a9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-30 15:17:24 +00:00
Pablo Gamito
daa40dbd3f Fix winscope test
Meant to be amended to ag/23466728 but was auto-merged before I could

Test: npm run test:all
Bug: 23466728
Change-Id: I2cfb1ba7214ec0b316b489ef84a138861d878321
2023-05-30 15:08:02 +00:00
Pablo Gamito
115ed2f243 Add tooltips to icons
And add icon next to the trace name in more places for better association of icon to trace

Fixes: 284100487
Test: npm run test:all
Change-Id: I2cfb1ba7214ec0b316b489ef84a138861d878345
2023-05-30 10:31:12 +00:00
Pablo Gamito
6dab99f012 Make sure upload traces responds to changes in tracePipeline am: 8ee2c132b6
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23410897

Change-Id: Ic94aece5db3fd4bddf2e914cf3e5dc0b42ab79a6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-26 10:35:24 +00:00
Pablo Gamito
8ee2c132b6 Make sure upload traces responds to changes in tracePipeline
Otherwise clearing or removing loaded traces has no effect on the UI.

Test: load traces in winscope, delete some, clear all and make sure UI responds to those changes
Fixes: 283976269
Change-Id: I7eb981367191a47d3b50b31ee68492809275833a
2023-05-24 10:42:16 +00:00
Pablo Gamito
7579e5e904 Format transition viewer timestamps to the right type am: 9b0562b4d7
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23390677

Change-Id: I71ef2202e7477b6feaf37dca72dd706afe808d9c
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-23 22:28:41 +00:00
Pablo Gamito
2d4b1186b2 Add status to transition rows am: c12b72ac70
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23390676

Change-Id: I8b1e7c541dcd1cf23eb66450290d4ff404c3e195
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-23 22:28:27 +00:00
Pablo Gamito
9b0562b4d7 Format transition viewer timestamps to the right type
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb981367191a47d3b50b31ee68492848572833a
2023-05-23 16:41:25 +00:00
Pablo Gamito
c12b72ac70 Add status to transition rows
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb981367191a47d3b50b31ee68492844936923a
2023-05-23 15:24:55 +00:00
Pablo Gamito
f317ff11da Update Long parameters in WmTransitionData to Strings am: fa6d593ca2
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23389178

Change-Id: I27d431e980b517ed901931efaf93de49517c5a76
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-23 15:01:16 +00:00
Pablo Gamito
c9db79a9da Rename getLoadedTraceFiles to getLoadedTraces am: 1e01280f59
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23389177

Change-Id: Ibeeb6aecb3ad8ace517282b405d01c0b20b3cb0e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-23 13:38:49 +00:00
Pablo Gamito
28577f3450 Fix typo am: deee75f952
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23389176

Change-Id: Ia29478a5a51d74f6ecb8aef563f7a817c564f841
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-23 13:38:43 +00:00
Pablo Gamito
fa6d593ca2 Update Long parameters in WmTransitionData to Strings
For JS compatibility in Winscope

Test: load transition trace and make sure we have transaction ids
Bug: 277181336
Change-Id: I174222b90ecc9b6a82d48c0e469104bd857387e5
2023-05-23 13:27:35 +00:00
Pablo Gamito
1e01280f59 Rename getLoadedTraceFiles to getLoadedTraces
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb981367191a47d3b50b31ee66f14774936923a
2023-05-23 12:57:30 +00:00
Pablo Gamito
deee75f952 Fix typo
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb176367191a47d3b50b31ee66f147ff1d6923a
2023-05-23 12:56:37 +00:00
Pablo Gamito
1edb434345 Update e2e tests am: 8317c066aa
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23369063

Change-Id: I2c2616c1b3e73ba23b08223b3ea6093e1526ed5a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-23 07:37:20 +00:00
Pablo Gamito
2f42dbc493 Update protractor config am: 4d49b499a9
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23369059

Change-Id: Iffad851942eca848f3e511163bcdce5c40b63fa2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-22 16:42:27 +00:00
Pablo Gamito
244b03b3ed Add transition trace viewer am: ac6415277c
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23369058

Change-Id: I25ad5f78a62af882e5c64b9d8b8d885168e750ad
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-22 16:42:26 +00:00
Pablo Gamito
6894c4e86f Add transition traces parser am: a735b202f7
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23363199

Change-Id: I9410a15aa377c388734b5deef661ba4dd3e1c137
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-22 16:42:24 +00:00
Pablo Gamito
03786e5f58 Support multiple files for a single trace am: 99afc1013b
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23363200

Change-Id: Iad66be7b9ad1f7e79698fd2b06ba49d28c55a852
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-22 16:42:23 +00:00
Pablo Gamito
08cbbfdaa1 Add WM & Shell side transitions parsers am: 28c77be00f
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23367118

Change-Id: I17c85057921e652810572f12c8c4efef5e62b7b2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-22 16:41:51 +00:00
Pablo Gamito
8317c066aa Update e2e tests
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb923367191a47d3b50b31ee66f147ff1d6923a
2023-05-22 16:40:57 +00:00
Pablo Gamito
4d49b499a9 Update protractor config
Test: npm run test:e2e
Change-Id: I7eb981367191c47d3b50b31ee66f147ff1d1223c
2023-05-22 14:30:23 +00:00
Pablo Gamito
ac6415277c Add transition trace viewer
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb981367191a47d3b50b31ee66f147ff1d6923a
2023-05-22 14:30:23 +00:00
Pablo Gamito
a735b202f7 Add transition traces parser
Bug: 277181336
Test: npm run test:all
Change-Id: I7eb981367191c47d3b50b31ee66f147ff1d6923c
2023-05-22 14:29:56 +00:00
Pablo Gamito
99afc1013b Support multiple files for a single trace
Bug: 277181336
Test: npm run build:all && npm run test:all
Change-Id: I7eb981367191c47d3b50b31ee66f147ff1d69431
2023-05-22 14:20:39 +00:00
Pablo Gamito
28c77be00f Add WM & Shell side transitions parsers
Bug: 277181336
Test: npm run test:all

Change-Id: I7eb981367191c47d3b50b31ee66f147ff1d6943d
2023-05-22 08:29:03 +00:00
Treehugger Robot
0b1f44d5b0 Merge "Support delete" am: d50ebda005 am: 176525b3d4 am: 8b6f72ac33
Original change: https://android-review.googlesource.com/c/platform/development/+/2545512

Change-Id: Ib66cd12c32932d0597d4a7015fb3b831599faeff
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-18 14:01:43 +00:00
Treehugger Robot
176525b3d4 Merge "Support delete" am: d50ebda005
Original change: https://android-review.googlesource.com/c/platform/development/+/2545512

Change-Id: Ie166b2e12535d8ee19ade5aced9b445190fec8f6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-18 11:14:43 +00:00
SzuWei Lin
2aa95c6a9c Support delete
Bug: 278860852
Test: repo_review.py -g ... --delete "..."
Change-Id: Ib986cdf0d99b5cd29e3abf8b2e282d1e586a478d
2023-05-17 11:43:01 +08:00
Pablo Gamito
ac8a2b42ea Strip bugreport am: b6eb4bb828
Original change: https://googleplex-android-review.googlesource.com/c/platform/development/+/23214101

Change-Id: I849cfcb777c95be73ece1355541eeeef8f6047e9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-16 12:19:17 +00:00
Pablo Gamito
b6eb4bb828 Strip bugreport
Test: npm run test:e2e

Bug: 277181336
Change-Id: I241450fca4b6950ca0e41ff7f62adb85cf37af5f
2023-05-16 08:58:36 +00:00
Treehugger Robot
95c7350314 Merge "cargo_embargo: dedup output modules" am: 32be457c7a am: c1058e7efc am: fa52117f71
Original change: https://android-review.googlesource.com/c/platform/development/+/2583718

Change-Id: I87f4f442a85ba0f84c7f14ee910a0a36764174f2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-16 01:46:03 +00:00
Treehugger Robot
c1058e7efc Merge "cargo_embargo: dedup output modules" am: 32be457c7a
Original change: https://android-review.googlesource.com/c/platform/development/+/2583718

Change-Id: Ic3fe6b120fa34e1be3b41ca09fa5200591b84b45
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-05-16 00:19:15 +00:00
Frederick Mayle
3d6e609f03 cargo_embargo: dedup output modules
This is needed because of https://crrev.com/c/4517945. The new resolver
causes there to be multiple `rustc` invocations for some libraries. They
are only trivially different though, so dedup'ing the final output is
sufficient for now.

Bug: 280489895
Test: m cargo_embargo && ./android-merge-2-cargo-embargo.sh
Change-Id: If4a36e4f3847d18d03f76bad63bab6c45a9aef97
2023-05-10 14:57:13 -07:00
Pablo Gamito
edd0219c54 Update Winscope proxy to collect both transition trace files
Bug: 277181336
Test: npm run build:all && npm run test:all
Change-Id: I6697812fe929efe0927358fe14dc0ea155199631
2023-05-04 10:33:08 +00:00