Bug 1873976 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
I did a quick pass over the tests we expect to be fixed by this bug. I'm pretty sure these ones should be addressed here:

```Parsing: <file:c:\foo\bar.html> against <file:///tmp/mock/path>
Parsing: < File:c|////foo\bar.html> against <file:///tmp/mock/path>
Parsing: <C|/foo/bar> against <file:///tmp/mock/path>
Parsing: <..> against <file:///C:/>
Parsing: </> against <file:///C:/a/b>
Parsing: </> against <file://h/C:/a/b>```

@valentin, I noticed that these ones were attributed to Bug 1507354, however they don't seem to be addressed by the current patch we have. Can you comment on whether they should be included in this bug, Bug 1507354, or maybe we need a new bug?:
```Parsing: <C|> against <file://host/dir/file>
Parsing: <C|> against <file://host/D:/dir1/dir2/file>
Parsing: <C|#> against <file://host/dir/file>
Parsing: <C|?> against <file://host/dir/file>
Parsing: <C|/> against <file://host/dir/file>
Parsing: <C| /> against <file://host/dir/file>
Parsing: <C|\> against <file://host/dir/file>
Parsing: <file://[1::8]/C:/> against <about:blank>```
Revision 1 by
on 
I did a quick pass over the tests we expect to be fixed by this bug. I'm pretty sure these ones should be addressed here:

```Parsing: <file:c:\foo\bar.html> against <file:///tmp/mock/path>
Parsing: < File:c|////foo\bar.html> against <file:///tmp/mock/path>
Parsing: <C|/foo/bar> against <file:///tmp/mock/path>
Parsing: <..> against <file:///C:/>
Parsing: </> against <file:///C:/a/b>
Parsing: </> against <file://h/C:/a/b>
```

@valentin, I noticed that these ones were attributed to Bug 1507354, however they don't seem to be addressed by the current patch we have. Can you comment on whether they should be included in this bug, Bug 1507354, or maybe we need a new bug?:
```Parsing: <C|> against <file://host/dir/file>
Parsing: <C|> against <file://host/D:/dir1/dir2/file>
Parsing: <C|#> against <file://host/dir/file>
Parsing: <C|?> against <file://host/dir/file>
Parsing: <C|/> against <file://host/dir/file>
Parsing: <C| /> against <file://host/dir/file>
Parsing: <C|\> against <file://host/dir/file>
Parsing: <file://[1::8]/C:/> against <about:blank>```

Back to Bug 1873976 Comment 5