Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Issue 1157993005: Give RenderObject a useful toString(). (Closed)

Created:
5 years, 6 months ago by Hixie
Modified:
5 years, 6 months ago
Reviewers:
eseidel
CC:
abarth-chromium, gregsimon, jackson_old, mojo-reviews_chromium.org, qsr+mojo_chromium.org
Base URL:
https://github.com/domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Give RenderObject a useful toString(). This makes debugging the render tree a lot easier. Just print the node you care about, and you get an indented tree view of its subtree, including settings. New subclasses should implement the new virtual method debugDescribeSettings() to expose new settings. R=eseidel@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/f2326b33aa651853671a7893c088631e9466c310

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -0 lines) Patch
M sky/sdk/lib/framework/components2/scaffold.dart View 1 chunk +3 lines, -0 lines 0 comments Download
M sky/sdk/lib/framework/rendering/box.dart View 6 chunks +19 lines, -0 lines 0 comments Download
M sky/sdk/lib/framework/rendering/flex.dart View 1 chunk +1 line, -0 lines 0 comments Download
M sky/sdk/lib/framework/rendering/object.dart View 4 chunks +28 lines, -0 lines 0 comments Download
M sky/sdk/lib/framework/rendering/paragraph.dart View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
eseidel
lgtm
5 years, 6 months ago (2015-06-04 20:59:36 UTC) #2
Hixie
5 years, 6 months ago (2015-06-04 21:10:12 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f2326b33aa651853671a7893c088631e9466c310 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698