Remove grow box rect support, renderer part.

Remove grow box rect support, renderer part.

Depends on https://codereview.chromium.org/2400243002/

The grow box was always (0, 0, 0, 0) in practice, which means
all this code was a no-op.

BUG=653271
Committed: https://crrev.com/4d7eccaaf925ceeb938d79d57cea1b9067435317
Cr-Commit-Position: refs/heads/master@{#424145}

[Nico, 2016-10-07 17:19:01]

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

[commit-bot: I haz the power, 2016-10-07 17:19:23]

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

[Nico, 2016-10-07 17:19:40]

Description was changed from

==========
Remove grow box rect support, renderer part.

BUG=653271
==========

to

==========
Remove grow box rect support, renderer part.

Depends on https://codereview.chromium.org/2400243002/

The grow box was always (0, 0, 0, 0) in practice, which means
all this code was a no-op.

BUG=653271
==========

[Nico, 2016-10-07 17:23:37]

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

[commit-bot: I haz the power, 2016-10-07 17:24:01]

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

[Nico, 2016-10-07 17:27:24]

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

[Nico, 2016-10-07 17:27:26]

https://codereview.chromium.org/2395393003/diff/90001/content/renderer/render...
File content/renderer/render_widget.cc (left):

https://codereview.chromium.org/2395393003/diff/90001/content/renderer/render...
content/renderer/render_widget.cc:516:
IPC_MESSAGE_HANDLER(ViewMsg_ChangeResizeRect, OnChangeResizeRect)
Nothing ever sends this after the browser-side patch (the dependency)

https://codereview.chromium.org/2395393003/diff/90001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/frame/FrameView.cpp (left):

https://codereview.chromium.org/2395393003/diff/90001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/frame/FrameView.cpp:3768: if (!rect.isEmpty() &&
!windowResizerRect().isEmpty()) {
the windowResizerRect() was always empty, and following from that everything
else deleted in this file became unused, and this function ended up just
returning rect.

https://codereview.chromium.org/2395393003/diff/90001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/scroll/Scrollbar.h (left):

https://codereview.chromium.org/2395393003/diff/90001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/scroll/Scrollbar.h:176: bool
overlapsResizer() const { return m_overlapsResizer; }
always false now

[commit-bot: I haz the power, 2016-10-07 17:36:58]

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

[commit-bot: I haz the power, 2016-10-07 17:36:58]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Nico, 2016-10-07 19:50:01]

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

[commit-bot: I haz the power, 2016-10-07 19:50:20]

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, 2016-10-07 21:18:08]

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

[commit-bot: I haz the power, 2016-10-07 21:18:09]

Dry run: This issue passed the CQ dry run.

[Nico, 2016-10-10 12:08:53]

jochen: ping

[Nico, 2016-10-10 12:09:02]

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

[commit-bot: I haz the power, 2016-10-10 12:09:15]

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

[jochen (gone - plz use gerrit), 2016-10-10 12:31:24]

lgtm

[Nico, 2016-10-10 13:32:57]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2016-10-10 13:33:05]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-10-10 13:33: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, 2016-10-10 13:42:28]

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

[commit-bot: I haz the power, 2016-10-10 13:42:29]

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

[Nico, 2016-10-10 13:48:06]

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

[Nico, 2016-10-10 13:48:07]

rsesek: ipc owners for view_messages.h

[Robert Sesek, 2016-10-10 14:00:51]

messages lgtm

[Nico, 2016-10-10 14:03:52]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-10-10 14:03:58]

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-10-10 14:09:36]

Description was changed from

==========
Remove grow box rect support, renderer part.

Depends on https://codereview.chromium.org/2400243002/

The grow box was always (0, 0, 0, 0) in practice, which means
all this code was a no-op.

BUG=653271
==========

to

==========
Remove grow box rect support, renderer part.

Depends on https://codereview.chromium.org/2400243002/

The grow box was always (0, 0, 0, 0) in practice, which means
all this code was a no-op.

BUG=653271
==========

[commit-bot: I haz the power, 2016-10-10 14:09:37]

Committed patchset #7 (id:110001)

[commit-bot: I haz the power, 2016-10-10 14:12:02]

Description was changed from

==========
Remove grow box rect support, renderer part.

Depends on https://codereview.chromium.org/2400243002/

The grow box was always (0, 0, 0, 0) in practice, which means
all this code was a no-op.

BUG=653271
==========

to

==========
Remove grow box rect support, renderer part.

Depends on https://codereview.chromium.org/2400243002/

The grow box was always (0, 0, 0, 0) in practice, which means
all this code was a no-op.

BUG=653271

Committed: https://crrev.com/4d7eccaaf925ceeb938d79d57cea1b9067435317
Cr-Commit-Position: refs/heads/master@{#424145}
==========

[commit-bot: I haz the power, 2016-10-10 14:12:03]

Patchset 7 (id:??) landed as
https://crrev.com/4d7eccaaf925ceeb938d79d57cea1b9067435317
Cr-Commit-Position: refs/heads/master@{#424145}