Remove DEBUG_TEXT_AUTOSIZING_ON_DESKTOP altogether.

Remove DEBUG_TEXT_AUTOSIZING_ON_DESKTOP altogether.

Autosizing can be triggered through devtools on desktop nowadays.

BUG=675877
Review-Url: https://codereview.chromium.org/2874783003
Cr-Commit-Position: refs/heads/master@{#471090}
Committed: https://chromium.googlesource.com/chromium/src/+/c6aa511064b40136b0bf83d926690898b285f114

[Nico, 2017-05-10 16:07:40]

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

[Nico, 2017-05-10 16:07:59]

thakis@chromium.org changed reviewers:
+ pdr@chromium.org

[commit-bot: I haz the power, 2017-05-10 16:08:53]

Dry run: 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, 2017-05-10 17:53:58]

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

[commit-bot: I haz the power, 2017-05-10 17:53:59]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Nico, 2017-05-10 17:54:34]

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

[commit-bot: I haz the power, 2017-05-10 17:56:07]

Dry run: CQ is trying da patch.

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

[pdr., 2017-05-10 19:01:16]

https://codereview.chromium.org/2874783003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/frame/Settings.cpp (left):

https://codereview.chromium.org/2874783003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/frame/Settings.cpp:65: :
text_autosizing_window_size_override_(320, 480),
I think we need to fully remove text_autosizing_window_size_override_ with this
patch

[Nico, 2017-05-10 19:06:30]

https://codereview.chromium.org/2874783003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/frame/Settings.cpp (left):

https://codereview.chromium.org/2874783003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/frame/Settings.cpp:65: :
text_autosizing_window_size_override_(320, 480),
On 2017/05/10 19:01:16, pdr. wrote:
> I think we need to fully remove text_autosizing_window_size_override_ with
this
> patch

It's used in a few (not many) tests, e.g.
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/fast/text...
,
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/web/tests/Text...
– should those tests go away too?

[pdr., 2017-05-10 19:32:54]

https://codereview.chromium.org/2874783003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/frame/Settings.cpp (left):

https://codereview.chromium.org/2874783003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/frame/Settings.cpp:65: :
text_autosizing_window_size_override_(320, 480),
On 2017/05/10 at 19:06:30, Nico wrote:
> On 2017/05/10 19:01:16, pdr. wrote:
> > I think we need to fully remove text_autosizing_window_size_override_ with
this
> > patch
> 
> It's used in a few (not many) tests, e.g.
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/fast/text...
,
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/web/tests/Text...
– should those tests go away too?

Oohh, that's right. I forgot how those tests work. We do want to keep those
tests. LGTM to land as-is.

[Nico, 2017-05-10 19:35:00]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2017-05-10 19:35:01]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2017-05-10 19:35:37]

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, 2017-05-10 22:34:32]

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

[commit-bot: I haz the power, 2017-05-10 22:34:33]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[pdr., 2017-05-10 23:28:15]

On 2017/05/10 at 22:34:33, commit-bot wrote:
> Try jobs failed on following builders:
>   linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

This bot just cleared up. Marking for CQ again.

[pdr., 2017-05-10 23:28:17]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2017-05-10 23:29:16]

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, 2017-05-11 00:41:10]

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

[commit-bot: I haz the power, 2017-05-11 00:41:11]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Nico, 2017-05-11 13:27:41]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2017-05-11 13:28:18]

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, 2017-05-11 15:26:18]

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

[commit-bot: I haz the power, 2017-05-11 15:26:19]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Nico, 2017-05-11 15:27:15]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2017-05-11 15:27:52]

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, 2017-05-11 16:58:03]

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

[commit-bot: I haz the power, 2017-05-11 16:58:04]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Nico, 2017-05-11 16:59:04]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2017-05-11 17:00:01]

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, 2017-05-11 19:58:59]

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

[commit-bot: I haz the power, 2017-05-11 19:59:00]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Nico, 2017-05-11 20:03:29]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2017-05-11 20:04:25]

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, 2017-05-11 22:28:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494533009215800, "parent_rev":
"df9f27f551ca5f0686aea37ed459a03b35a7c9b5", "commit_rev":
"c6aa511064b40136b0bf83d926690898b285f114"}

[commit-bot: I haz the power, 2017-05-11 22:28:14]

Description was changed from

==========
Remove DEBUG_TEXT_AUTOSIZING_ON_DESKTOP altogether.

Autosizing can be triggered through devtools on desktop nowadays.

BUG=675877
==========

to

==========
Remove DEBUG_TEXT_AUTOSIZING_ON_DESKTOP altogether.

Autosizing can be triggered through devtools on desktop nowadays.

BUG=675877

Review-Url: https://codereview.chromium.org/2874783003
Cr-Commit-Position: refs/heads/master@{#471090}
Committed:
https://chromium.googlesource.com/chromium/src/+/c6aa511064b40136b0bf83d92669...
==========

[commit-bot: I haz the power, 2017-05-11 22:28:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c6aa511064b40136b0bf83d92669...