Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

LayoutThemeMac extends LayoutTheme directly, whereas other themes
such as LayoutTheme{Win,Linux} extend LayoutThemeDefault. The logic
to turn off caret blinking in LayoutTests was placed in
LayoutThemeDefault::caretBlinkInterval() and so was ineffective on Mac.

BUG=667403
Committed: https://crrev.com/0210f10d9f5ac5b1f47f374a31dc8f70f33e5539
Cr-Commit-Position: refs/heads/master@{#433780}

[wkorman, 2016-11-22 01:28:09]

Description was changed from

==========
Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

BUG=667403
==========

to

==========
Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

LayoutThemeMac extends LayoutTheme directly, whereas other themes
such as LayoutTheme{Win,Linux} extend LayoutThemeDefault. The logic
to turn off caret blinking in LayoutTests was placed in
LayoutThemeDefault::caretBlinkInterval() and so was ineffective on Mac.

BUG=667403
==========

[wkorman, 2016-11-22 01:28:41]

wkorman@chromium.org changed reviewers:
+ wangxianzhu@chromium.org

[wkorman, 2016-11-22 01:28:48]

The CQ bit was checked by wkorman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 01:29:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xianzhu, 2016-11-22 01:31:48]

LGTM.

Please also rebase and update TestExpectations.

[wkorman, 2016-11-22 01:38:31]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2016-11-22 01:38:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
wangxianzhu@chromium.org
Link to the patchset: https://codereview.chromium.org/2525513002/#ps20001
(title: "Remove flaky marker from test.")

[wkorman, 2016-11-22 01:38:41]

On 2016/11/22 01:31:48, Xianzhu wrote:
> LGTM.
> 
> Please also rebase and update TestExpectations.

Done.

[commit-bot: I haz the power, 2016-11-22 01:39:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 03:52:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479778711870630,
"parent_rev": "7483e728a5f25a411f8a7d53b560cc667e251f27", "commit_rev":
"82b3f7a0f2e4c62daf4020c26e4d0b53a6a30ca3"}

[commit-bot: I haz the power, 2016-11-22 03:52:28]

Description was changed from

==========
Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

LayoutThemeMac extends LayoutTheme directly, whereas other themes
such as LayoutTheme{Win,Linux} extend LayoutThemeDefault. The logic
to turn off caret blinking in LayoutTests was placed in
LayoutThemeDefault::caretBlinkInterval() and so was ineffective on Mac.

BUG=667403
==========

to

==========
Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

LayoutThemeMac extends LayoutTheme directly, whereas other themes
such as LayoutTheme{Win,Linux} extend LayoutThemeDefault. The logic
to turn off caret blinking in LayoutTests was placed in
LayoutThemeDefault::caretBlinkInterval() and so was ineffective on Mac.

BUG=667403
==========

[commit-bot: I haz the power, 2016-11-22 03:52:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-22 03:55:37]

Description was changed from

==========
Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

LayoutThemeMac extends LayoutTheme directly, whereas other themes
such as LayoutTheme{Win,Linux} extend LayoutThemeDefault. The logic
to turn off caret blinking in LayoutTests was placed in
LayoutThemeDefault::caretBlinkInterval() and so was ineffective on Mac.

BUG=667403
==========

to

==========
Move logic to turn off caret blinking for LayoutTests to LayoutTheme.

LayoutThemeMac extends LayoutTheme directly, whereas other themes
such as LayoutTheme{Win,Linux} extend LayoutThemeDefault. The logic
to turn off caret blinking in LayoutTests was placed in
LayoutThemeDefault::caretBlinkInterval() and so was ineffective on Mac.

BUG=667403

Committed: https://crrev.com/0210f10d9f5ac5b1f47f374a31dc8f70f33e5539
Cr-Commit-Position: refs/heads/master@{#433780}
==========

[commit-bot: I haz the power, 2016-11-22 03:55:38]

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