Templatize FrameSelection::contains function

Templatize FrameSelection::contains function

This patch introduces a template function |containsAlgorithm<Strategy>|
to makes |FrameSelection::contains()| function to work
both on DOM tree and composed tree with introducing member functions to strategy
class |VisibleSelection::InDOMTree|.

This patch is result of collaboration work with hajimehoshi@chromium.org
for selection of web components. After this patch, we are going to
implement another strategies for the composed tree. We've already
prepared that at http://crrev.com/1106433002.

BUG=275851
TEST=n/a; No behavior changes.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197342

[yosin_UTC9, 2015-06-18 05:46:34]

Patchset #1 (id:1) has been deleted

[yosin_UTC9, 2015-06-18 06:18:15]

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

[yosin_UTC9, 2015-06-18 06:18:15]

PTAL

[yosin_UTC9, 2015-06-18 07:28:54]

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

[yosin_UTC9, 2015-06-18 07:28:55]

PTAL

[tkent, 2015-06-18 07:43:35]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-06-18 07:43:35]

lgtm

[commit-bot: I haz the power, 2015-06-18 07:44:19]

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

[commit-bot: I haz the power, 2015-06-18 07:49:07]

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

[commit-bot: I haz the power, 2015-06-18 07:49:08]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/59449)

[yoichio, 2015-06-18 07:59:14]

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

[yoichio, 2015-06-18 07:59:15]

https://codereview.chromium.org/1182253004/diff/20001/Source/core/editing/Fra...
File Source/core/editing/FrameSelection.cpp (right):

https://codereview.chromium.org/1182253004/diff/20001/Source/core/editing/Fra...
Source/core/editing/FrameSelection.cpp:1296: template <typename Strategy>
VisibleSelection::InDOMTree does not look like other strategy classes.
What does 'Strategy' mean?

[yosin_UTC9, 2015-06-18 10:16:07]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2015-06-18 10:16:08]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1182253004/#ps40001
(title: "2015-06-18T19:15:03 Rebase")

[commit-bot: I haz the power, 2015-06-18 10:16:44]

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

[yosin_UTC9, 2015-06-18 10:17:22]

On 2015/06/18 07:59:15, yoichio wrote:
>
https://codereview.chromium.org/1182253004/diff/20001/Source/core/editing/Fra...
> File Source/core/editing/FrameSelection.cpp (right):
> 
>
https://codereview.chromium.org/1182253004/diff/20001/Source/core/editing/Fra...
> Source/core/editing/FrameSelection.cpp:1296: template <typename Strategy>
> VisibleSelection::InDOMTree does not look like other strategy classes.
> What does 'Strategy' mean?

"Strategy" comes from "Strategy pattern" in design-pattern,
https://en.wikipedia.org/wiki/Strategy_pattern

[commit-bot: I haz the power, 2015-06-18 11:36:30]

Committed patchset #2 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197342