Remove element related checks for PseudoDisabled css class.

Remove element related checks for PseudoDisabled css class.

No behavior changes.

BUG=None

Committed: https://crrev.com/6f571f023efd60fb5f1231a78765a000b7f27985
Cr-Commit-Position: refs/heads/master@{#390007}

[ramya.v, 2016-04-27 04:24:05]

ramya.v@samsung.com changed reviewers:
+ tkent@chromium.org

[ramya.v, 2016-04-27 04:24:06]

PTAL!

Thanks

[tkent, 2016-04-27 04:40:43]

https://codereview.chromium.org/1928443002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/SelectorChecker.cpp (left):

https://codereview.chromium.org/1928443002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/SelectorChecker.cpp:833: // Can it be true
for elements not in the list below?
The answer to esprehn's question is NO.  So we don't need to indroduce
matchesDisabledPseudoClass, and the code here should be just:

case CSSSelector::PseudoDisabled:
    return element.isDisabledFormControl();

[ramya.v, 2016-04-27 05:41:12]

Description was changed from

==========
Refactor code related to PseudoDisabled css class.

Functionality is moved to DOM to avoid element specific checks in
CSS module. No behavior changes.

BUG=
==========

to

==========
Remove element related checks for PseudoDisabled css class.

No behavior changes.

BUG=None
==========

[ramya.v, 2016-04-27 05:41:35]

Updated as per review comments. PTAL!

Thanks

https://codereview.chromium.org/1928443002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/SelectorChecker.cpp (left):

https://codereview.chromium.org/1928443002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/SelectorChecker.cpp:833: // Can it be true
for elements not in the list below?
On 2016/04/27 04:40:43, tkent wrote:
> The answer to esprehn's question is NO.  So we don't need to indroduce
> matchesDisabledPseudoClass, and the code here should be just:
> 
> case CSSSelector::PseudoDisabled:
>     return element.isDisabledFormControl();
Thanks :)
Done.

[tkent, 2016-04-27 05:42:10]

The CQ bit was checked by tkent@chromium.org

[tkent, 2016-04-27 05:42:10]

lgtm

[commit-bot: I haz the power, 2016-04-27 05:42:24]

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

[commit-bot: I haz the power, 2016-04-27 06:55:47]

Description was changed from

==========
Remove element related checks for PseudoDisabled css class.

No behavior changes.

BUG=None
==========

to

==========
Remove element related checks for PseudoDisabled css class.

No behavior changes.

BUG=None
==========

[commit-bot: I haz the power, 2016-04-27 06:55:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-27 06:57:31]

Description was changed from

==========
Remove element related checks for PseudoDisabled css class.

No behavior changes.

BUG=None
==========

to

==========
Remove element related checks for PseudoDisabled css class.

No behavior changes.

BUG=None

Committed: https://crrev.com/6f571f023efd60fb5f1231a78765a000b7f27985
Cr-Commit-Position: refs/heads/master@{#390007}
==========

[commit-bot: I haz the power, 2016-04-27 06:57:33]

Patchset 2 (id:??) landed as
https://crrev.com/6f571f023efd60fb5f1231a78765a000b7f27985
Cr-Commit-Position: refs/heads/master@{#390007}