Uploading a non-existent file on Fuchsia returns ERR_NOT_IMPLEMENTED |
|||
Issue descriptionI'm working on adding file upload support to SimpleURLLoader (https://chromium-review.googlesource.com/c/chromium/src/+/811887). I have a test that tries to uploads a non-existent file that ends up with an ERR_NOT_IMPLEMENTED failure on Fuchsia instead of ERR_FILE_NOT_FOUND (Which is how it fails on all other platforms). I'll add an ifdef for now, but that seems a mighty odd failure to get.
,
Dec 7 2017
I think I'll disable it instead, just so it gets attention - maybe it's fine / doesn't matter, but best to be safe. Curiously, there is another test in net/ that uploads a non-existent file, and seems to be enabled on Fuchsia, and fails with the right error code.
,
Dec 7 2017
Can you give an example of a path which returns ERR_NOT_IMPLEMENTED ? And a path which returns ERR_FILE_NOT_FOUND? It's possible that the NOT_IMPLEMENTED code is being returned from a different filesystem type than the FILE_NOT_FOUND filesystem, hence the asymmetry. Hard to say if it's WAI without knowing more, though.
,
Dec 7 2017
For the one that returns ERR_NOT_IMPLEMENTED, I create a temp directory, and then use a path to a file in it that does not exist. Hrm... The ERR_FILE_NOT_FOUND_ONE looks to use c:\path\to\non\existant\file.randomness.12345"... Even on Linux. Wanting a cross-platform path was why I went with the other approach.
,
Dec 7 2017
Wait...that's not right. My test, because I was lazy, uses the path of a file used in other tests (Since I had it handy) that lives in content/test/data, and then appends a "foo" directory after the [valid, extant] file name.
,
Dec 7 2017
Given that the case is weird, I can presumably try to do something less weird, and the test will likely pass - but I think that case should be returning a file not found error.
,
Dec 7 2017
Removing HTTP label - seems to solely be a platform issue. Using a "normal" non-existent file seems to resolve the issue.
,
Jan 10
Archiving P3s older than 1 year with no owner or component. |
|||
►
Sign in to add a comment |
|||
Comment 1 by scottmg@chromium.org
, Dec 6 2017