DevTools: all swatches get default focused element (reland with fix)

DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

Instead of modifying the color picker's focus logic, this CL simply
adds a focus() call for all swatches and sets the default focused
element for the other swatch types.

BUG=665801
Committed: https://crrev.com/69cc45e5e9b178c78fede5e9c4271aad78847854
Cr-Commit-Position: refs/heads/master@{#434572}

[luoe, 2016-11-23 01:27:28]

Description was changed from

==========
DevTools: all swatches get default focused element with proper colorswatch

BUG=665801
==========

to

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

BUG=665801
==========

[luoe, 2016-11-23 01:35:54]

Description was changed from

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

BUG=665801
==========

to

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

Instead of modifying the color picker's focus logic, this CL simply
adds a focus() call for all swatches and sets the default focused
element for the other swatch types.

BUG=665801
==========

[luoe, 2016-11-23 02:02:56]

luoe@chromium.org changed reviewers:
+ dgozman@chromium.org

[luoe, 2016-11-23 02:02:57]

This reland doesn't add anything new, but rather subtracts the logic in the
first CL attempt related to the color swatch.  Please take a look.

[dgozman, 2016-11-23 22:11:02]

lgtm

[luoe, 2016-11-23 22:11:44]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-23 22:12:19]

Dry run: 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 23:30:32]

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

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

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[luoe, 2016-11-24 00:44:53]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-24 00:45:52]

Dry run: 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-24 02:47:12]

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

[commit-bot: I haz the power, 2016-11-24 02:47:13]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[luoe, 2016-11-24 02:47:58]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-24 02:48:25]

Dry run: 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-24 04:49:34]

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

[commit-bot: I haz the power, 2016-11-24 04:49:35]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[luoe, 2016-11-26 05:27:42]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2016-11-26 05:27:50]

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-26 07:21:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480138062590560,
"parent_rev": "db2f67995fa696d7923a3bf4d4187e7439db4d45", "commit_rev":
"41040cc61297c91f03ced4a9c6c5dbc1508994e4"}

[commit-bot: I haz the power, 2016-11-26 07:21:36]

Description was changed from

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

Instead of modifying the color picker's focus logic, this CL simply
adds a focus() call for all swatches and sets the default focused
element for the other swatch types.

BUG=665801
==========

to

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

Instead of modifying the color picker's focus logic, this CL simply
adds a focus() call for all swatches and sets the default focused
element for the other swatch types.

BUG=665801
==========

[commit-bot: I haz the power, 2016-11-26 07:21:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-26 07:23:43]

Description was changed from

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

Instead of modifying the color picker's focus logic, this CL simply
adds a focus() call for all swatches and sets the default focused
element for the other swatch types.

BUG=665801
==========

to

==========
DevTools: all swatches get default focused element (reland with fix)

This is a reland of another CL, but with an additional fix:
https://codereview.chromium.org/2510883002/

Instead of modifying the color picker's focus logic, this CL simply
adds a focus() call for all swatches and sets the default focused
element for the other swatch types.

BUG=665801

Committed: https://crrev.com/69cc45e5e9b178c78fede5e9c4271aad78847854
Cr-Commit-Position: refs/heads/master@{#434572}
==========

[commit-bot: I haz the power, 2016-11-26 07:23:44]

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