Deleted unused code about WebViewImpl::getSelectionRootBounds

Deleted unused code about WebViewImpl::getSelectionRootBounds

WebViewImpl::getSelectionRootBounds function was called by RenderViewImpl::GetSelectionRootBounds.
But nowdays anything don't call the function, so this code is dead code.
I deleted some html file and modified some source code.

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

[Byoungkwon Ko, 2014-12-04 15:48:56]

The CQ bit was checked by gogag2@gmail.com

[Byoungkwon Ko, 2014-12-04 15:49:05]

The CQ bit was unchecked by gogag2@gmail.com

[Byoungkwon Ko, 2014-12-04 15:52:40]

gogag2@gmail.com changed reviewers:
+ pdr@chromium.org, zino@chromium.org

[Byoungkwon Ko, 2014-12-04 15:53:34]

On 2014/12/04 15:52:40, Byoungkwon Ko wrote:
> mailto:gogag2@gmail.com changed reviewers:
> + mailto:pdr@chromium.org, mailto:zino@chromium.org

Could you review for me?

[pdr., 2014-12-04 19:01:59]

Nice! Just some minor comments below. I'll kick off some tryjob runs for you as
well.

https://codereview.chromium.org/781743002/diff/1/Source/web/tests/WebViewTest...
File Source/web/tests/WebViewTest.cpp (right):

https://codereview.chromium.org/781743002/diff/1/Source/web/tests/WebViewTest...
Source/web/tests/WebViewTest.cpp:1957: IntRect expectedIntRect =
expectedRootBounds;
Lets remove "WebRect expectedRootBounds = ..." above and set it here:
IntRect expectedIntRect = ExpectedRootBounds(document,
webView->pageScaleFactor());

https://codereview.chromium.org/781743002/diff/1/Source/web/tests/WebViewTest...
Source/web/tests/WebViewTest.cpp:1963: TEST_F(WebViewTest,
GetSelectionRootBoundsBrokenHeight)
Lets just remove this test entirely.

[Byoungkwon Ko, 2014-12-07 15:36:18]

Thanks for your kind review.

I modified WebViewTest.cpp and deleted select_range_basic_broken_height.html

Could you review for me?

Thank you.:)

[pdr., 2014-12-07 21:51:02]

pdr@chromium.org changed reviewers:
+ adamk@chromium.org, pfeldman@chromium.org
- zino@chromium.org

[pdr., 2014-12-07 21:52:35]

LGTM. We'll need to wait for a Source/web owner for the final OK.

Adam or Pavel, do you like deadcode?

[pfeldman, 2014-12-07 23:52:57]

lgtm.

[pdr., 2014-12-07 23:54:07]

The CQ bit was checked by pdr@chromium.org

[Byoungkwon Ko, 2014-12-07 23:54:45]

The CQ bit was checked by gogag2@gmail.com

[commit-bot: I haz the power, 2014-12-07 23:54:59]

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

[commit-bot: I haz the power, 2014-12-08 01:27:03]

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