Move implementation of WebFrameImpl::executeCommand to Editor

Move implementation of WebFrameImpl::executeCommand to Editor

This patch moves |WebFrameImpl::executeComamnd()| to |Editor| class to make
"web" as thin API layer and ease of maintenance.

This patch also get rid of |WebViewImpl::bubblingScroll|, which
becomes unused function as result of moving |WebFrameImpl::executeCommand()|.

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

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

[yosin_UTC9, 2014-06-27 09:07:24]

Could you review this patch?
Thanks in advance.

[tkent, 2014-06-30 00:22:02]

lgtm

https://codereview.chromium.org/337473010/diff/20001/Source/core/editing/Edit...
File Source/core/editing/Editor.h (right):

https://codereview.chromium.org/337473010/diff/20001/Source/core/editing/Edit...
Source/core/editing/Editor.h:158: bool executeCommand(const String&);
We should have a comment that these functions are NOT implementations of
document.execCommand().

[yosin_UTC9, 2014-06-30 01:22:08]

Thanks for reviewing!
Committing...

https://codereview.chromium.org/337473010/diff/20001/Source/core/editing/Edit...
File Source/core/editing/Editor.h (right):

https://codereview.chromium.org/337473010/diff/20001/Source/core/editing/Edit...
Source/core/editing/Editor.h:158: bool executeCommand(const String&);
On 2014/06/30 00:22:02, tkent wrote:
> We should have a comment that these functions are NOT implementations of
> document.execCommand().

Done.

[yosin_UTC9, 2014-06-30 01:22:11]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2014-06-30 01:22:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yosin@chromium.org/337473010/40001

[commit-bot: I haz the power, 2014-06-30 02:09:13]

Change committed as 177179