Bug 78718 - Avoid full style recalc when 'style' attribute changes.
Summary: Avoid full style recalc when 'style' attribute changes.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andreas Kling
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-15 08:17 PST by Andreas Kling
Modified: 2012-11-17 13:34 PST (History)
13 users (show)

See Also:


Attachments
Patch (1.29 KB, patch)
2012-02-15 08:18 PST, Andreas Kling
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Patch (1.32 KB, patch)
2012-11-17 11:12 PST, Andreas Kling
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Kling 2012-02-15 08:17:00 PST
SSIA
Comment 1 Andreas Kling 2012-02-15 08:18:31 PST
Created attachment 127187 [details]
Patch
Comment 2 WebKit Review Bot 2012-02-15 08:20:20 PST
Attachment 127187 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

Updating OpenSource
First, rewinding head to replay your work on top of it...
Applying: [Qt] Move Qt platform specific GL Context/Surface creation out of WebCore into WebKit
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
Auto-merging Source/WebCore/ChangeLog
Auto-merging Source/WebCore/platform/graphics/GraphicsContext3D.h
CONFLICT (content): Merge conflict in Source/WebCore/platform/graphics/GraphicsContext3D.h
Auto-merging Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp
CONFLICT (content): Merge conflict in Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp
Auto-merging Source/WebKit/qt/ChangeLog
CONFLICT (content): Merge conflict in Source/WebKit/qt/ChangeLog
Auto-merging Source/WebKit/qt/WebCoreSupport/PageClientQt.cpp
CONFLICT (content): Merge conflict in Source/WebKit/qt/WebCoreSupport/PageClientQt.cpp
Failed to merge in the changes.
Patch failed at 0001 [Qt] Move Qt platform specific GL Context/Surface creation out of WebCore into WebKit

When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".

rebase refs/remotes/origin/master: command returned error: 1

Died at Tools/Scripts/update-webkit line 164.


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 WebKit Review Bot 2012-02-15 08:50:56 PST
Comment on attachment 127187 [details]
Patch

Attachment 127187 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11518875

New failing tests:
fast/dom/HTMLProgressElement/progress-bar-value-pseudo-element.html
Comment 4 Eric Seidel (no email) 2012-02-16 15:02:08 PST
The patch looks OK, but it looks like some other part of the code is depending on this style resolution.  We'll need to debug that layout test failure.
Comment 5 Andreas Kling 2012-02-17 01:46:10 PST
(In reply to comment #4)
> The patch looks OK, but it looks like some other part of the code is depending on this style resolution.  We'll need to debug that layout test failure.

Yep, looks like setAttribute('style', '-webkit-appearance: none') works for this today, but .style.webkitAppearance = 'none' does not. Crazy. CC'ing morrita who added the test.
Comment 6 Andreas Kling 2012-11-17 11:12:27 PST
Created attachment 174821 [details]
Patch

Let's try this again, as I can't reproduce the failure anymore. I suspect that something got fixed in shadow DOM styling over the last 9 months and this now works as it should.. *fingers crossed for ews*
Comment 7 WebKit Review Bot 2012-11-17 13:00:56 PST
Comment on attachment 174821 [details]
Patch

Clearing flags on attachment: 174821

Committed r135051: <http://trac.webkit.org/changeset/135051>
Comment 8 WebKit Review Bot 2012-11-17 13:01:00 PST
All reviewed patches have been landed.  Closing bug.