Revert of Fix selection of label having control element.

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

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

[Xianzhu, 2014-05-16 17:10:25]

Created Revert of Fix selection of label having control element.

[commit-bot: I haz the power, 2014-05-16 17:10:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/290153002/1

[commit-bot: I haz the power, 2014-05-16 17:10:51]

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

[commit-bot: I haz the power, 2014-05-16 17:10:51]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Xianzhu, 2014-05-16 17:11:58]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-05-16 17:12:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/290153002/1

[commit-bot: I haz the power, 2014-05-16 17:12:32]

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

[commit-bot: I haz the power, 2014-05-16 17:12:33]

Failed to apply patch for Source/core/html/HTMLLabelElement.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/html/HTMLLabelElement.cpp
  Hunk #2 FAILED at 136.
  1 out of 2 hunks FAILED -- saving rejects to file
Source/core/html/HTMLLabelElement.cpp.rej

Patch:       Source/core/html/HTMLLabelElement.cpp
Index: Source/core/html/HTMLLabelElement.cpp
diff --git a/Source/core/html/HTMLLabelElement.cpp
b/Source/core/html/HTMLLabelElement.cpp
index
9dd855b64d1279d39ba3b612d288938f91175e61..65d71604ee8999d090a814c33155be190de33774
100644
--- a/Source/core/html/HTMLLabelElement.cpp
+++ b/Source/core/html/HTMLLabelElement.cpp
@@ -27,9 +27,7 @@
 
 #include "HTMLNames.h"
 #include "core/dom/ElementTraversal.h"
-#include "core/editing/FrameSelection.h"
 #include "core/events/Event.h"
-#include "core/frame/LocalFrame.h"
 #include "core/html/FormAssociatedElement.h"
 
 namespace WebCore {
@@ -138,11 +136,6 @@
     static bool processingClick = false;
 
     if (evt->type() == EventTypeNames::click && !processingClick) {
-        // If text of label element is selected, do not pass
-        // the event to control element.
-        if (document().frame()->selection().selection().isRange())
-            return;
-
         RefPtr<HTMLElement> element = control();
 
         // If we can't find a control or if the control received the click

[Xianzhu, 2014-05-16 17:16:40]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-05-16 17:16:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/290153002/70001

[Xianzhu, 2014-05-16 17:44:28]

Committed patchset #2 manually as r174193 (presubmit successful).

[Xianzhu, 2014-05-16 19:08:11]

A revert of this CL has been created in
https://codereview.chromium.org/282343005/ by wangxianzhu@chromium.org.

The reason for reverting is: Sorry, there is a bad conflict resolution..

[Xianzhu, 2014-05-16 19:14:09]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2014-05-16 19:14:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wangxianzhu@chromium.org/290153002/90001

[Xianzhu, 2014-05-16 21:11:30]

Committed patchset #3 manually as r174210 (presubmit successful).

[tkent, 2014-05-18 23:43:03]

not lgtm.
I don't think we should revert it.

[Xianzhu, 2014-05-19 16:03:17]

On 2014/05/18 23:43:03, tkent wrote:
> not lgtm.
> I don't think we should revert it.

It caused bad regression, so I thought we should revert it first before we have
a better solution.

[Xianzhu, 2014-05-19 17:05:51]

On 2014/05/19 16:03:17, Xianzhu wrote:
> On 2014/05/18 23:43:03, tkent wrote:
> > not lgtm.
> > I don't think we should revert it.
> 
> It caused bad regression, so I thought we should revert it first before we
have
> a better solution.

Sorry, just found my previous test was inaccurate. Will reapply the original
patch (https://codereview.chromium.org/282343005/).