New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 884273 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

SuperSize: Some android resources have incorrect source paths

Project Member Reported by agrieve@chromium.org, Sep 14

Issue description

E.g. from:
https://chromium-review.googlesource.com/c/chromium/src/+/1220177

Size logs:
https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8935396127474216320/+/steps/Show_Supersize_Diff/0/logs/s_____Show_Diff____/0

They show:
+ 3)       9374 (57.5%) o@0x0        859 (0->859)       $root_gen_dir//b/swarming/w/ir/tmp/t/tmprLezrd/v14/layout-v17/download_manager_in_progress_video_item.xml
               res/layout-v17/download_manager_in_progress_video_item.xml
+ 5)      10479 (64.2%) o@0x0        503 (0->503)       chrome/android/java/res_download/drawable/circular_progress_bar_indeterminate_large.xml
               res/drawable/$circular_progress_bar_indeterminate_large__0.xml

While #5 looks correct, #3 is clearly wrong:
- It should not be marked as GENERATED
- It's path should be chrome/android/java/res_download/layout/download_manager_in_progress_video_item.xml


 
Cc: wnwen@chromium.org
Owner: ----
Status: Available (was: Assigned)
Making this available in case someone else can get to it before I can.

Sign in to add a comment