Open Bug 821693 Opened 13 years ago Updated 3 years ago

janky Scrolling on smartsheet.com Sheets

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Platform:
x86_64
Windows 7
Type:
defect

Tracking

()

Project Flags:
Performance Impact low

People

(Reporter: xtc4uall, Unassigned)

References

(
URL
)

Details

(Keywords: perf, perf:responsiveness)

Using Mozilla/5.0 (Windows NT 6.1; WOW64; rv:20.0) Gecko/20121213 Firefox/20.0 ID:20121213030751 Scrollbar-, Mousewheel and Key-Scrolling is janky. SPS Profile of scrolling several Times up/down: http://people.mozilla.com/~bgirard/cleopatra/#report=d5bf2c34d009e898782dccb7683285c7f7351457 Selection - Most time spent in: mozilla::gfx::AlphaBoxBlur::BoxBlur_SSE2(int,int,int,int,unsigned int *,unsigned int) mozilla::gfx::AlphaBoxBlur::BoxBlur_SSE2(int,int,int,int,unsigned int *,unsigned int) mozilla::gfx::AlphaBoxBlur::Blur() gfxAlphaBoxBlur::Paint(gfxContext *,gfxPoint const &) nsContextBoxBlur::DoPaint() nsCSSRendering::PaintBoxShadowOuter(nsPresContext *,nsRenderingContext &,nsIFrame *,nsRect const &,nsRect const &) nsDisplayBoxShadowOuter::Paint(nsDisplayListBuilder *,nsRenderingContext *) nsDisplayBoxShadowOuter::Paint mozilla::FrameLayerBuilder::DrawThebesLayer(mozilla::layers::ThebesLayer *,gfxContext *,nsIntRegion const &,nsIntRegion const &,void *) gfx::DrawThebesLayer ... Graphics Adapter Description ATI Radeon HD 4800 Series Adapter Drivers aticfx64 aticfx64 aticfx64 aticfx32 aticfx32 aticfx32 atiumd64 atidxx64 atidxx64 atiumdag atidxx32 atidxx32 atiumdva atiumd6a atitmm64 Adapter RAM 512 ClearType Parameters Gamma: 1800 Pixel Structure: RGB ClearType Level: 100 Enhanced Contrast: 400 Device ID 0x9442 Direct2D Enabled true DirectWrite Enabled true (6.2.9200.16440) Driver Date 7-3-2012 Driver Version 8.970.100.3000 GPU #2 Active false GPU Accelerated Windows 1/1 Direct3D 10 Vendor ID 0x1002 WebGL Renderer Google Inc. -- ANGLE (ATI Radeon HD 4800 Series) AzureCanvasBackend direct2d AzureContentBackend direct2d AzureFallbackCanvasBackend cairo
FWIW, a more recent Profile since there have been several GFX Fixes in the Meantime: http://people.mozilla.com/~bgirard/cleopatra/#report=bb15b54702c2a9b4e4b5efe79d310a898cb6d6bd
We've just started using Smartsheet at work and this is still an issue. Most people access it using Chrome instead because it doesn't jank while scrolling.
Old bug, but I'd like this to be investigated. I can provide info from Nightly if it helps resolve the jank on SmartSheet.
Severity: normal → S3

So the page is mutating a bunch for each scroll, but we're causing some trashing of our own. In my profile, a bunch of the problematic reflows come from NotifyIMEOfPositionChange. Masayuki, why does IME need to flush synchronously? Can't it rely on the information being roughly up-to-date? Once the page or we flush again, shouldn't we get another notification with the right styling? Example profile: https://share.firefox.dev/3xAArvL

Flags: needinfo?(masayuki)

Yeah, it's fine to reduce the frequency of getting position for IME. It may be used by IME to adjust their own UI to adjust for scrolling. Therefore, it's enough to notify IME even in lower frequency.

Flags: needinfo?(masayuki)

The Performance Impact Calculator has determined this bug's performance impact to be low. If you'd like to request re-triage, you can reset the Performance Impact flag to "?" or needinfo the triage sheriff.

Platforms: [x] Windows [x] macOS [x] Linux [x] Android
Impact on site: Causes noticeable jank
Websites affected: Rare
[x] Reproduces in Chrome

Performance Impact: ? → low
Keywords: perf:responsiveness
You need to log in before you can comment on or make changes to this bug.