Fix accidental updates of host's baselines_ in Elide().

Fix accidental updates of host's baselines_ in Elide().

This invocation causes a side effect of setting wrong max to
the baselines_ during ElideEmail(), which causes an infinite loop
later in ItemizeTextToRuns().

BUG=470179
R=msw@chromium.org
TEST=the new test case covers (it doesn't end without this patch)

Committed: https://crrev.com/a98db8504afb14ca24e5152f1731c6a49b5e099e
Cr-Commit-Position: refs/heads/master@{#322410}

[msw, 2015-03-25 23:17:09]

lgtm with nits

https://codereview.chromium.org/1037573006/diff/1/ui/gfx/render_text_unittest.cc
File ui/gfx/render_text_unittest.cc (right):

https://codereview.chromium.org/1037573006/diff/1/ui/gfx/render_text_unittest...
ui/gfx/render_text_unittest.cc:722: TEST_F(RenderTextTest, HarfBuzz_ElideEmail)
{
nit: Rename Elide[d]Email, move after ElidedText or ElidedObscuredText.

https://codereview.chromium.org/1037573006/diff/1/ui/gfx/render_text_unittest...
ui/gfx/render_text_unittest.cc:733: EXPECT_NE(long_email,
render_text->GetDisplayText());
optional nit: you can remove this, the statement below is sufficient.

[Jun Mukai, 2015-03-26 16:34:12]

https://codereview.chromium.org/1037573006/diff/1/ui/gfx/render_text_unittest.cc
File ui/gfx/render_text_unittest.cc (right):

https://codereview.chromium.org/1037573006/diff/1/ui/gfx/render_text_unittest...
ui/gfx/render_text_unittest.cc:722: TEST_F(RenderTextTest, HarfBuzz_ElideEmail)
{
On 2015/03/25 23:17:09, msw wrote:
> nit: Rename Elide[d]Email, move after ElidedText or ElidedObscuredText.

Done.

https://codereview.chromium.org/1037573006/diff/1/ui/gfx/render_text_unittest...
ui/gfx/render_text_unittest.cc:733: EXPECT_NE(long_email,
render_text->GetDisplayText());
On 2015/03/25 23:17:09, msw wrote:
> optional nit: you can remove this, the statement below is sufficient.

Done.

[Jun Mukai, 2015-03-26 16:34:16]

The CQ bit was checked by mukai@chromium.org

[Jun Mukai, 2015-03-26 16:34:16]

The patchset sent to the CQ was uploaded after l-g-t-m from msw@chromium.org
Link to the patchset: https://codereview.chromium.org/1037573006/#ps20001
(title: "comments addressed")

[commit-bot: I haz the power, 2015-03-26 16:35:05]

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

[commit-bot: I haz the power, 2015-03-26 17:27:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-26 17:28:23]

Patchset 2 (id:??) landed as
https://crrev.com/a98db8504afb14ca24e5152f1731c6a49b5e099e
Cr-Commit-Position: refs/heads/master@{#322410}