PrintPreview:[MAC] Added "Open PDF In Preview" option to open the pdf in native preview app.

PrintPreview:[MAC] Added "Open PDF In Preview" option to open the pdf in native preview app.

BUG=101724
TEST=Preview a webpage. Click "Open PDF In Preview" option Observe that the pdf is displayed in native preview app. Complete the printing workflow by clicking "Print" or 
"Cancel" button at the bottom of the preview app.

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

[csilv, 2011-11-14 19:34:14]

lgtm

http://codereview.chromium.org/8425008/diff/12012/chrome/app/generated_resour...
File chrome/app/generated_resources.grd (right):

http://codereview.chromium.org/8425008/diff/12012/chrome/app/generated_resour...
chrome/app/generated_resources.grd:6338: Opening PDF in Preview
This should probably end in "..." like "Printing..."

http://codereview.chromium.org/8425008/diff/12012/chrome/app/generated_resour...
chrome/app/generated_resources.grd:6429: Open PDF in Preview
This should probably end in "..." like the similar string "Print using system
dialog..."

http://codereview.chromium.org/8425008/diff/12012/chrome/browser/resources/pr...
File chrome/browser/resources/print_preview/print_preview.js (right):

http://codereview.chromium.org/8425008/diff/12012/chrome/browser/resources/pr...
chrome/browser/resources/print_preview/print_preview.js:159:
'click',onOpenPdfInPreviewLinkClicked);
nit: space after comma, indent 4 spaces

[kmadhusu, 2011-11-16 17:59:40]

Fixed nits and addressed comments.

Thanks.

http://codereview.chromium.org/8425008/diff/12012/chrome/app/generated_resour...
File chrome/app/generated_resources.grd (right):

http://codereview.chromium.org/8425008/diff/12012/chrome/app/generated_resour...
chrome/app/generated_resources.grd:6338: Opening PDF in Preview
On 2011/11/14 19:34:14, csilv wrote:
> This should probably end in "..." like "Printing..."

We add the dancing dots in the code :)

http://codereview.chromium.org/8425008/diff/12012/chrome/app/generated_resour...
chrome/app/generated_resources.grd:6429: Open PDF in Preview
On 2011/11/14 19:34:14, csilv wrote:
> This should probably end in "..." like the similar string "Print using system
> dialog..."

UI leads preferred this way (without dots).

http://codereview.chromium.org/8425008/diff/12012/chrome/browser/resources/pr...
File chrome/browser/resources/print_preview/print_preview.js (right):

http://codereview.chromium.org/8425008/diff/12012/chrome/browser/resources/pr...
chrome/browser/resources/print_preview/print_preview.js:159:
'click',onOpenPdfInPreviewLinkClicked);
On 2011/11/14 19:34:14, csilv wrote:
> nit: space after comma, indent 4 spaces

Done.

[commit-bot: I haz the power, 2011-11-16 18:13:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kmadhusu@chromium.org/8425008/17001

[commit-bot: I haz the power, 2011-11-16 19:15:58]

Try job failure for 8425008-17001 (retry) on linux_rel for step "browser_tests".
It's a second try, previously, steps "browser_tests, ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2011-11-16 22:40:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kmadhusu@chromium.org/8425008/21001

[commit-bot: I haz the power, 2011-11-16 23:56:04]

Try job failure for 8425008-21001 (retry) (retry) on linux_rel for step
"unit_tests".
It's a second try, previously, step "unit_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...