Bug 135397

Summary:

Web Inspector: Eliminate more forced layouts during timeline recordings

Product:

WebKit

Reporter:

Joseph Pecoraro <joepeck>

Component:

Web Inspector

Assignee:

Joseph Pecoraro <joepeck>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

buildbot, commit-queue, graouts, joepeck, rniwa, timothy, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
[PATCH] Proposed Fix	none
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2	none

Description Joseph Pecoraro 2014-07-29 15:31:36 PDT

Again, calculation of seconds per pixel by sizing elements can be eliminated by accessing the cached property elsewhere.

Comment 1 Radar WebKit Bug Importer 2014-07-29 15:33:18 PDT

<rdar://problem/17848773>

Comment 2 Joseph Pecoraro 2014-07-29 15:36:58 PDT

Created attachment 235702 [details]
[PATCH] Proposed Fix

Comment 3 Build Bot 2014-07-29 16:08:49 PDT

Comment on attachment 235702 [details]
[PATCH] Proposed Fix

Attachment 235702 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5892012406472704

New failing tests:
media/track/add-and-remove-track.html

Comment 4 Build Bot 2014-07-29 16:08:52 PDT

Created attachment 235709 [details]
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-16  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

Comment 5 WebKit Commit Bot 2014-07-29 17:31:16 PDT

Comment on attachment 235702 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 235702

Committed r171783: <http://trac.webkit.org/changeset/171783>

Comment 6 WebKit Commit Bot 2014-07-29 17:31:19 PDT

All reviewed patches have been landed.  Closing bug.