search

mitodl / micromasters

Portal for learners and course teams to access MITx Micromasters® programs
https://mm.mit.edu
BSD 3-Clause "New" or "Revised" License
29 stars 17 forks source link

Chrome on Android crashing when adding photo to cropper #2028

Open noisecapella opened 7 years ago

noisecapella commented 7 years ago

I think it's running out of memory? I'm not sure. The photo being uploaded is 2.4mb. The screen turns black and then I get an 'Aw snap!' crash message after a minute or so

pdpinch commented 7 years ago

Is this relate?

https://sentry.io/mit-office-of-digital-learning/micromasters/issues/191306984/

noisecapella commented 7 years ago

No, that profile is @aliceriot's and this bug is before the 'Save' button is clicked

noisecapella commented 7 years ago

This is an intermittent issue BTW, sometimes I can reproduce this 100% of the time and other times things just work fine. I feel like it's a memory issue fundamentally

alicewriteswrongs commented 7 years ago

hum not sure what we could do about this, we can't intercept the photo before the user adds it to the cropper, except maybe with some messaging that encourages them to avoid large photos or close open applications/tabs before attempting.

noisecapella commented 7 years ago

Maybe there are memory allocations we can avoid duplicating, though I agree it's hard to know if we can fix this. I'll bring in my Android debug cable tomorrow so I can see what the memory use is like.

alicewriteswrongs commented 7 years ago

@noisecapella did you have a chance to do more investigation of this?

noisecapella commented 7 years ago

I forgot to bring my debug cable today, I'll look at this tomorrow. I haven't seen this happen today so far FWIW