Refactoring: Replace boolean parameters with enum classes

Refactoring: Replace boolean parameters with enum classes

This CL replaces boolean parameters with enum classes newly introduced.

BUG=n/a
TEST=n/a; no behavior change

[hajimehoshi, 2015-06-02 03:42:31]

hajimehoshi@chromium.org changed reviewers:
+ yosin@chromium.org

[hajimehoshi, 2015-06-02 03:42:32]

PTAL

[hajimehoshi, 2015-06-02 03:43:19]

hajimehoshi@chromium.org changed reviewers:
+ tkent@chromium.org

[hajimehoshi, 2015-06-02 03:43:20]

+tkent for core/xml

[yosin_UTC9, 2015-06-02 03:47:04]

Could you split this patch into two patches?
 1 SerializationType
 2 RangeFullySelectsNode, IsBlock, AddDisplayInline

How about member of enum class to use Yes No? Just asking not suggestion.

[hajimehoshi, 2015-06-02 03:51:34]

On 2015/06/02 03:47:04, Yosi_UTC9 wrote:
> Could you split this patch into two patches?
>  1 SerializationType
>  2 RangeFullySelectsNode, IsBlock, AddDisplayInline

Sure.

> How about member of enum class to use Yes No? Just asking not suggestion.

I prefer your suggestion but I'm worried that there is not such usages in Blink.

[hajimehoshi, 2015-06-02 03:58:25]

On 2015/06/02 03:51:34, hajimehoshi wrote:
> On 2015/06/02 03:47:04, Yosi_UTC9 wrote:
> > Could you split this patch into two patches?
> >  1 SerializationType
> >  2 RangeFullySelectsNode, IsBlock, AddDisplayInline
> 
> Sure.

Created the patch for SerializationType
(https://codereview.chromium.org/1157363007/). Now I'm fixing this patch.

[hajimehoshi, 2015-06-02 03:58:39]

hajimehoshi@chromium.org changed reviewers:
- tkent@chromium.org

[hajimehoshi, 2015-06-02 03:58:39]

-tkent

[hajimehoshi, 2015-06-02 05:17:09]

This CL now includes changes only for RangeFullySelectsNode, IsBlock,
AddDisplayInline. PTAL