Fix re-layout logic in LayoutScrollbar::updateScrollbarParts.

Fix re-layout logic in LayoutScrollbar::updateScrollbarParts.

When an iframe's custom scrollbar changes its thickness, it is the LayoutView
within the iframe that should be marked for relayout, not the LayoutIFrame.

BUG=512914
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200694

[skobes, 2015-08-15 06:26:45]

skobes@chromium.org changed reviewers:
+ chrishtr@chromium.org

[chrishtr, 2015-08-15 14:30:37]

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

[chrishtr, 2015-08-15 14:30:38]

I'm out the next week, passing on the review.

[Xianzhu, 2015-08-17 16:17:06]

lgtm

https://codereview.chromium.org/1296033002/diff/1/LayoutTests/scrollbars/cust...
File
LayoutTests/scrollbars/custom-scrollbar-enable-changes-thickness-with-iframe.html
(right):

https://codereview.chromium.org/1296033002/diff/1/LayoutTests/scrollbars/cust...
LayoutTests/scrollbars/custom-scrollbar-enable-changes-thickness-with-iframe.html:12:
<iframe width=800 height=600 src="resources/empty.html"></iframe>
Nit: Would srcdoc="" work (so that we can avoid another file)?

[skobes, 2015-08-17 20:39:27]

https://codereview.chromium.org/1296033002/diff/1/LayoutTests/scrollbars/cust...
File
LayoutTests/scrollbars/custom-scrollbar-enable-changes-thickness-with-iframe.html
(right):

https://codereview.chromium.org/1296033002/diff/1/LayoutTests/scrollbars/cust...
LayoutTests/scrollbars/custom-scrollbar-enable-changes-thickness-with-iframe.html:12:
<iframe width=800 height=600 src="resources/empty.html"></iframe>
On 2015/08/17 16:17:05, Xianzhu wrote:
> Nit: Would srcdoc="" work (so that we can avoid another file)?

Done.

[skobes, 2015-08-17 20:39:30]

The CQ bit was checked by skobes@chromium.org

[skobes, 2015-08-17 20:39:30]

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/1296033002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-08-17 20:39:54]

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

[commit-bot: I haz the power, 2015-08-17 22:26:13]

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

[commit-bot: I haz the power, 2015-08-17 22:26:14]

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

[skobes, 2015-08-17 22:27:12]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2015-08-17 22:27:39]

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

[commit-bot: I haz the power, 2015-08-18 00:49:48]

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

[commit-bot: I haz the power, 2015-08-18 00:49:48]

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

[skobes, 2015-08-18 00:50:12]

The CQ bit was checked by skobes@chromium.org

[commit-bot: I haz the power, 2015-08-18 00:50:31]

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

[commit-bot: I haz the power, 2015-08-18 04:33:24]

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