Emil Ghitta, QA [:emilghitta] Reporter
	Description • 7 years ago

[Affected versions]:
Firefox 56.0a1 (Build Id:20170712100301)
Firefox 55.0b8 (Build Id: 20170710155419)

[Unaffected versions]
Firefox 54.0.1 (Build Id: 20170628145605)

[Affected platforms]:
Windows 10 64 bit.
macOS 10.12.6.
Ubuntu 16.04 64bit.

[Steps to reproduce]:
1. Launch Firefox with a clean profile.
2. Enable e10s (if it is not already enabled).
3. Go to https://plus.google.com/ webpage.
4. Create a collection.
5. Drag a picture to the collection.


[Expected result]:
The picture is uploaded successfully in a short time.

[Actual result]:
The picture takes a lot of time to load.

[Regression range]:

This seems to be a regression:
Last good revision: c35f0ad34397990adec0612015eb1a2a566434da
First bad revision: cf6065f64f9a8aef42144e7938c9c1aec92d8677

Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=c35f0ad34397990adec0612015eb1a2a566434da&tochange=cf6065f64f9a8aef42144e7938c9c1aec92d8677

[Additional notes]
Please note that this issue occurs only when e10s is enabled.

For further information regarding this issue please access the following link: https://goo.gl/f7irw3

	Emil Ghitta, QA [:emilghitta] Reporter
	Comment 1 • 7 years ago

Hi Andrea!

Can you please have a look into this ? 

Thanks!

	Andrea Marchesini [:baku] Assignee
	Comment 2 • 7 years ago

This patch was previously proposed as part of another bug.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 3 • 7 years ago

Comment on attachment 8886144 [details] [diff] [review]
FileReader should not do I/O on main-thread

I don't understand the comment. It says something about pipe, but no pipe is created in this code. So please improve the comment, it should say that CreateInputTransport creates a pipe.

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 4 • 7 years ago

Hi :baku, is this something we'd consider to uplift to ff55?

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 6 • 7 years ago

Looking at bug 1381465 comment 7, won't fix it in 55.

	Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)
	Comment 7 • 7 years ago

Looks like this was fixed in 56 in bug 1377589.