896234 - Scrolling lags when a background tab is loading and the current page is zoomed in

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Yuan Pengfei]

STR:
1. Open http://news.google.com/ and press Ctrl + '+' to zoom in the page
2. Right click a link and choose "Open Link in New Tab"
3. Scrolling lags when the background tab is loading

First bad:
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-win32/1374179404/
Last good:
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-win32/1374177427/

Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=b9fe5a05edd9&tochange=bdf71cb870ac

Reverting the changes to layout/base/FrameLayerBuilder.cpp in changeset 4c89002460da fixes the problem, which seems a little weird.

Comment 1

[Benoit Girard (:BenWa)]

This is happening on windows 7? Can you paste your about:support and run these steps on the first bad build:

https://developer.mozilla.org/en-US/docs/Performance/Reporting_a_Performance_Problem

This will tell us more about your configuration and which renders are eating up time.

Comment 2

[Yuan Pengfei]

(In reply to Benoit Girard (:BenWa) from comment #1)
> This is happening on windows 7? Can you paste your about:support and run
> these steps on the first bad build:
> 
> https://developer.mozilla.org/en-US/docs/Performance/
> Reporting_a_Performance_Problem
> 
> This will tell us more about your configuration and which renders are eating
> up time.

http://people.mozilla.com/~bgirard/cleopatra/#report=6ec6548277f5e566a9e4ed243995426c204624f0

Clean profile. Reproduced with the steps in Description

about:support
=======================================================

Application Basics
------------------

Name: Firefox
Version: 25.0a1
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130720 Firefox/25.0

Extensions
----------

Name: Gecko Profiler
Version: 1.12.5
Enabled: true
ID: jid0-edalmuivkozlouyij0lpdx548bc@jetpack

Important Modified Preferences
------------------------------

browser.cache.disk.capacity: 358400
browser.cache.disk.smart_size.first_run: false
browser.cache.disk.smart_size.use_old_max: false
browser.cache.disk.smart_size_cached_value: 358400
browser.places.smartBookmarksVersion: 4
browser.sessionstore.upgradeBackup.latestBuildID: 20130720030214
browser.startup.homepage_override.buildID: 20130720030214
browser.startup.homepage_override.mstone: 25.0a1
extensions.lastAppVersion: 25.0a1
general.autoScroll: false
gfx.direct3d.last_used_feature_level_idx: 0
javascript.options.mem.notify: true
network.cookie.prefsMigrated: true
places.history.expiration.transient_current_max_pages: 104858
plugin.disable_full_page_plugin_for_types: application/pdf
plugin.importedState: true
privacy.sanitize.migrateFx3Prefs: true

Graphics
--------

Adapter Description: NVIDIA GeForce 405
Adapter Drivers: nvd3dumx,nvwgf2umx,nvwgf2umx nvd3dum,nvwgf2um,nvwgf2um
Adapter RAM: 512
Device ID: 0x10c5
Direct2D Enabled: true
DirectWrite Enabled: true (6.2.9200.16571)
Driver Date: 3-14-2013
Driver Version: 9.18.13.1422
GPU #2 Active: false
GPU Accelerated Windows: 2/2 Direct3D 10
Vendor ID: 0x10de
WebGL Renderer: Google Inc. -- ANGLE (NVIDIA GeForce 405 Direct3D9Ex vs_3_0 ps_3_0)
windowLayerManagerRemote: false
AzureCanvasBackend: direct2d
AzureContentBackend: direct2d
AzureFallbackCanvasBackend: cairo
AzureSkiaAccelerated: 0

JavaScript
----------

Incremental GC: true

Accessibility
-------------

Activated: false
Prevent Accessibility: 0

Library Versions
----------------

NSPR
Expected minimum version: 4.10
Version in use: 4.10

NSS
Expected minimum version: 3.15.1 Basic ECC Beta
Version in use: 3.15.1 Basic ECC Beta

NSSSMIME
Expected minimum version: 3.15.1 Basic ECC Beta
Version in use: 3.15.1 Basic ECC Beta

NSSSSL
Expected minimum version: 3.15.1 Basic ECC Beta
Version in use: 3.15.1 Basic ECC Beta

NSSUTIL
Expected minimum version: 3.15.1 Beta
Version in use: 3.15.1 Beta

Comment 3

[Milan Sreckovic [:milan] (needinfo for best results)]

Does a fix to bug 896250 make the problem go away?

Comment 4

[Yuan Pengfei]

(In reply to Milan Sreckovic [:milan] from comment #3)
> Does a fix to bug 896250 make the problem go away?

Theoretically it should fix this problem.
I did not notice that the return value of nsDisplayItem::MaxActiveLayers was wrong -_-#