Add 'caret-color' CSS property to _colorAwareProperties

DevTools: Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG=703552
Review-Url: https://codereview.chromium.org/2764113003
Cr-Commit-Position: refs/heads/master@{#458696}
Committed: https://chromium.googlesource.com/chromium/src/+/f9f76a2c8a9757bfada489528fe3d0a8eb9ff374

[kdzwinel, 2017-03-21 22:17:04]

Description was changed from

==========
Add 'caret-color' CSS property to _colorAwareProperties

BUG=703552
==========

to

==========
Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG=703552
==========

[kdzwinel, 2017-03-21 22:18:19]

kdzwinel@gmail.com changed reviewers:
+ chenwilliam@chromium.org, pfeldman@chromium.org

[kdzwinel, 2017-03-21 22:18:29]

PTAL

[chenwilliam, 2017-03-21 22:43:04]

Description was changed from

==========
Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG=703552
==========

to

==========
DevTools: Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG=703552
==========

[chenwilliam, 2017-03-21 22:43:59]

thanks! I updated the title to match our style. lgtm (I'm not an owner so will
need pfeldman's stamp)

[kdzwinel, 2017-03-21 22:51:57]

On 2017/03/21 at 22:43:59, chenwilliam wrote:
> thanks! I updated the title to match our style. lgtm (I'm not an owner so will
need pfeldman's stamp)

oops, I forgot about the prefix. Will start using it from now on. Thanks!

BTW: Contribution guide says to include you and pfeldman as reviewers by
default, but I'd like to avoid annoying more people than needed with my CLs. How
do I figure out who is the owner of the affected part of the code? Looking at
git blame?

[chenwilliam, 2017-03-21 23:40:00]

On 2017/03/21 22:51:57, kdzwinel wrote:
> On 2017/03/21 at 22:43:59, chenwilliam wrote:
> > thanks! I updated the title to match our style. lgtm (I'm not an owner so
will
> need pfeldman's stamp)
> 
> oops, I forgot about the prefix. Will start using it from now on. Thanks!
> 
> BTW: Contribution guide says to include you and pfeldman as reviewers by
> default, but I'd like to avoid annoying more people than needed with my CLs.
How
> do I figure out who is the owner of the affected part of the code? Looking at
> git blame?

No worries - it's easy for me to add the right person (e.g. @allada for
network-related stuff),
but yeah I usually use git blame myself when I'm unsure. There is a tool "git cl
owners" to select
OWNERS but it's not always that helpful so I rarely use it.

[pfeldman, 2017-03-22 00:58:01]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2017-03-22 00:58:02]

lgtm

[commit-bot: I haz the power, 2017-03-22 00:58:42]

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, 2017-03-22 03:01:19]

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

[commit-bot: I haz the power, 2017-03-22 03:01:20]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_TIMED_OUT, build hasn't
started yet, builder probably lacks capacity)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_TIMED_OUT,
build hasn't started yet, builder probably lacks capacity)

[kdzwinel, 2017-03-22 09:15:00]

The CQ bit was checked by kdzwinel@gmail.com

[commit-bot: I haz the power, 2017-03-22 09:15:16]

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, 2017-03-22 09:18:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490174100515250, "parent_rev":
"fd1a23a745f72ad8116a8c6ec278022a2cc5a074", "commit_rev":
"f9f76a2c8a9757bfada489528fe3d0a8eb9ff374"}

[commit-bot: I haz the power, 2017-03-22 09:18:59]

Description was changed from

==========
DevTools: Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG=703552
==========

to

==========
DevTools: Add 'caret-color' CSS property to _colorAwareProperties

http://i.imgur.com/i1C0xby.png

BUG=703552

Review-Url: https://codereview.chromium.org/2764113003
Cr-Commit-Position: refs/heads/master@{#458696}
Committed:
https://chromium.googlesource.com/chromium/src/+/f9f76a2c8a9757bfada489528fe3...
==========

[commit-bot: I haz the power, 2017-03-22 09:19:00]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f9f76a2c8a9757bfada489528fe3...