Mouse up updates scrollbar hovered part if needed.

Mouse up updates scrollbar hovered part if needed.

BUG=671214
TEST=WebFrameTest.MouseReleaseUpdatesScrollbarHoveredPart

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

[sahel, 2016-12-16 21:34:30]

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

[commit-bot: I haz the power, 2016-12-16 21:36:18]

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-12-17 00:40:09]

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

[commit-bot: I haz the power, 2016-12-17 00:40:10]

Dry run: This issue passed the CQ dry run.

[sahel, 2016-12-19 15:29:33]

sahel@chromium.org changed reviewers:
+ bokan@chromium.org

[bokan, 2016-12-19 18:23:50]

lgtm

https://codereview.chromium.org/2587563002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/scroll/ScrollbarThemeOverlayMock.h
(right):

https://codereview.chromium.org/2587563002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/scroll/ScrollbarThemeOverlayMock.h:63: return
false;
This is fine, but was it a problem to implement Platform or the testing subclass
of it to return a default a problem?

[sahel, 2016-12-19 19:05:52]

sahel@chromium.org changed reviewers:
+ esprehn@chromium.org

[sahel, 2016-12-19 19:05:53]

esprehn@chromium.org: Please review changes in

https://codereview.chromium.org/2587563002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/scroll/ScrollbarThemeOverlayMock.h
(right):

https://codereview.chromium.org/2587563002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/scroll/ScrollbarThemeOverlayMock.h:63: return
false;
On 2016/12/19 18:23:50, bokan wrote:
> This is fine, but was it a problem to implement Platform or the testing
subclass
> of it to return a default a problem?

These unittests don't use testingPlatformSupport.

[esprehn, 2016-12-19 21:57:34]

lgtm

[sahel, 2016-12-19 22:10:45]

The CQ bit was checked by sahel@chromium.org

[commit-bot: I haz the power, 2016-12-19 22:12:55]

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-12-20 00:11:42]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482185445942930, "parent_rev":
"8e4801382e4983935a0fb112449d61aadc134b73", "commit_rev":
"9a5ddcad06b6f3b029cd6b85e2aec504b0a689a7"}

[commit-bot: I haz the power, 2016-12-20 00:12:07]

Description was changed from

==========
Mouse up updates scrollbar hovered part if needed.

BUG=671214
TEST=WebFrameTest.MouseReleaseUpdatesScrollbarHoveredPart
==========

to

==========
Mouse up updates scrollbar hovered part if needed.

BUG=671214
TEST=WebFrameTest.MouseReleaseUpdatesScrollbarHoveredPart

Review-Url: https://codereview.chromium.org/2587563002
==========

[commit-bot: I haz the power, 2016-12-20 00:12:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-20 00:13:36]

Description was changed from

==========
Mouse up updates scrollbar hovered part if needed.

BUG=671214
TEST=WebFrameTest.MouseReleaseUpdatesScrollbarHoveredPart

Review-Url: https://codereview.chromium.org/2587563002
==========

to

==========
Mouse up updates scrollbar hovered part if needed.

BUG=671214
TEST=WebFrameTest.MouseReleaseUpdatesScrollbarHoveredPart

Committed: https://crrev.com/db90376264139744e86f2fbc55ead46c2d55bf15
Cr-Commit-Position: refs/heads/master@{#439628}
==========

[commit-bot: I haz the power, 2016-12-20 00:13:37]

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