Copy WebFrameImpl.{cpp,h} to TextFinder.{cpp,h}

Copy WebFrameImpl.{cpp,h} to TextFinder.{cpp,h}

This patch copies Source/web/WebFrameImpl.{cpp,h} into Source/web/TextFinder.{cpp,h} for decoupling WebFrameImpl and text finder.

Another patch will complete decoupling.

BUG=

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

[yosin_UTC9, 2013-11-14 01:42:44]

ACK

+tkent for committing

[tkent, 2013-11-14 01:50:38]

I feel the name 'InFrameFinder' doesn't represent the role well.

> ... for decoupling WebFrameImpl and text finder. 

You say 'text finder', so the name should be TextFinder, or something.

[Andrey Kraynov, 2013-11-26 08:53:03]

On 2013/11/14 01:50:38, tkent wrote:
> I feel the name 'InFrameFinder' doesn't represent the role well.
> 
> > ... for decoupling WebFrameImpl and text finder. 
> 
> You say 'text finder', so the name should be TextFinder, or something.

Could I name it WebFrameTextFinder or even WebFrameImplTextFinder to show
that it is linked to WebFrameImpl? Or just TextFinder is OK?

[tkent, 2013-11-27 01:24:39]

> Could I name it WebFrameTextFinder or even WebFrameImplTextFinder to show
> that it is linked to WebFrameImpl? Or just TextFinder is OK?

I prefer TextFinder.  I don't think we need to declare WebFrame association in
the class name.

[Andrey Kraynov, 2013-11-29 13:15:53]

On 2013/11/27 01:24:39, tkent wrote:
> > Could I name it WebFrameTextFinder or even WebFrameImplTextFinder to show
> > that it is linked to WebFrameImpl? Or just TextFinder is OK?
> 
> I prefer TextFinder.  I don't think we need to declare WebFrame association in
> the class name.

I have renamed InFrameFinder class to TextFinder.

[tkent, 2013-12-01 23:52:36]

lgtm

[commit-bot: I haz the power, 2013-12-01 23:52:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/iceman@yandex-team.ru/68263009/40002

[commit-bot: I haz the power, 2013-12-02 01:16:02]

Change committed as 162937

[eseidel, 2013-12-10 21:34:02]

These two files are already out of date.  Please don't do this.

[eseidel, 2013-12-10 21:34:25]

Also, please use a BUG so we can track changes between multiple revisions.

[eseidel, 2013-12-10 21:36:18]

Removing this copy in:
https://codereview.chromium.org/112083002

This came up because I made local changes to that file, not realizing it was
some work-in-progress-patch.