Fix lack of preview when GCP printer is selected.

Fix lack of preview when GCP printer is selected.
Add handling for the cloud print dialog to update settings code on Mac and Windows.

BUG=98929
TEST=


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

[kmadhusu, 2011-10-03 22:15:10]

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_mac.mm
File printing/printing_context_mac.mm (right):

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_mac.m...
printing/printing_context_mac.mm:130:
job_settings.GetBoolean(printing::kSettingCloudPrintDialog, &is_cloud_dialog);
Please add this line to the previous if() condition.

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_win.cc
File printing/printing_context_win.cc (right):

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_win.c...
printing/printing_context_win.cc:349:
job_settings.GetBoolean(printing::kSettingCloudPrintDialog, &is_cloud_dialog);
same here.

[Albert Bodenhamer, 2011-10-03 22:38:39]

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_mac.mm
File printing/printing_context_mac.mm (right):

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_mac.m...
printing/printing_context_mac.mm:130:
job_settings.GetBoolean(printing::kSettingCloudPrintDialog, &is_cloud_dialog);
On 2011/10/03 22:15:10, kmadhusu wrote:
> Please add this line to the previous if() condition. 

Done.

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_win.cc
File printing/printing_context_win.cc (right):

http://codereview.chromium.org/8122006/diff/1/printing/printing_context_win.c...
printing/printing_context_win.cc:349:
job_settings.GetBoolean(printing::kSettingCloudPrintDialog, &is_cloud_dialog);
On 2011/10/03 22:15:10, kmadhusu wrote:
> same here.

Done.

[kmadhusu, 2011-10-03 22:43:03]

lgtm

[commit-bot: I haz the power, 2011-10-04 16:46:15]

CQ is trying the patch. Follow status at
https://chromium-status.appspot.com/cq/abodenha@chromium.org/8122006/3002

[commit-bot: I haz the power, 2011-10-04 18:06:31]

Change committed as 103937