Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Browser crashes when trying to save image #8665

Closed
srirambv opened this issue May 3, 2017 · 5 comments
Closed

Browser crashes when trying to save image #8665

srirambv opened this issue May 3, 2017 · 5 comments

Comments

@srirambv
Copy link
Collaborator

srirambv commented May 3, 2017

Test plan

  1. Go to https://imgur.com/gallery/rQIb4Vw
  2. Click "..." at bottom-right of image
  3. Select Download post

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Browser crashes when trying to save image

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version (revision SHA):
    Brave 0.15.2
    rev 71f7dfd

  • Steps to reproduce:

    1. Go to https://imgur.com/gallery/rQIb4Vw
    2. Click "..." at bottom-right of image
    3. Select Download post
    4. Wait for it...wait for it...wait for it...CRASH
  • Actual result:
    Browsers crashes

  • Expected result:
    Should not crash

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    No

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:
    Download file doesn't show as completed nor the progress indicator stops even after successfully downloading the file #8548

@srirambv srirambv added this to the 0.15.2 milestone May 3, 2017
@alexwykoff alexwykoff modified the milestones: 0.15.3, 0.15.2 May 4, 2017
@alexwykoff
Copy link
Contributor

@srirambv
Copy link
Collaborator Author

srirambv commented May 8, 2017

Still an issue on Preview 3 of 0.15.3

TypeError: Cannot read property 'downloadFinished' of undefined
    at module.exports.updateElectronDownloadItem (C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\app\browser\electronDownloadItem.js:17:15)
    at updateDownloadState (C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\app\filtering.js:467:3)
    at DownloadItem.<anonymous> (C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\app\filtering.js:525:7)
    at emitTwo (events.js:106:13)
    at DownloadItem.emit (events.js:194:7)
C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\node_modules\node-anonize2-relic-emscripten\anonize2-jumbo.js:67
            throw ex
            ^

TypeError: Cannot read property 'downloadFinished' of undefined
    at module.exports.updateElectronDownloadItem (C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\app\browser\electronDownloadItem.js:17:15)
    at updateDownloadState (C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\app\filtering.js:467:3)
    at DownloadItem.<anonymous> (C:\Users\Sriram BV\AppData\Local\brave\app-0.15.3\resources\app.asar\app\filtering.js:525:7)
    at emitTwo (events.js:106:13)
    at DownloadItem.emit (events.js:194:7)

@srirambv srirambv reopened this May 8, 2017
@NejcZdovc
Copy link
Contributor

NejcZdovc commented May 8, 2017

I think that this commit d3ff9d9 will close this issue as well

cc @bbondy

@bsclifton
Copy link
Member

confirmed this worked for me on Windows 10, x64 😄 Please retest when you get a chance, @srirambv 😄

@srirambv
Copy link
Collaborator Author

srirambv commented May 9, 2017

Works fine on Windows 10 x64. with 0.15.300 build 👍 . Closing the issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.