Make FrameSelection::create() to take LocalFrame as required parameter

Make FrameSelection::create() to take LocalFrame as required parameter

This patch makes |FrameSelection::create()| to take |LocalFrame| as required
parameter instead of optional parameter by utilizing |SelectionModifier| to
get rid of null check for |m_frame| member variable in |FrameSelection| class
for improving code health.

Following patch will get rid of null check for |m_frame| in |FrameSelection|.

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

Committed: https://crrev.com/86d103eaec6f31e3c11d1ea7698231f4a589b871
Cr-Commit-Position: refs/heads/master@{#394652}

[yosin_UTC9, 2016-05-18 09:52:37]

Description was changed from

==========
2016-05-18T18:48:47

BUG=
==========

to

==========
Make FrameSelection::create() to take LocalFrame as required parameter

This patch makes |FrameSelection::create()| to take |LocalFrame| as required
parameter instead of optional parameter by utilizing |SelectionModifier| to
get rid of null check for |m_frame| member variable in |FrameSelection| class
for improving code health.

Following patch will get rid of null check for |m_frame| in |FrameSelection|.

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

[yosin_UTC9, 2016-05-18 09:52:37]

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

[yosin_UTC9, 2016-05-18 09:53:08]

PTAL

[yoichio, 2016-05-19 00:58:43]

lgtm

cool!

[yosin_UTC9, 2016-05-19 01:04:55]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-05-19 01:05:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1987973003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1987973003/1

[commit-bot: I haz the power, 2016-05-19 03:24:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-19 03:25:47]

Description was changed from

==========
Make FrameSelection::create() to take LocalFrame as required parameter

This patch makes |FrameSelection::create()| to take |LocalFrame| as required
parameter instead of optional parameter by utilizing |SelectionModifier| to
get rid of null check for |m_frame| member variable in |FrameSelection| class
for improving code health.

Following patch will get rid of null check for |m_frame| in |FrameSelection|.

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

to

==========
Make FrameSelection::create() to take LocalFrame as required parameter

This patch makes |FrameSelection::create()| to take |LocalFrame| as required
parameter instead of optional parameter by utilizing |SelectionModifier| to
get rid of null check for |m_frame| member variable in |FrameSelection| class
for improving code health.

Following patch will get rid of null check for |m_frame| in |FrameSelection|.

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

Committed: https://crrev.com/86d103eaec6f31e3c11d1ea7698231f4a589b871
Cr-Commit-Position: refs/heads/master@{#394652}
==========

[commit-bot: I haz the power, 2016-05-19 03:25:48]

Patchset 1 (id:??) landed as
https://crrev.com/86d103eaec6f31e3c11d1ea7698231f4a589b871
Cr-Commit-Position: refs/heads/master@{#394652}