Apply automatic range checks to content enum types across IPC.

Apply automatic range checks to content enum types across IPC.

This is part of a long-running background task to remove the remaining
uses of the unchecked IPC_ENUM_TRAITS() macro.

BUG=246708
Committed: https://crrev.com/098510bfbef457641abed93223f5294c13c2cd74
Cr-Commit-Position: refs/heads/master@{#298923}

[anujsharma, 2014-10-08 10:37:52]

anujk.sharma@samsung.com changed reviewers:
+ nasko@chromium.org, vitalybuka@chromium.org

[anujsharma, 2014-10-08 10:37:53]

vitalybuka@chromium.org: Please review changes in printing module

nasko@chromium.org: Please review changes in chrome module

[nasko, 2014-10-08 17:20:51]

https://codereview.chromium.org/635253002/diff/1/chrome/common/chrome_utility...
File chrome/common/chrome_utility_printing_messages.h (right):

https://codereview.chromium.org/635253002/diff/1/chrome/common/chrome_utility...
chrome/common/chrome_utility_printing_messages.h:53:
printing::PWG_RASTER_TRANSFORM_TYPE_LAST)
Please align vertically the two lines. Refer to the style guide for details.

[Vitaly Buka (NO REVIEWS), 2014-10-08 17:40:05]

https://codereview.chromium.org/635253002/diff/1/printing/pwg_raster_settings.h
File printing/pwg_raster_settings.h (right):

https://codereview.chromium.org/635253002/diff/1/printing/pwg_raster_settings...
printing/pwg_raster_settings.h:20: PWG_RASTER_TRANSFORM_TYPE_LAST =
TRANSFORM_FLIP_VERTICAL
TRANSFORM_TYPE_LAST

[anujsharma, 2014-10-09 02:37:51]

On 2014/10/08 17:40:05, Vitaly Buka wrote:
>
https://codereview.chromium.org/635253002/diff/1/printing/pwg_raster_settings.h
> File printing/pwg_raster_settings.h (right):
> 
>
https://codereview.chromium.org/635253002/diff/1/printing/pwg_raster_settings...
> printing/pwg_raster_settings.h:20: PWG_RASTER_TRANSFORM_TYPE_LAST =
> TRANSFORM_FLIP_VERTICAL
> TRANSFORM_TYPE_LAST

Thanks nasko and vitaly Buka for your comments.
I have made changes as per your suggestions.
Please review the updates.

[Vitaly Buka (NO REVIEWS), 2014-10-09 03:59:08]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2014-10-09 03:59:09]

lgtm

[commit-bot: I haz the power, 2014-10-09 04:01:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/635253002/20001

[commit-bot: I haz the power, 2014-10-09 04:11:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-09 04:11:48]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[anujsharma, 2014-10-09 04:25:14]

On 2014/10/09 04:11:48, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
>
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    chrome/common/chrome_utility_printing_messages.h

Need one more LGTM :(
@nasko, 
Could you please give thumbs up for this CL.

[nasko, 2014-10-09 16:44:14]

lgtm

[anujsharma, 2014-10-09 16:51:38]

On 2014/10/09 16:44:14, nasko wrote:
> lgtm

Thanks nasko for lgtm.

[anujsharma, 2014-10-09 16:51:51]

The CQ bit was checked by anujk.sharma@samsung.com

[commit-bot: I haz the power, 2014-10-09 16:52:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/635253002/20001

[commit-bot: I haz the power, 2014-10-09 17:25:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-09 17:26:08]

Patchset 2 (id:??) landed as
https://crrev.com/098510bfbef457641abed93223f5294c13c2cd74
Cr-Commit-Position: refs/heads/master@{#298923}