Closed Bug 1171391 Opened 9 years ago Closed 8 years ago

[e10s] Some sites slower to resolve with AdBlock Plus and NoScript enabled

Categories

(Firefox :: Extension Compatibility, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
e10s + ---

People

(Reporter: tech4pwd, Unassigned)

References

()

Details

Attachments

(2 files)

(deleted), application/octet-stream
Details
(deleted), application/octet-stream
Details
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:41.0) Gecko/20100101 Firefox/41.0
Build ID: 20150513191204

Steps to reproduce:

Confirmed as per the e10s thread on MozillaZine:
According to the web tools, it takes about 556ms to parse the index html. With e10s off, its taking 385ms. This may be worth filing a bug for.
Paul,  are you running this with any addons installed?  If so, please try disabling them all or trying in a clean profile.
Flags: needinfo?(pwd.mozilla)
These actually weren't my numbers. My numbers are much slower, we suspect the delay are exacerbated on Linux.
Flags: needinfo?(pwd.mozilla)
Flags: needinfo?(mconley)
So on my local Ubuntu VM, Nightly Linux does appear to retrieve the HTML document at the URL slightly faster without e10s than with e10s, but only around 10ms-20ms... not the hundreds of milliseconds that others are reporting.

Hey pwd - are you willing to gather a performance profile for me while loading this page in non-e10s and e10s mode? I show you how to get the profiler add-on and submit profiles here: https://www.youtube.com/watch?v=kGBs0BQsoQg&feature=youtu.be
QA Whiteboard: http://bato.to/
Flags: needinfo?(mconley) → needinfo?(pwd.mozilla)
(In reply to Mike Conley (:mconley) - Needinfo me! from comment #3)
> So on my local Ubuntu VM, Nightly Linux does appear to retrieve the HTML
> document at the URL slightly faster without e10s than with e10s, but only
> around 10ms-20ms... not the hundreds of milliseconds that others are
> reporting.
> 
> Hey pwd - are you willing to gather a performance profile for me while
> loading this page in non-e10s and e10s mode? I show you how to get the
> profiler add-on and submit profiles here:
> https://www.youtube.com/watch?v=kGBs0BQsoQg&feature=youtu.be

Sure thing: http://people.mozilla.org/~bgirard/cleopatra/#report=47729bf54b1fed2f6bb42a127590e846ae5ad413
Flags: needinfo?(pwd.mozilla)
Hrm. I'm afraid there are no symbols in this profile. :( I have a feeling our symbol server was misbehaving when you gathered this.

Would you mind going into about:config, and setting profiler.symbolicationUrl to http://symbolapi.mocotoolsprod.net/ and trying again?
Flags: needinfo?(pwd.mozilla)
(In reply to Mike Conley (:mconley) - Needinfo me! from comment #5)
> Hrm. I'm afraid there are no symbols in this profile. :( I have a feeling
> our symbol server was misbehaving when you gathered this.
> 
> Would you mind going into about:config, and setting
> profiler.symbolicationUrl to http://symbolapi.mocotoolsprod.net/ and trying
> again?

I cannot get the profile out because it keeps crashing my Firefox with OOM problems.
Flags: needinfo?(pwd.mozilla)
(In reply to Paul [pwd] from comment #6)
> (In reply to Mike Conley (:mconley) - Needinfo me! from comment #5)
> > Hrm. I'm afraid there are no symbols in this profile. :( I have a feeling
> > our symbol server was misbehaving when you gathered this.
> > 
> > Would you mind going into about:config, and setting
> > profiler.symbolicationUrl to http://symbolapi.mocotoolsprod.net/ and trying
> > again?
> 
> I cannot get the profile out because it keeps crashing my Firefox with OOM
> problems.

Hrm. Ok, if you go into the settings of the profiler (Add-on button > Settings > Advanced), can you reduce the number of samples? Maybe by like... 30%?
Flags: needinfo?(pwd.mozilla)
Attached file mGvo8cev.bin (deleted) —
For some reason, it wouldn't upload, so here's one.
Flags: needinfo?(pwd.mozilla)
Attached file xlFq96Wq.bin (deleted) —
Here's another one.
Flags: needinfo?(mconley)
Great, ok, I can see that one - I've uploaded it here:

http://people.mozilla.org/~bgirard/cleopatra/#report=c0f501990d47dc4d9b3c0fe1983e5ba84ce6fefa&selection=0,1,2,2,3,4,5,5,6,7,8,6,9,10,5,6,6,6,6,6,12,1132

The problem with it is that it only shows the main process. I also need samples from the content process - so I'm going to ask you to do this again, but can you do it with a recent Nightly build, and using the latest version of the Gecko Profiler Add-on (1.16.2 or higher)?
Flags: needinfo?(mconley) → needinfo?(pwd.mozilla)
It is Profiler 1.16.2. Also I'd love to but the Ubuntu nightly repo hasn't been updated since 0513, if you know of anyone who can get them kicked off again, I'll be happy to provide you any data you require.

I've filed a bug about the Ubuntu nightly builds being outdated here: https://bugs.launchpad.net/ubuntu-mozilla-ppa-bugs/+bug/1466230

I've not cleared the needinfo as hopefully a new build will pop out in the next few days and I can get a more meaningful profile for you.
Alternatively, you could download one of the Mozilla builds for Linux at https://nightly.mozilla.org/
(In reply to Mike Conley (:mconley) - Needinfo me! from comment #12)
> Alternatively, you could download one of the Mozilla builds for Linux at
> https://nightly.mozilla.org/

I have a slight OCD in regards to directory structure. Believe me, with the inconsistency of the Ubuntu Nightly releases, I'd love to ditch them for something whereby I'd get my nightly releases directly from Mozilla.
Fair enough. :)
New profile from the nightly of 20150718 which is apparently too big to attach: https://www.dropbox.com/s/0nqxrf5h0mtavmd/knDuqug%2B.bin?dl=0
Flags: needinfo?(pwd.mozilla)
Flags: needinfo?(mconley)
Ah, okay, I see what's happening - we're spending a lot of time inside the ContentPolicy add-on shim.

Can you please list me your enabled add-ons from about:support?
Flags: needinfo?(mconley) → needinfo?(pwd.mozilla)
Adblock Plus	2.6.10.3970	true	{d10d0bf8-f5b5-c8b4-a8b2-2b9879e08c5d}
Brief	2.0b3	true	brief@mozdev.org
British English Dictionary	1.19.1	true	en-GB@dictionaries.addons.mozilla.org
Bugzilla Tweaks	1.12.1.1.1-signed	true	jid0-qBnIpLfDFa4LpdrjhAC6vBqN20Q@jetpack
DictionarySearch	28.0.0.1-signed	true	{a0faa0a4-f1a7-4098-9a74-21efc3a92372}
FlashGot	1.5.6.12rc2	true	{19503e42-ca3c-4c27-b1e2-9cdb2170ee34}
ImageSearch	0.3.1.1-signed	true	jid1-NOlwYOe3E3vApg@jetpack
NoScript	2.6.9.33rc2	true	{73a6fe31-595d-460b-a920-fcc0f8843232}
PDF Viewer	1.0.277.1-signed	true	uriloader@pdf.js
Places Maintenance	1.3.1-signed	true	places-maintenance@bonardo.net
Saved Password Editor	2.9.1-signed	true	savedpasswordeditor@daniel.dawson
Send Tab to Device	0.7.1-signed	true	jid1-mdjmA7if6lo8lA@jetpack
Shumway	0.11.400	true	shumway@research.mozilla.org
Stylish	2.0.2.1-signed	true	{46551EC9-40F0-4e47-8E18-8E5CF550CFB8}
Stylish Sync	0.1.7.1-signed	true	{0e3fc079-afbb-4a00-87e5-9486062d0f9c}
Clean Links	2.7.1-signed	false	{158d7cb3-7039-4a75-8e0b-3bd0a464edd2}
Customize about:newtab	1.1.4.1-signed	false	customizenewtab@alejandrobrizuela.com.ar
Firefox Interest Dashboard	0.9.1.1-signed	false	firefox.interest.dashboard@up.mozilla
Gecko Profiler	1.16.3	false	jid0-edalmuivkozlouyij0lpdx548bc@jetpack
Tumblr Post	1.36.1-signed	false	{99210d54-6321-41e8-bd1b-2b4c55874efb}
Ubuntu Firefox Modifications	3.0	false	ubufox@ubuntu.com
Ubuntu Online Accounts	0.5	false	online-accounts@lists.launchpad.net
Unity Desktop Integration	3.0.2	false	webapps-team@lists.launchpad.net
Unity Websites integration	2014.07.01.beta	false	{2e1445b0-2682-11e1-bfc2-0800200c9a66}
Flags: needinfo?(pwd.mozilla)
Flags: needinfo?(mconley)
Okay Paul, I have a test for you to try:

Can you please disable NoScript and Adblock Plus, and then do a comparison between non-e10s and e10s? 

Does the time to download the page seem to match up with those disabled?
Flags: needinfo?(mconley) → needinfo?(pwd.mozilla)
The time with those add-ons is disabled (separate profile) is definitely better.
Flags: needinfo?(pwd.mozilla)
Blocks: e10s-addons
Summary: [e10s] Some sites slower to resolve with e10s enabled → [e10s] Some sites slower to resolve with AdBlock Plus and NoScript enabled
As far as Adblock Plus is concerned, this is a duplicate of bug 1161798.
Note that bug 1161798 is fixed as of Adblock Plus 2.7 - the content policy implementation is no longer relying on shims.
Cleaning up untriaged items. If it is not a proper component, please assist. Thanks
Component: Untriaged → Extension Compatibility
Status: UNCONFIRMED → NEW
Ever confirmed: true
closing based on comment 21.  we have bug 1192585 looking at any perf differences post 2.7.x
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: