Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(111)

Issue 282343005: Revert of Revert of Fix selection of label having control element. (Closed)

Created:
6 years, 7 months ago by Xianzhu
Modified:
6 years, 7 months ago
Reviewers:
tkent, keishi, deepak.sa
CC:
blink-reviews, dglazkov+blink, adamk+blink_chromium.org, ojan
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

Revert of Revert of Fix selection of label having control element. (https://codereview.chromium.org/290153002/) Reason for revert: The previously reverted behavior is actually correct and what we wanted. BUG=148912, 370859 Original issue's description: > Revert of Fix selection of label having control element. (https://codereview.chromium.org/258933006/) > > Reason for revert: > Causes bug 370859 > > BUG=148912, 370859 > TBR=keishi > > Original issue's description: > > Fix selection of label having control element. > > > > Any click event on label goes to its control element. So, if > > we tried to select the text inside label by dragging the mouse > > or by double clicking on the text, it went to its control > > element. This patch checks whether there is selection or not, > > after the click is happened. If there is selection then do > > nothing else continue. > > > > New Layout test added: > > fast/forms/label/label-selection.html > > > > BUG=148912 > > > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=173310 > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174193 TBR=tkent@chromium.org,keishi@chromium.org,deepak.sa@samsung.com NOTREECHECKS=true NOTRY=true BUG=148912, 370859 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174202 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174296

Patch Set 1 #

Patch Set 2 : Revert again #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -2 lines) Patch
A LayoutTests/fast/forms/label/label-selection.html View 1 1 chunk +29 lines, -0 lines 0 comments Download
A + LayoutTests/fast/forms/label/label-selection-expected.txt View 1 1 chunk +2 lines, -2 lines 0 comments Download
M Source/core/html/HTMLLabelElement.cpp View 1 2 chunks +7 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Xianzhu
Created Revert of Revert of Fix selection of label having control element.
6 years, 7 months ago (2014-05-16 19:08:11 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/282343005/1
6 years, 7 months ago (2014-05-16 19:08:38 UTC) #2
commit-bot: I haz the power
Change committed as 174202
6 years, 7 months ago (2014-05-16 19:09:52 UTC) #3
tkent
lgtm
6 years, 7 months ago (2014-05-18 23:42:25 UTC) #4
Xianzhu
The CQ bit was checked by wangxianzhu@chromium.org
6 years, 7 months ago (2014-05-19 17:06:10 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/282343005/90001
6 years, 7 months ago (2014-05-19 17:06:23 UTC) #6
commit-bot: I haz the power
6 years, 7 months ago (2014-05-19 17:06:55 UTC) #7
Message was sent while issue was closed.
Change committed as 174296

Powered by Google App Engine
This is Rietveld 408576698