Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display resume download ability #4

Closed
Infocatcher opened this issue Aug 24, 2013 · 3 comments

Comments

@Infocatcher
Copy link
Owner

commented Aug 24, 2013

Accept-Ranges: bytes + available Content-Length ?
But what about FTP?

We can use color indication to don't increase window height: use green font for Status: 200 OK.

Infocatcher added a commit that referenced this issue Aug 24, 2013
Infocatcher added a commit that referenced this issue Aug 24, 2013
Infocatcher added a commit that referenced this issue Aug 26, 2013
Infocatcher added a commit that referenced this issue Aug 26, 2013
Infocatcher added a commit that referenced this issue Aug 26, 2013
Add context menu item to test download resumability
(hidden if testDownloadResumability == true)
(#4)
Infocatcher added a commit that referenced this issue Aug 26, 2013
Use the same load options to get properties and to check resumability
Also remove isManualCall argument: we already have filtered HTTP referer in "own window" and we should always filter "invisible" (in UI) referer.
(#4)
Infocatcher added a commit that referenced this issue Aug 26, 2013
Simplify code
(related to #4)
Infocatcher added a commit that referenced this issue Aug 26, 2013
Remove lpp_canResumeDownload and lpp_resumeDownloadTested before new …
…request

And provide "global" lpp_resumeDownloadTested
(#4)
Infocatcher added a commit that referenced this issue Sep 22, 2013
Automatically run test, if "testDownloadResumability" changed from fa…
…lse to true

+ ensure only one active test request
(#4)
Infocatcher added a commit that referenced this issue Jan 14, 2014
Add special "testDownloadResumability.download" preference:
test may break downloading from some sites with per-IP limits
(#4)
Infocatcher added a commit that referenced this issue Jan 15, 2014
Infocatcher added a commit that referenced this issue Jan 15, 2014
Set "Range" request header
Note: server may ignore range end...
(#4)
Infocatcher added a commit that referenced this issue Jan 15, 2014
Infocatcher added a commit that referenced this issue Jan 15, 2014
Ignore request callbacks, if window is already closed
window.close()
=> nsIRequest.cancel()
=> some dalay (so window is already destroyed)
=> nsIRequestObserver.onStopRequest()
(#4)
Infocatcher added a commit that referenced this issue Jan 15, 2014
Infocatcher added a commit that referenced this issue Jan 15, 2014
Correctly handle canceled request
(correction for 965fb80 commit)
(#4)
@Infocatcher

This comment has been minimized.

Copy link
Owner Author

commented Jan 17, 2014

With #8 we probably can write disabled data into hidden nodes and apply settings immediately.

Infocatcher added a commit that referenced this issue Jan 17, 2014
Use CSS to hide disabled things from HTTP headers
(and apply settings immediately)
(#4, #8)
@Infocatcher

This comment has been minimized.

Copy link
Owner Author

commented Jan 18, 2014

@Infocatcher

This comment has been minimized.

Copy link
Owner Author

commented Jan 19, 2014

Infocatcher added a commit that referenced this issue Jan 28, 2014
Infocatcher added a commit that referenced this issue Mar 12, 2014
Infocatcher added a commit that referenced this issue Sep 29, 2014
Enable "Test Resumability" menu item after small delay
(it's useless to call it right after previous request)
(#4)
Infocatcher added a commit that referenced this issue Oct 19, 2014
Simply workaround to correctly compare two requests with fake URIs
See checkFakeURINeeded():
checkChannelResumable() -> this.getRequestHash(ch) != this.requestHash
(#4)
Infocatcher added a commit that referenced this issue Oct 19, 2014

@Infocatcher Infocatcher closed this Nov 8, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.