Revert of range.getBoundingClientRect() returns incorrect value for collapsed ranges

Revert of range.getBoundingClientRect() returns incorrect value for collapsed ranges (patchset #3 id:40001 of https://codereview.chromium.org/998263002/)

Reason for revert:
Seems to have broken CvoxShadowUnitTest.MultilineWrap in the chromevox_tests.

Original issue's description:
> range.getBoundingClientRect() returns incorrect value for collapsed ranges
> 
> Current implementation returns empty rectangle if
> either width or height is zero. But as per W3C specification [1],
> getBoundingClientRect() should return union of rectangles
> of which the height or width is not zero.
> 
> [1] http://www.w3.org/TR/cssom-view/#dom-range-getclientrects
> 
> BUG=435438
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192699

TBR=keishi@chromium.org,vivekg@chromium.org,eae@chromium.org,kandalkar.abhijeet58@gmail.com,abhijeet.k@samsung.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=435438
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192714

[Peter Kasting, 2015-03-28 03:00:33]

Created Revert of range.getBoundingClientRect() returns incorrect value for
collapsed ranges

[Peter Kasting, 2015-03-28 03:05:19]

More detail:

Flakiness dashboard link:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=...
This implicates
http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?u...
or
https://chromium.googlesource.com/chromium/src/+/14bab52bbe1d8bf33034300f741d...
.

The most likely two changes are Blink r192699 and r192700.

Sample failure output from the test:

[ RUN      ] CvoxShadowUnitTest.MultilineWrap
[30939:30939:0327/142713:WARNING:chrome_browser_main_chromeos.cc(289)] Running
as stub user with profile dir: test-user
[30939:30939:0327/142713:INFO:audio_manager_pulse.cc(258)] Failed to connect to
the context.  Error: Connection refused
Xlib:  extension "RANDR" missing on display ":9".
[30939:30977:0327/142713:ERROR:owner_key_util_impl.cc(27)] Could not get size of
/tmp/.org.chromium.Chromium.FwiTrv/dvlImIW/stub_owner.key
[30939:30977:0327/142713:WARNING:local_extension_cache.cc(260)] Extensions will
not be installed from update URLs until /var/cache/external_cache/.initialized
exists.
[30939:30939:0327/142713:WARNING:child_account_service.cc(380)] User instance
wasn't found while setting child account flag.
[30939:30954:0327/142713:WARNING:freezer_cgroup_process_manager.cc(58)] Cgroup
freezer does not exist or is not writable. Unable to freeze renderer processes.
[30939:30958:0327/142713:WARNING:data_reduction_proxy_config.cc(330)] SPDY proxy
OFF at startup
[30939:30958:0327/142713:WARNING:data_reduction_proxy_config.cc(330)] SPDY proxy
OFF at startup
[30939:30958:0327/142713:WARNING:ipc_message_attachment_set.cc(37)]
MessageAttachmentSet destroyed with unconsumed descriptors: 0/1
[30939:30939:0327/142713:INFO:CONSOLE(1300)] "Running TestCase
CvoxShadowUnitTest.MultilineWrap", source: test_api.js (1300)
[30939:30939:0327/142713:ERROR:web_ui_test_handler.cc(75)] undefinedFailed:
RUN_TEST_F("CvoxShadowUnitTest","MultilineWrap")
Error: Test Error assertEquals(Math.floor(i / 4), shadow.getLineIndex(i))
Actual: 1
Expected: 0
at assertEquals (test_api.js:965:13)
at Object.<anonymous> (editable_text_area_shadow_test.unitjs:63:44)
at Object.Test.runTest (test_api.js:317:16)
at Object.TestCase.runTest (test_api.js:454:22)
at Object.TestCase.run (test_api.js:484:14)
at RUN_TEST_F (test_api.js:1301:21)
at expectFunc (test_api.js:1096:20)
at runTestFunction (test_api.js:1169:37)
at runTest (test_api.js:1143:18)
at <anonymous>:1:1
obj/chrome/browser/resources/chromeos/chromevox/chromevox_tests.gen/common/editable_text_area_shadow_test-gen.cc:35:
Failure
Value of: RunJavascriptTestF(false, "CvoxShadowUnitTest", "MultilineWrap")
Actual: false
Expected: true
[  FAILED  ] CvoxShadowUnitTest.MultilineWrap, where TypeParam =  and GetParam()
=  (659 ms)

So we see getLineIndex() is returning the wrong number.

(Full test source code at:
https://chromium.googlesource.com/chromium/src/+/master/chrome/browser/resour...
)

getLineIndex() is implemented in
https://chromium.googlesource.com/chromium/src/+/master/chrome/browser/resour...
.  This returns values from characterToLineMap_, which is updated in a function
that relies on getBoundingClientRect().

Because I'm about to go off shift, this is blocking rolls, and I don't
immediately know how to skip/disable the test, I'm going to revert rather than
leave this broken all weekend.

Hopefully, next week you'll be able to reland with a fixed version of this
source code without too much trouble.

[Peter Kasting, 2015-03-28 03:05:28]

The CQ bit was checked by pkasting@chromium.org

[commit-bot: I haz the power, 2015-03-28 03:05:32]

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

[commit-bot: I haz the power, 2015-03-28 03:05:56]

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

[commit-bot: I haz the power, 2015-03-28 03:05:57]

Failed to apply patch for LayoutTests/TestExpectations:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file LayoutTests/TestExpectations
  Hunk #1 FAILED at 812.
  1 out of 1 hunk FAILED -- saving rejects to file
LayoutTests/TestExpectations.rej

Patch:       LayoutTests/TestExpectations
Index: LayoutTests/TestExpectations
diff --git a/LayoutTests/TestExpectations b/LayoutTests/TestExpectations
index
0ad99dec113cd483d00f5cc632c6a08f55b3f41a..8cfd2a9de7b44b474e0387624ec879a81e742355
100644
--- a/LayoutTests/TestExpectations
+++ b/LayoutTests/TestExpectations
@@ -812,8 +812,6 @@
 crbug.com/321857 [ Win7 Debug ]
fast/filesystem/workers/file-writer-events-shared-worker.html [ Pass Failure ]
 
 crbug.com/343892 [ Linux Mac ] fast/workers/shared-worker-name.html [ Pass
Crash ]
-
-crbug.com/435438 [ Win Mac ] fast/dom/Range/getBoundingClientRect-caret.html [
NeedsRebaseline ]
 
 crbug.com/368560 [ Win ] http/tests/svg/image-fetch-fail.svg [ Timeout Pass ]

[commit-bot: I haz the power, 2015-03-28 03:05:58]

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

[Peter Kasting, 2015-03-28 03:12:18]

The CQ bit was checked by pkasting@chromium.org

[commit-bot: I haz the power, 2015-03-28 03:12:24]

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

[commit-bot: I haz the power, 2015-03-28 03:12:57]

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