Revert of DevTools: all swatches should have a default focused element

Revert of DevTools: all swatches should have a default focused element (patchset #4 id:60001 of https://codereview.chromium.org/2510883002/ )

Reason for revert:
This introduced a color picker bug crbug.com/667739.  To land, we need to fix the regression.

Original issue's description:
> DevTools: all swatches should have a default focused element
>
> Color swatches correctly respond to 'Enter' and 'Esc' because Spectrum calls
> focus() whenever it loads. Bezier and shadow editors did not. This CL makes
> bezier and shadow editors set a default focused element. SwatchPopoverHelper now
> calls focus() after showing a view, instead of requiring each swatch-popover
> implementation call focus.
>
> One peculiarity with color swatches is they may hide and show themselves after
> loading a palette. To keep focus on the popover, focus restoring logic was added
> to the SwatchPopoverHelper whenever hiding an old popover.
>
> BUG=665801
>
> Committed: https://crrev.com/0384ce04d25ba97446b20e7ca2d423cec10c2ffd
> Cr-Commit-Position: refs/heads/master@{#433382}

TBR=dgozman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=665801
Committed: https://crrev.com/7f89a2a5efbd41b83b66c8d18bea8eb9804e175b
Cr-Commit-Position: refs/heads/master@{#434060}

[luoe, 2016-11-22 19:59:57]

The CQ bit was checked by luoe@chromium.org

[luoe, 2016-11-22 19:59:58]

Created Revert of DevTools: all swatches should have a default focused element

[commit-bot: I haz the power, 2016-11-22 20:00:23]

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-11-22 20:00:24]

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

[commit-bot: I haz the power, 2016-11-22 20:00:25]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-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.

[kozy, 2016-11-22 20:57:17]

kozyatinskiy@chromium.org changed reviewers:
+ kozyatinskiy@chromium.org

[kozy, 2016-11-22 20:57:18]

lgtm

[kozy, 2016-11-22 20:58:05]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-11-22 20:58:37]

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-11-23 01:12:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479848285462190, "parent_rev":
"09199b7e6ac400667a32616d36c02614224d6ec8", "commit_rev":
"1558359eed5a67c8f3ceced4fc1ead4c5e614931"}

[commit-bot: I haz the power, 2016-11-23 01:12:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-23 01:17:32]

Description was changed from

==========
Revert of DevTools: all swatches should have a default focused element (patchset
#4 id:60001 of https://codereview.chromium.org/2510883002/ )

Reason for revert:
This introduced a color picker bug crbug.com/667739.  To land, we need to fix
the regression.

Original issue's description:
> DevTools: all swatches should have a default focused element
> 
> Color swatches correctly respond to 'Enter' and 'Esc' because Spectrum calls
> focus() whenever it loads. Bezier and shadow editors did not. This CL makes
> bezier and shadow editors set a default focused element. SwatchPopoverHelper
now
> calls focus() after showing a view, instead of requiring each swatch-popover
> implementation call focus.
> 
> One peculiarity with color swatches is they may hide and show themselves after
> loading a palette. To keep focus on the popover, focus restoring logic was
added
> to the SwatchPopoverHelper whenever hiding an old popover.
> 
> BUG=665801
> 
> Committed: https://crrev.com/0384ce04d25ba97446b20e7ca2d423cec10c2ffd
> Cr-Commit-Position: refs/heads/master@{#433382}

TBR=dgozman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=665801
==========

to

==========
Revert of DevTools: all swatches should have a default focused element (patchset
#4 id:60001 of https://codereview.chromium.org/2510883002/ )

Reason for revert:
This introduced a color picker bug crbug.com/667739.  To land, we need to fix
the regression.

Original issue's description:
> DevTools: all swatches should have a default focused element
>
> Color swatches correctly respond to 'Enter' and 'Esc' because Spectrum calls
> focus() whenever it loads. Bezier and shadow editors did not. This CL makes
> bezier and shadow editors set a default focused element. SwatchPopoverHelper
now
> calls focus() after showing a view, instead of requiring each swatch-popover
> implementation call focus.
>
> One peculiarity with color swatches is they may hide and show themselves after
> loading a palette. To keep focus on the popover, focus restoring logic was
added
> to the SwatchPopoverHelper whenever hiding an old popover.
>
> BUG=665801
>
> Committed: https://crrev.com/0384ce04d25ba97446b20e7ca2d423cec10c2ffd
> Cr-Commit-Position: refs/heads/master@{#433382}

TBR=dgozman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=665801

Committed: https://crrev.com/7f89a2a5efbd41b83b66c8d18bea8eb9804e175b
Cr-Commit-Position: refs/heads/master@{#434060}
==========

[commit-bot: I haz the power, 2016-11-23 01:17:33]

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