DevTools: migrate EventListenersSidebarPane to shadow

DevTools: migrate EventListenersSidebarPane to shadow

BUG=726972
Review-Url: https://codereview.chromium.org/2902553002
Cr-Commit-Position: refs/heads/master@{#475236}
Committed: https://chromium.googlesource.com/chromium/src/+/8823fc31cc8bf4a2033326d7214e77963ea74977

[luoe, 2017-05-22 18:42:43]

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

[luoe, 2017-05-22 18:42:44]

Please take a look

This is part of an effort to move more things into components.

Some of the css rules are unused/need cleaning, but I'd like to do clean sources
styles together in a separate patch.

[luoe, 2017-05-24 23:35:34]

Description was changed from

==========
DevTools: migrate EventListenersSidebarPane to shadow

BUG=none
==========

to

==========
DevTools: migrate EventListenersSidebarPane to shadow

BUG=none
==========

[luoe, 2017-05-24 23:35:35]

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

[einbinder, 2017-05-25 00:10:42]

https://codereview.chromium.org/2902553002/diff/1/third_party/WebKit/Source/d...
File
third_party/WebKit/Source/devtools/front_end/sources/eventListenerBreakpoints.css
(right):

https://codereview.chromium.org/2902553002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sources/eventListenerBreakpoints.css:7:
:host {
Why did this need to move?

[dgozman, 2017-05-25 00:17:04]

lgtm

[luoe, 2017-05-27 03:02:47]

Description was changed from

==========
DevTools: migrate EventListenersSidebarPane to shadow

BUG=none
==========

to

==========
DevTools: migrate EventListenersSidebarPane to shadow

BUG=726972
==========

[luoe, 2017-05-27 03:07:58]

https://codereview.chromium.org/2902553002/diff/1/third_party/WebKit/Source/d...
File
third_party/WebKit/Source/devtools/front_end/sources/eventListenerBreakpoints.css
(right):

https://codereview.chromium.org/2902553002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sources/eventListenerBreakpoints.css:7:
:host {
On 2017/05/25 00:10:42, einbinder wrote:
> Why did this need to move?

This style is only used by event listeners sidebar pane.  I think
breakPointsList.css should only have the general rules, breakpoint-hit,
breakpoint-list, etc.

[luoe, 2017-05-27 03:08:03]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2017-05-27 03:08:17]

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-05-27 04:56:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495854483087830, "parent_rev":
"fb9004bf71f341d4a072f4d83ef3431a36e31250", "commit_rev":
"8823fc31cc8bf4a2033326d7214e77963ea74977"}

[commit-bot: I haz the power, 2017-05-27 04:56:11]

Description was changed from

==========
DevTools: migrate EventListenersSidebarPane to shadow

BUG=726972
==========

to

==========
DevTools: migrate EventListenersSidebarPane to shadow

BUG=726972

Review-Url: https://codereview.chromium.org/2902553002
Cr-Commit-Position: refs/heads/master@{#475236}
Committed:
https://chromium.googlesource.com/chromium/src/+/8823fc31cc8bf4a2033326d7214e...
==========

[commit-bot: I haz the power, 2017-05-27 04:56:13]

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