Copy SpellChecker.{cpp,h} to SpellCheckRequester.{cpp.h}

Copy SpellChecker.{cpp,h} to SpellCheckRequester.{cpp.h}

Regarding ojan@'s suggestion(https://codereview.chromium.org/22732003/), this patch reverts r155831 which copies SpellChcker.{cpp,h} to SpellCheckerRequestManager.{cpp,h}, and copies SpellChecker.{cpp,h} to SpellCheckRequester.{cpp.h} for preparation of renaming SpellChecker class to SpellCheckRequester, to make class name more explicitly denotes what it does, and free its name to use name SpellChecker for decoupling spell checking and Editor class.

Note: Differences between original file and new file are for style checking in pre-submit.

BUG=n/a

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

[yosin_UTC9, 2013-08-12 02:05:39]

Could you review this patch?
Thanks in advance.

We'll use SpellCheckRequester rather than SpellCheckRequestManager. Blink
doesn't want to use term "manager".

[ojan, 2013-08-12 02:38:31]

lgtm. thanks!


On Sun, Aug 11, 2013 at 7:05 PM, <yosin@chromium.org> wrote:

> Reviewers: tkent,
>
> Message:
> Could you review this patch?
> Thanks in advance.
>
> We'll use SpellCheckRequester rather than SpellCheckRequestManager. Blink
> doesn't want to use term "manager".
>
> Description:
> Copy SpellChecker.{cpp,h} to SpellCheckRequester.{cpp.h}
>
> Regarding ojan@'s
suggestion(https://codereview.**chromium.org/22732003/<https://codereview.chromium.org/22732003/>),
> this
> patch reverts r155831 which copies SpellChcker.{cpp,h} to
> SpellCheckerRequestManager.{**cpp,h}, and copies SpellChecker.{cpp,h} to
> SpellCheckRequester.{cpp.h} for preparation of renaming SpellChecker class
> to
> SpellCheckRequester, to make class name more explicitly denotes what it
> does,
> and free its name to use name SpellChecker for decoupling spell checking
> and
> Editor class.
>
> Note: Differences between original file and new file are for style
> checking in
> pre-submit.
>
> BUG=n/a
>
>
> Please review this at
https://codereview.chromium.**org/22475010/<https://codereview.chromium.org/2...
>
> SVN Base:
svn://svn.chromium.org/blink/**trunk<http://svn.chromium.org/blink/trunk>
>
> Affected files:
>   D Source/core/editing/**SpellCheckRequestManager.h
>   D Source/core/editing/**SpellCheckRequestManager.cpp
>   A + Source/core/editing/**SpellCheckRequester.h
>   A + Source/core/editing/**SpellCheckRequester.cpp
>
>
>

[yosin_UTC9, 2013-08-12 03:22:43]

Committed patchset #1 manually as r155920 (presubmit successful).