Cleaned up the print preview API used by app printing browser tests

Cleaned up the print preview API used by app printing browser tests

Added an anonymous namespace to make a couple of methods of this testing
API private.

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223363

[dharcourt, 2013-09-15 10:13:37]

This is a small cleaning up follow-up on http://crrev.com/24104004.

As always, thanks for the review!

    - Charles

[Vitaly Buka (NO REVIEWS), 2013-09-15 22:56:49]

lgtm

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
File chrome/browser/ui/webui/print_preview/print_preview_ui.cc (right):

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
chrome/browser/ui/webui/print_preview/print_preview_ui.cc:341: }  // namespace
Please remove lines 341-343

[not at google - send to devlin, 2013-09-16 13:45:06]

lgtm, thanks for following that up Charles!

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
File chrome/browser/ui/webui/print_preview/print_preview_ui.cc (right):

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
chrome/browser/ui/webui/print_preview/print_preview_ui.cc:345: static int
g_auto_cancel_count_for_testing_ = -1;  // Disabled if count < 0.
doesn't need to be static anymore

[dharcourt, 2013-09-16 15:49:01]

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
File chrome/browser/ui/webui/print_preview/print_preview_ui.cc (right):

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
chrome/browser/ui/webui/print_preview/print_preview_ui.cc:341: }  // namespace
On 2013/09/15 22:56:49, Vitaly Buka wrote:
> Please remove lines 341-343

Done.

https://codereview.chromium.org/24158002/diff/1/chrome/browser/ui/webui/print...
chrome/browser/ui/webui/print_preview/print_preview_ui.cc:345: static int
g_auto_cancel_count_for_testing_ = -1;  // Disabled if count < 0.
On 2013/09/16 13:45:06, kalman wrote:
> doesn't need to be static anymore

Done.

[commit-bot: I haz the power, 2013-09-16 15:49:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dharcourt@chromium.org/24158002/12001

[commit-bot: I haz the power, 2013-09-16 18:22:37]

Change committed as 223363