Enable page selection for WebView printing

android_webview/renderer/aw_print_web_view_helper_delegate.cc

Index: android_webview/renderer/aw_print_web_view_helper_delegate.cc
diff --git a/android_webview/renderer/aw_print_web_view_helper_delegate.cc b/android_webview/renderer/aw_print_web_view_helper_delegate.cc
index ccb7b2afad1728ff35e7ca71224e5c533e3891ac..2eb7214e3cedc7bf5e0dcf2c9ec78514eadae350 100644
--- a/android_webview/renderer/aw_print_web_view_helper_delegate.cc
+++ b/android_webview/renderer/aw_print_web_view_helper_delegate.cc
@@ -25,7 +25,7 @@ bool AwPrintWebViewHelperDelegate::IsPrintPreviewEnabled() {
 }
 
 bool AwPrintWebViewHelperDelegate::IsAskPrintSettingsEnabled() {
-  return false;
+  return true;

[sgurun-gerrit only, 2017/03/23 19:16:59]

two comments:

I think this is to enable browser asking print settings from user, or at least
that was the original intent in desktop, which does not apply to WebView. Can
you update the comment about showing UI in
https://cs.chromium.org/chromium/src/components/printing/renderer/print_web_v...

also the base class has  default implementation which  true, so you can now
remove this method altogether.

[Lei Zhang, 2017/03/23 19:35:18]

Yes, that was the original intent, but this is fine. The browser side code can
handle the "ask for settings" task however it likes. Better comments are
welcome.

[sgurun-gerrit only, 2017/03/23 19:38:18]

alright then, no worries about updating comment.

[Shimi Zhang, 2017/03/23 23:21:56]

Done.

 }
 
 bool AwPrintWebViewHelperDelegate::IsScriptedPrintEnabled() {