Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/trace: traceviewer tool doesn't work in non-chrome browsers #19207

Closed
narqo opened this issue Feb 20, 2017 · 1 comment
Closed

cmd/trace: traceviewer tool doesn't work in non-chrome browsers #19207

narqo opened this issue Feb 20, 2017 · 1 comment
Labels
Documentation FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@narqo
Copy link
Contributor

narqo commented Feb 20, 2017

What version of Go are you using (go version)?

go version go1.8

What operating system and processor architecture are you using (go env)?

darwin/amd64

What did you do?

Call go tool trace trace.file as described in https://godoc.org/cmd/trace and open the "View trace" link using Safari (or other non-Chrome) web browser.

What did you expect to see?

Expected to see the Trace viewer.

What did you see instead?

There is a JavaScript error, saying "ReferenceError: Can't find variable: tr" in the console of the browser.

I know that the trace-viewer itself comes from the catapult project, as well as that the project was built specifically for Chrome and/or Android, but I think it would be worth to add a notice to cmd/trace documentation about this fact.

@ALTree ALTree changed the title traceviewer tool doesn't work in non-chrome browsers cmd/trace: traceviewer tool doesn't work in non-chrome browsers Feb 21, 2017
@ALTree ALTree added Documentation NeedsFix The path to resolution is known, but the work has not been done. labels Feb 21, 2017
@ALTree ALTree added this to the Go1.9 milestone Feb 21, 2017
@gopherbot
Copy link

CL https://golang.org/cl/37312 mentions this issue.

@golang golang locked and limited conversation to collaborators Apr 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Documentation FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

3 participants