Need a better debug approach for PaintArtifacts |
||
Issue description
I keep re-writing the same debug code and others on the team are as well.
I think we should add the following:
String PaintArtifact::toString() {
StringBuilder sb;
sb.append("DisplayItems:");
for (auto& item : m_displayItemList) {
sb.append([index]);
sb.append(item.toString();
}
sb.append("PaintChunks:");
for (auto& chunk : paintChunks)
sb.append(chunk.toString());
sb.append("Property trees:");
// Collect and print the trees, similar to PaintPropertyTreePrinter::showAllPropertyTrees
}
String PaintChunk::toString() {
begin index, end index, and pointers for the 4 property tree nodes
}
,
Sep 25 2017
Is this done now?
,
Sep 25 2017
We added a lot more logging so this is mostly done, but I think I still need a ToString on PaintArtifact. Will add. |
||
►
Sign in to add a comment |
||
Comment 1 by sheriffbot@chromium.org
, Sep 25 2017Status: Untriaged (was: Available)