Andreas Tolfsen ❲:ato❳ Assignee
	Description • 7 years ago

In certain places, the Marionette action module calls error.pprint
as if it is a function. pprint is a ES6 string template and should
be used like pprint`${replacement}`.

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 3 • 7 years ago

This doesn’t block marionette-window-tracking so much, but I am
trying to split up that patch into landable chunks.

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 4 • 7 years ago

(In reply to Andreas Tolfsen ‹:ato› from comment #3)
> This doesn’t block marionette-window-tracking so much, but I am
> trying to split up that patch into landable chunks.

Now it does block it, so I’m slightly overloading the meaning of
a blocking bug to keep track of the individual chunks.

	David Burns :automatedtester
	Comment 6 • 7 years ago
mozreview-review

Comment on attachment 8899540 [details]
Bug 1392323 - Correct use of pprint in action module.

https://reviewboard.mozilla.org/r/170828/#review176340

	Pulsebot
	Comment 7 • 7 years ago

Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/85c71fd3d6aa
Correct use of pprint in action module. r=automatedtester

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 8 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/85c71fd3d6aa