Print Preview Refresh: Fix some pages and link container polish bugs

- Make cr-input full height of radio button so there is no border
area where the hand pointer appears.
- Ignore pointer events on link container when disabled.
- Fix bug where "----" was considered valid input in pages.

Bug: 884603, 887823, 888493
Change-Id: Ie9b7f2c42b12d7e2a7026f405b1a61b774578589
Reviewed-on: https://chromium-review.googlesource.com/1242037Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594013}

chrome/browser/resources/print_preview/new/link_container.html

@@ -9,6 +9,10 @@
 <dom-module id="print-preview-link-container">
   <template>
     <style include="print-preview-shared throbber cr-hidden-style">
+      :host([disabled]) {
+        pointer-events: none;
+      }
+
       .throbber {
         margin: 8px;
         min-height: 16px;

chrome/browser/resources/print_preview/new/pages_settings.html

@@ -31,6 +31,7 @@
 
       #pageSettingsCustomInput {
         cursor: default;
+        height: 100%;
       }
     </style>
     <print-preview-settings-section

chrome/browser/resources/print_preview/new/pages_settings.js

@@ -139,7 +139,14 @@ Polymer({
         this.onRangeChange_();
         return this.pagesToPrint_;
       }
+
       const limits = range.split('-');
+      if (limits.length > 2) {
+        this.errorState_ = PagesInputErrorState.INVALID_SYNTAX;
+        this.onRangeChange_();
+        return this.pagesToPrint_;
+      }
+
       let min = Number.parseInt(limits[0], 10);
       if ((limits[0].length > 0 && Number.isNaN(min)) || min < 1) {
         this.errorState_ = PagesInputErrorState.INVALID_SYNTAX;

chrome/test/data/webui/print_preview/pages_settings_test.js

@@ -172,6 +172,10 @@ cr.define('pages_settings_test', function() {
             validateErrorState(syntaxError);
             return setupInput('-1,1,2\u3001\u300156', 100);
           })
+          .then(function() {
+            validateErrorState(syntaxError);
+            return setupInput('--', 100);
+          })
           .then(function() {
             validateErrorState(syntaxError);
           });