Closed Bug 146748 Opened 22 years ago Closed 22 years ago

No Visual Progress When Downloading Large PDF Files

Categories

(Core Graveyard :: Plug-ins, defect, P3)

All
Windows 2000
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.2alpha

People

(Reporter: lmcquarr, Assigned: srgchrpv)

References

()

Details

(Whiteboard: [PL2:NA][NEED SR=][Acrobat])

Attachments

(1 file)

Comments from bug 119494:

"It seems fixed on 1.0 RC3 (Build 2002052306). However, it still does not show
the progress correctly when downloading a huge pdf.


http://www.novell.com/products/edirectory/competitive/ad.pdf 
(around 800k)


It does load, however during downloading, the status bar at the bottom shows
"Document Done". The mozilla icon also does not spin to show it's downloading.
Only indication of activity is my network status blinking, until it finally
finished downloading and show up inside the browser.


Nothing critical, just minor cosmetic stuff. 
Otherwise it opens all my pdf correctly. =) "


Both myself and Adobe QE Jeff Moran also observed this behavior.

Note that 58% of Acrobat users are dial up users and the average
connection speed is 28.8!   (We folks in High Tech with our
VPN, DSL/Cable modems, etc, do not represent
the average american internet connection ... :-)

The Average American, therefore, is going to be quite confused as to
what is happening when they download their PDF files, and may even
assume that the application has hung.

This is, however, cosmetic.
The branch does not have the load groups fix from bug 106253. However, the trunk
does. 

Liz, do trunk builds give you the correct status?
isn't this a dup of 106253?
no, I do not think this is a dup, this one about progress when byte range
failed, and I would like to take care of it. 
Assignee: beppe → serge
ok, that's why I asked
Jeff Moran, Acrobat QE, was using the trunk builds from yesterday (mac and win).
He noted the lack of progress behavior to me.
pops 1.0.1
Priority: -- → P3
Target Milestone: --- → mozilla1.1alpha
On 1.1alpha, build 2002061104, the problem still occurs, but there is something
changed. The status message at the bottom shows "Stopped". Previously was
showing "Done."
 
Whiteboard: [Acrobat]
Attached patch patch proposalSplinter Review
Comment on attachment 91021 [details] [diff] [review]
patch proposal

r=peterl
Attachment #91021 - Flags: review+
Whiteboard: [Acrobat] → [PL2:NA][NEED SR=][Acrobat]
Comment on attachment 91021 [details] [diff] [review]
patch proposal

>Index: nsPluginHostImpl.cpp

>+  nsWeakPtr               mWeakPtrChannelCallbacs;

nit: why not mWeakPtrChannelCallbacks?	what's one more letter anyways? ;-)

otherwise this patch looks good.  sr=darin if you fix the spelling.
Attachment #91021 - Flags: superreview+
that will be changed.
thanks darin.
Checked in
/mozilla/modules/plugin/base/src/nsPluginHostImpl.cpp
new revision: 1.408; previous revision: 1.407
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
uh oh,,I see something funny here on 0812 trunk. When I load this file, I see a 
progress bar which works fine (on right nottom). However, on the left bottom, I 
see something like "Document :Done(%elapsed%sec)" instead of the actual time 
taken to load this url. This message stays even after I leave that pdf and go to 
some other webpage. reopen this? What is this fix really supposed to show me 
happening?
reopening to get on Serge's bug list
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: mozilla1.1alpha → mozilla1.2alpha
shrir , what makes you think 
"Document :Done(%elapsed%sec)" is the regression from this check in?
on fresh installed 2002081209 w2k build I see the same status string after
visiting only netscape.com
I agree,netscape.com shows this same thing on 0812 build. I would like to know 
what this fix is supposed to do ? Is this showing the progress bar on the bottom 
right and showing the elapsed time? Thx!
most of all it should shows the progress bar, and can shows "Stopped" or "Done"
in the status bar.
I'm resolving this bug as fixed
"Document :Done(%elapsed%sec)" has nothing to do with this (attachment 91021 [details] [diff] [review]) patch
moreover I see no such message in 20020820 build
Status: REOPENED → RESOLVED
Closed: 22 years ago22 years ago
Resolution: --- → FIXED
visual progress is seen now. verif
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: