1399351 - It taks about 30-60sec to start uploading a video file to Youtube on Nightly57.0a1

Status: RESOLVED DUPLICATE of bug 1398556

(Core :: DOM: File, defect, P1)

Description

[Alice0775 White]

Build Identifier:
https://hg.mozilla.org/mozilla-central/rev/b0e945eed81db8bf076daf64e381c514f70144f0
Build ID 	20170912100139
User Agent 	Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0


[Tracking Requested - why for this release]: regression on Youtube upload


When uploading a video file(I have tested with 2.5MB mp4 video) to Youtube,
1. Upload progress status will not progress at 0% for about 30 seconds
2. After that period, the uploading process will be normally proceeded

The problem does not happen on Firefox56.0b11.

Reproduced: always

Steps To Reproduce:
1. login to https://www.youtube.com/
2. Click on Upload icon at top-right of window
3. Click on "Select files to upload" and choose a video file

Actual Results:
Upload progress status will stay at 0% for about 30 seconds.
After that period, the uploading process will be normally proceeded.


Expected Results:
The upload will start almost instantaneously(i.e within 1-2 sec)


Regression window(probability is 5/5, i.e, 100%):
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=7f1714325cb3e40427e40031a2191fe20cdc3241&tochange=6aa1cc819058deaa13f85278e2f5970b8ced0998

Regressed by: Bug 1290481

Once the problem occurred, the problem seems to be reproduced even if on the good build of nightly57.0a1 cycle.

Comment 1

[Alice0775 White]

Attachment: about:support

Comment 2

[Tom Tung [:tt, :ttung]]

I'll look into it.

Bug 1290481 may slow down while caching an opaque response since it needs to take one more mutex lock/unlock, and one more padding file read/write. However, 30 sec seems to be too much.

Comment 3

[Tom Tung [:tt, :ttung]]

(In reply to Alice0775 White from comment #0)
> Once the problem occurred, the problem seems to be reproduced even if on the
> good build of nightly57.0a1 cycle.

Bug 1290481 will upgrade the DBs (QuotaManager, (DOM) Cache) in your profile. Once you upgrade them and try to fallback to a previous version of Nightly/Beta/Firefox without cleaning your profile, you cannot use (DOM) Cache API, IndexedDB and AsmJSCache.

Comment 4

[Alice0775 White]

(In reply to Tom Tung [:tt] (OoO: 9/22 ~ 9/29) from comment #3)
> (In reply to Alice0775 White from comment #0)
> > Once the problem occurred, the problem seems to be reproduced even if on the
> > good build of nightly57.0a1 cycle.
> 
> Bug 1290481 will upgrade the DBs (QuotaManager, (DOM) Cache) in your
> profile. Once you upgrade them and try to fallback to a previous version of
> Nightly/Beta/Firefox without cleaning your profile, you cannot use (DOM)
> Cache API, IndexedDB and AsmJSCache.

Yes, I thought like that. 
So, when I tried to find the regression range, I used new profile every attempt.

Comment 5

[Tom Tung [:tt, :ttung]]

I'm currently working on another bug. Maybe Shawn can help to investigate this.

Comment 6

[Julien Cristau [:jcristau]]

This sounds similar to bug 1398556?

Comment 7

[Alice0775 White]

Maybe same.
But, in my case, The uploading is successfully performed on Nightly57.0a1..
And the progress % does not jump. 


BTW, Now, my youtube upload seems to be locked for 12 hr... Because I uploaded frequently and canceled it for this test. :(

Comment 8

[Andrea Marchesini [:baku]]

I just landed a fix for bug 1398556. If somebody can test this issue, with tomorrow nightly, maybe we can mark this bug as duplicate.

Comment 9

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

(In reply to Andrea Marchesini [:baku] from comment #8)
> I just landed a fix for bug 1398556. If somebody can test this issue, with
> tomorrow nightly, maybe we can mark this bug as duplicate.

I've tried the build with your patch, I didn't see "uploading pending" and uploading progress bar has been started immediately.
So it seems the fix actually work.

Comment 10

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

I've also tried the build that the patch in bug 1398556 has been backed out. "Uploading" in progress bar has no update for a long period of time.

Comment 11

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

(In reply to Alice0775 White from comment #7)
> Maybe same.
> But, in my case, The uploading is successfully performed on Nightly57.0a1..
> And the progress % does not jump. 
> 
I've played several times w/ or w/o the patch in bug 1398556.
With the patch in bug 1398556, progress bar % jumps immediately. So I think it's okay to mark this bug as duplicate.

Comment 12

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

Mark this bug as duplicate of bug 1398556.

Comment 13

[Shawn Huang [:shawnjohnjr][:shuang] (as a happy gecko contributor)]

Remove dependency.

Comment 14

[Sylvestre Ledru [:Sylvestre]]

The duplicate is fixed in 57, not tracking.