PrintPreview Cleanup: Rename and refactor getSelectedPagesValidity() function.

PrintPreview Cleanup: Rename and refactor getSelectedPagesValidity() function.

BUG=none
TEST=Preview a webpage. Enter a page range text. If the text is invalid, an error message is displayed.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88402

[dpapad, 2011-06-08 18:58:13]

http://codereview.chromium.org/7003060/diff/1/chrome/browser/resources/print_...
File chrome/browser/resources/print_preview.js (right):

http://codereview.chromium.org/7003060/diff/1/chrome/browser/resources/print_...
chrome/browser/resources/print_preview.js:800: var successfullyParsed = 0;
We don't need this variable either anymore. Remove and simply return true at the
end of the function.

[dpapad, 2011-06-08 20:10:06]

LGTM

http://codereview.chromium.org/7003060/diff/1/chrome/browser/resources/print_...
File chrome/browser/resources/print_preview.js (right):

http://codereview.chromium.org/7003060/diff/1/chrome/browser/resources/print_...
chrome/browser/resources/print_preview.js:800: var successfullyParsed = 0;
On 2011/06/08 18:58:13, dpapad wrote:
> We don't need this variable either anymore. Remove and simply return true at
the
> end of the function.

Summarizing our online discussion, lets leave this as is for now to allow ranges
like 1,,,5 to be valid.

[commit-bot: I haz the power, 2011-06-08 21:10:00]

Change committed as 88402