New issue
Advanced search Search tips

Issue 919290 link

Starred by 1 user
Status: Duplicate
Merged: issue 856911
Owner: ----
Closed: Jan 7
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

'change' event is not dispatched if the same path is selected again

Reported by thereald...@gmail.com, Jan 5 
UserAgent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.110 Safari/537.36

Steps to reproduce the problem:
1. Visit https://narcolepticinsomniac.github.io/base64-encoder/
2. Convert a local image
3. Dismiss converted image, removing it from local storage
4. Convert that same image again

What is the expected behavior?
Image should convert and save to local storage.

What went wrong?
??

Did this work before? No 

Does this work in other browsers? Yes

Chrome version: 70.0.3538.110  Channel: n/a
OS Version: 6.3
Flash Version: 

Screen record: https://streamable.com/1h4zb

The first few times in the demo I alternate images. When I select the same image I just converted and then removed from local storage, it fails with no error.

This only occurs if you convert, remove from storage and then attempt to convert the same image again, in consecutive order. Any other conversion between or a page reload, and it works as expected.

Tested Firefox with no issue. Not positive it's strictly a storage bug, but it's definitely a bug.

Comment 1 by thereald...@gmail.com, Jan 5 

Probably filed this in the wrong category. Since it's failing altogether it's likely an issue with FileReader converting the same value consecutively. Not seeing any way to edit though.

Comment 2 by phistuck@chromium.org, Jan 5

Components: -Blink>Storage Blink>Storage>FileAPI

Comment 3 by thereald...@gmail.com, Jan 5 

Nope, not that either. Apparently onchange doesn't register if you load the same file consecutively. Would've suspected it was my own garbage code and investigated further if it didn't work in Firefox. 

Not sure which behavior is technically correct, but resetting the value in the function works. Move it accordingly if this is an actual bug, or feel free to close it. I'm curious to hear the verdict.

Comment 4 by jaydeele...@gmail.com, Jan 5

1544057712551.pdf
13.7 KB Download

Comment 5 by phanindra.mandapaka@chromium.org, Jan 5

Labels: Needs-Milestone

Comment 6 by thereald...@gmail.com, Jan 5 

I'm gonna fix the repo used as an example in the OP, but I made a test file which alerts when onchange registers. Loading the same file consecutively registers in Firefox, but fails in Chromium. I get that the name is a duplicate value, so IDK which browser handles it correctly. 

Sorry the the OP was so wildly inaccurate. I'd edit it if I could.
onchange-test.html
615 bytes View Download

Comment 7 by phistuck@chromium.org, Jan 5

Components: Blink>Forms>File

Comment 8 by tkent@chromium.org, Jan 7

Components: -Blink>Storage>FileAPI
Mergedinto: 856911
Status: Duplicate (was: Unconfirmed)
Summary: 'change' event is not dispatched if the same path is selected again (was: Local storage won't consecutively set a value, remove that value, then set that same value)
Essentially, this is same as Issue 856911.
►

Sign in to add a comment