Move static member functions next{Previous,Next}Offset() and uncheckedPreviousOffsetForBackwardDelet

Move static member functions next{Previous,Next}Offset() and uncheckedPreviousOffsetForBackwardDeletion() out from PositionAlgorithm<Strategy> template class

This patch moves a member function |next{Previous,Next}Offset()| and
|uncheckedPreviousOffsetForBackwardDeletion()| out from
|PositionAlgorithm<Strategy>| template class as non-member functions since
they are independent from |Strategy| template parameter to simplify class
declaration for improving code halth.

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

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

[yosin_UTC9, 2015-08-19 05:59:10]

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

[yosin_UTC9, 2015-08-19 05:59:11]

PTAL

[yoichio, 2015-08-19 06:07:15]

How about moving to PositionIterator.cpp as static function because only it uses
them?

[yosin_UTC9, 2015-08-19 06:27:40]

On 2015/08/19 06:07:15, yoichio wrote:
> How about moving to PositionIterator.cpp as static function because only it
uses
> them?

They are also used in Position::next() and Position::previous(). So, we can't
move
them as static functions in PositionIterator.cpp.

[yoichio, 2015-08-20 04:25:58]

lgtm

[yosin_UTC9, 2015-08-20 04:41:07]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-08-20 04:41:10]

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

[commit-bot: I haz the power, 2015-08-20 04:42:23]

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

[commit-bot: I haz the power, 2015-08-20 04:42:24]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[yosin_UTC9, 2015-08-20 05:29:05]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2015-08-20 05:29:05]

The patchset sent to the CQ was uploaded after l-g-t-m from yoichio@chromium.org
Link to the patchset: https://codereview.chromium.org/1301023003/#ps20001
(title: "2015-08-20T14:28:12 Rebase")

[commit-bot: I haz the power, 2015-08-20 05:29:13]

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

[commit-bot: I haz the power, 2015-08-20 05:59:15]

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

[commit-bot: I haz the power, 2015-08-20 05:59:15]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)

[yosin_UTC9, 2015-08-20 08:26:20]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-08-20 08:26:35]

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

[commit-bot: I haz the power, 2015-08-20 09:27:04]

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