Pass clickEvent to controlElement inside label if targetNode can't start selection

third_party/WebKit/Source/core/html/HTMLLabelElement.cpp

Index: third_party/WebKit/Source/core/html/HTMLLabelElement.cpp
diff --git a/third_party/WebKit/Source/core/html/HTMLLabelElement.cpp b/third_party/WebKit/Source/core/html/HTMLLabelElement.cpp
index 06670f2cf1a1c1623ce571f30741bf5481761a0d..9ce1de6e51058a6aea6d1c3d6cd5892e2e15abe3 100644
--- a/third_party/WebKit/Source/core/html/HTMLLabelElement.cpp
+++ b/third_party/WebKit/Source/core/html/HTMLLabelElement.cpp
@@ -163,7 +163,8 @@ void HTMLLabelElement::DefaultEventHandler(Event* evt) {
     // click event to control element.
     // Note: check if it is a MouseEvent because a click event may
     // not be an instance of a MouseEvent if created by document.createEvent().
-    if (evt->IsMouseEvent() && ToMouseEvent(evt)->HasPosition()) {
+    if (evt->IsMouseEvent() && ToMouseEvent(evt)->HasPosition() &&
+        isHTMLLabelElement(evt->target()->ToNode())) {

[yosin_UTC9, 2017/05/09 00:56:06]

I think we should check Event#srcElement in selection based on comment in the
code:
 // Check if there is a selection and click is not on the
 // selection.

[tkent, 2017/05/09 02:14:26]

Does this work if <label> has a child element?  e.g. <label><input
type=checkbox> <b><img ...> Check this</b></label>

[tanvir, 2017/05/09 12:02:32]

1) On using the label element as below

<label><input type=checkbox>
<b><img ...> Check this</b></label>

"If there is selection of label element by dragging, no click
event is passed. Also, no focus on control element"  is failing.
 
The reason is isHTMLLabelElement(evt->target()->ToNode()) fails in this case, as
evt->target()->ToNode() is a <B> element.

With PatchSet#3 i submitted this works fine.

2) There is one difference i observed with Firefox behaviour (Please check this
for reference https://jsfiddle.net/tanvirR/egcpLffh/4/ ),
When we select any text, and then click on <img> inside a Label, then control
gets focused and the selection gets cleared.
But in our case, the selection is not cleared. 

The reason as SelectionController::HandleSingleClick prevents it by the below
condition
  if (!(inner_node && inner_node->GetLayoutObject() &&
        mouse_down_may_start_select_))
    return false;
As on certain nodes canStartSelection returns false, like <img> Element.

This is the reason i added evt->target()->ToNode()->CanStartSelection() in my
PS#3 to by pass those
nodes on which CanStartSelection() can not be done.

[tanvir, 2017/05/10 13:44:36]

Hi Yosin, I couldn't get what you meant by " we should check Event#srcElement in
selection ", Did you meant at SelectionController ?

       if (LocalFrame* frame = GetDocument().GetFrame()) {
         // Check if there is a selection and click is not on the
         // selection.