1160161 - Autophone - do not chmod -R full profile directory

Status: RESOLVED FIXED

(Testing Graveyard :: Autophone, defect)

Description

[Bob Clary [:bc] (inactive)]

As part of the fix for the missing minidumps when checking for crashes, I used chmod -R on the profile directory. For new/empty profiles this is not an issue but I found when running a large number of the supported unittests that the profile's cache2 directory became very large and it took significant (many minutes) to chmod the full directory. We should only chmod the path to the minidumps directory and any other locations we need to access without chmodding the world.

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: pr 28

The proof will be when I push this to production, but it tests well here though my devices run as root by default.

Comment 2

[Geoff Brown [:gbrown]]

You could probably rm the cache quickly, or use prefs to turn the cache off or limit its size -- but all of those ideas have disadvantages too.

Comment 3

[Bob Clary [:bc] (inactive)]

https://github.com/mozilla/autophone/commit/b904378250e418bef4b0ae0075437a6065db6de3