Fix bug in graph visualizer.

Fix bug in graph visualizer.

R=mstarzinger@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=23622

[sigurds, 2014-09-02 14:23:10]

As reported offline.

[Michael Starzinger, 2014-09-02 14:34:03]

LGTM.

https://codereview.chromium.org/526373002/diff/1/src/compiler/graph-visualize...
File src/compiler/graph-visualizer.cc (right):

https://codereview.chromium.org/526373002/diff/1/src/compiler/graph-visualize...
src/compiler/graph-visualizer.cc:215: << (NodeProperties::IsContextEdge(edge) ?
"style=dashed, " : "") << "]";
Are trailing slashes OK for GraphViz? I am not a GraphViz ninja.

[Michael Starzinger, 2014-09-02 14:34:33]

https://codereview.chromium.org/526373002/diff/1/src/compiler/graph-visualize...
File src/compiler/graph-visualizer.cc (right):

https://codereview.chromium.org/526373002/diff/1/src/compiler/graph-visualize...
src/compiler/graph-visualizer.cc:215: << (NodeProperties::IsContextEdge(edge) ?
"style=dashed, " : "") << "]";
On 2014/09/02 14:34:03, Michael Starzinger wrote:
> Are trailing slashes OK for GraphViz? I am not a GraphViz ninja.

s/slashes/commas/

[sigurds, 2014-09-02 14:42:15]

On 2014/09/02 14:34:33, Michael Starzinger wrote:
>
https://codereview.chromium.org/526373002/diff/1/src/compiler/graph-visualize...
> File src/compiler/graph-visualizer.cc (right):
> 
>
https://codereview.chromium.org/526373002/diff/1/src/compiler/graph-visualize...
> src/compiler/graph-visualizer.cc:215: << (NodeProperties::IsContextEdge(edge)
?
> "style=dashed, " : "") << "]";
> On 2014/09/02 14:34:03, Michael Starzinger wrote:
> > Are trailing slashes OK for GraphViz? I am not a GraphViz ninja.
> 
> s/slashes/commas/

I tested with dot on my machine, there dot accepts trailing commas.

[sigurds, 2014-09-02 14:59:50]

Committed patchset #1 manually as 23622 (presubmit successful).