Update print preview test to not use CallJavascriptFunctionUnsafe

Update print preview test to not use CallJavascriptFunctionUnsafe

Manual test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
Review-Url: https://codereview.chromium.org/2978673002
Cr-Commit-Position: refs/heads/master@{#485767}
Committed: https://chromium.googlesource.com/chromium/src/+/82a8567da54863660f1743b70a8bfb9d79658905

[rbpotter, 2017-07-11 19:33:20]

Description was changed from

==========
Update disabled test to not use CallJavascriptFunctionUnsafe

Disabled test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

to

==========
Update disabled test to not use CallJavascriptFunctionUnsafe

Disabled test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

[rbpotter, 2017-07-11 19:33:20]

rbpotter@chromium.org changed reviewers:
+ dpapad@chromium.org, thestig@chromium.org

[rbpotter, 2017-07-11 19:33:24]

The CQ bit was checked by rbpotter@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-11 19:33:56]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dpapad, 2017-07-11 21:09:57]

Is this test disabled because it was failing on some bots, or is it a manual
test on purpose?

[rbpotter, 2017-07-11 21:37:50]

Description was changed from

==========
Update disabled test to not use CallJavascriptFunctionUnsafe

Disabled test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

to

==========
Update test to not use CallJavascriptFunctionUnsafe

Manual test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

[rbpotter, 2017-07-11 21:38:01]

On 2017/07/11 21:09:57, dpapad wrote:
> Is this test disabled because it was failing on some bots, or is it a manual
> test on purpose?


Apparently it is an intentionally manual test that is not generally run.
I had confused it with the tests that were disabled here 
https://crbug.com/7721 called "PrintingLayoutTests", which have been
disabled for some time. Modified description accordingly.

[Lei Zhang, 2017-07-11 21:51:11]

lgtm

Can we mention "print preview test" in first line of the CL description?

The tests can be manually enable to do pixel tests for print preview.

[rbpotter, 2017-07-11 21:53:28]

Description was changed from

==========
Update test to not use CallJavascriptFunctionUnsafe

Manual test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

to

==========
Update print preview test to not use CallJavascriptFunctionUnsafe

Manual test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

[commit-bot: I haz the power, 2017-07-11 22:30:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-07-11 22:30:04]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build has not started yet; builder either lacks capacity or does
not exist (misspelled?))

[dpapad, 2017-07-11 23:32:50]

lgtm

[rbpotter, 2017-07-12 00:08:33]

The CQ bit was checked by rbpotter@chromium.org

[commit-bot: I haz the power, 2017-07-12 00:09:30]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-12 01:26:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499818113473790, "parent_rev":
"5ffe4dd335accb5a9172c39a8a2f91284e5bf96d", "commit_rev":
"82a8567da54863660f1743b70a8bfb9d79658905"}

[commit-bot: I haz the power, 2017-07-12 01:27:13]

Description was changed from

==========
Update print preview test to not use CallJavascriptFunctionUnsafe

Manual test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296
==========

to

==========
Update print preview test to not use CallJavascriptFunctionUnsafe

Manual test uses CallJavascriptFunctionUnsafe. Print preview has been
updated to use WebUI listener for this function instead, so update the
test to match.

BUG=717296

Review-Url: https://codereview.chromium.org/2978673002
Cr-Commit-Position: refs/heads/master@{#485767}
Committed:
https://chromium.googlesource.com/chromium/src/+/82a8567da54863660f1743b70a8b...
==========

[commit-bot: I haz the power, 2017-07-12 01:27:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/82a8567da54863660f1743b70a8b...