Bug 37885 – backgroundColor is oddly indented in layer tree dump

Bug 37885 - backgroundColor is oddly indented in layer tree dump


Summary:	backgroundColor is oddly indented in layer tree dump

Status:	RESOLVED FIXED

Product:	WebKit
Component:	Layout and Rendering
Version:	528+ (Nightly build)
Platform:	All Mac OS X 10.5

Importance:	P2 Normal
Assigned To:	Simon Fraser (smfr)

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-04-20 13:32 PST by Simon Fraser (smfr)
Modified:	2010-04-20 22:42 PST (History)

Attachments
Patch (4.35 KB, patch) 2010-04-20 13:38 PST, Simon Fraser (smfr)	andersca: review+	Review Patch \| 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 From Simon Fraser (smfr) 2010-04-20 13:32:51 PST

In the layer tree dump obtained via layoutTestController.layerTreeAsText(), the backgroundColor property is weirdly indented:

       (backfaceVisibility visible)
            (backgroundColor none)
      (transform identity)

------- Comment #1 From Simon Fraser (smfr) 2010-04-20 13:38:32 PST -------

Created an attachment (id=53875) [details]
Patch

------- Comment #2 From Anders Carlsson 2010-04-20 13:39:14 PST -------

(From update of attachment 53875 [details])
r=me

------- Comment #3 From Simon Fraser (smfr) 2010-04-20 13:41:26 PST -------

http://trac.webkit.org/changeset/57913

------- Comment #4 From WebKit Review Bot 2010-04-20 16:08:19 PST -------

http://trac.webkit.org/changeset/57913 might have broken Qt Windows 32-bit Debug
The following changes are on the blame list:
http://trac.webkit.org/changeset/57909
http://trac.webkit.org/changeset/57910
http://trac.webkit.org/changeset/57911
http://trac.webkit.org/changeset/57913
http://trac.webkit.org/changeset/57914
http://trac.webkit.org/changeset/57915
http://trac.webkit.org/changeset/57916
http://trac.webkit.org/changeset/57917
http://trac.webkit.org/changeset/57918

------- Comment #5 From Eric Seidel 2010-04-20 22:42:15 PST -------

I think this broke Snow Leopard.