Bug 96554 - REGRESSION(r127438): Google Docs to renders text too small.
Summary: REGRESSION(r127438): Google Docs to renders text too small.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Andreas Kling
URL:
Keywords: InRadar, NeedsReduction
Depends on:
Blocks:
 
Reported: 2012-09-12 13:50 PDT by Ojan Vafai
Modified: 2012-09-17 11:07 PDT (History)
5 users (show)

See Also:

Attachments
Patch IV: A New Hope (7.83 KB, patch)
2012-09-13 20:32 PDT, Andreas Kling 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ojan Vafai 2012-09-12 13:50:00 PDT 
1. Make your window small (e.g. 800x600 or smaller).
2. Load https://docs.google.com/a/google.com/document/d/1I3RfUbuVVoMB4JY3o6qypiEUv7Y7zfEGmG3V78Fa5Ok/edit.

Initially it loads fine, then a few seconds after load, all the text resizes smaller, but the red underlines for spelling mistakes don't move.

I bisected webkit revisions to http://trac.webkit.org/changeset/127438. I was testing on Chromium Linux.
Comment 1 Andreas Kling 2012-09-12 14:28:14 PDT 
Repro'd on applemac WebKit. Taking.
Comment 2 Andreas Kling 2012-09-12 22:33:21 PDT 
Looks like something goes wrong with sharing of ElementAttributeData for elements with inline style. I don't fully understand the problem yet, will continue investigating tomorrow.
Comment 3 Andreas Kling 2012-09-13 20:32:35 PDT 
Created attachment 164039 [details]
Patch IV: A New Hope
Comment 4 Radar WebKit Bug Importer 2012-09-13 20:50:08 PDT 
<rdar://problem/12298302>
Comment 5 WebKit Review Bot 2012-09-15 16:16:27 PDT 
Comment on attachment 164039 [details]
Patch IV: A New Hope

Clearing flags on attachment: 164039

Committed r128697: <http://trac.webkit.org/changeset/128697>
Comment 6 WebKit Review Bot 2012-09-15 16:16:30 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 7 Andreas Kling 2012-09-17 11:02:14 PDT 
Big thanks to Ojan for bisecting this! :)
Comment 8 Ojan Vafai 2012-09-17 11:07:14 PDT 
(In reply to comment #7)
> Big thanks to Ojan for bisecting this! :)

Glad you were able to fix it so quickly!