Generated `userDataDir` are not removed from the disk

[MSIH]

My /tmp folder is filling up with 1000s of chrome browser session data like cookies and local storage.

drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:40 z1EvRqWIbNseXvZo2saou
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:38 ZboBqMZWbLjneQ8qXc6l_
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:39 Zg56KutgK36lXDf0Wl1zm
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:36 zJWoqBFejkCaAjxl0SGq_
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:48 ZN1ajYldDtyuiFU1xUbp7
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:36 zqHD899Hx7amfawm5BDRT
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:44 ZtXhTSjYuv_MCKVkrUGRl
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:38 zw13NsDoBSZPxRFCvCtfg
drwxrwxr-x   3 msihadmin msihadmin    4096 May  3 07:51 ZZ66wFo-KSHke6jfozQ7Y
msihadmin@msih004:/tmp$ cd zw13NsDoBSZPxRFCvCtfg/
msihadmin@msih004:/tmp/zw13NsDoBSZPxRFCvCtfg$ cd Default/
msihadmin@msih004:/tmp/zw13NsDoBSZPxRFCvCtfg/Default$ ls
 blob_storage  'Code Cache'   Cookies-journal  'Local Storage'
 Cache          Cookies       GPUCache         'Session Storage'
msihadmin@msih004:/tmp/zw13NsDoBSZPxRFCvCtfg/Default$ 

[szmarczak]

This works as expected. userDataDir is meant to reuse an existing profile and will create a new one if it does not exist.

[szmarczak]

Ah, the issue is that it doesn't remove the directories. Reproducible by running the example from readme.

[szmarczak]

This happens on both playwright and puppeteer

[szmarczak]

It's generated here:

browser-pool/src/launch-context.ts

Line 80 in 2f89cb5

userDataDir = join(tmpdir(), nanoid()),

but it doesn't exist so chrome creates it but doesn't clean it up since it was explicitly provided. Definitely an issue in browser-pool.

[szmarczak]

https://github.com/microsoft/playwright/blob/c4eb2d4b1e17a0de1cea7220be45545c7c524614/src/server/browserType.ts#L146-L149 we don't have to provide userDataDir

[szmarczak]

The bug is since f971af9

[szmarczak]

microsoft/playwright#9069