[DevTools] Tweak ListControl API.

[DevTools] Tweak ListControl API.

BUG=none

Review-Url: https://codereview.chromium.org/2609363002
Cr-Commit-Position: refs/heads/master@{#443487}
Committed: https://chromium.googlesource.com/chromium/src/+/497dcf0daf5f6e6bc8b644d85a75a6816305b93a

[dgozman, 2017-01-04 01:28:33]

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

[dgozman, 2017-01-04 01:28:34]

Take a look please.

[dgozman, 2017-01-04 01:28:38]

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

[commit-bot: I haz the power, 2017-01-04 01:29:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pfeldman, 2017-01-04 02:07:50]

https://codereview.chromium.org/2609363002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/ListControl.js (right):

https://codereview.chromium.org/2609363002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/ListControl.js:45:
itemElementClicked(item, element, event) {},
It does not help much in case you decide to listen to events on subtitles, etc.

https://codereview.chromium.org/2609363002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/ListControl.js:51:
SameHeightItems: Symbol('UI.ListMode.SameHeightItems'),
Equal/Various

https://codereview.chromium.org/2609363002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/ListControl.js:197:
invalidateSameHeight() {
invalidateItemsHeight

https://codereview.chromium.org/2609363002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui_lazy/FilteredListWidget.js
(right):

https://codereview.chromium.org/2609363002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui_lazy/FilteredListWidget.js:181:
itemElementClicked(item, element, event) {
Can we rely upon event bubbling here?

[commit-bot: I haz the power, 2017-01-04 02:59:37]

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

[commit-bot: I haz the power, 2017-01-04 02:59:37]

Dry run: This issue passed the CQ dry run.

[dgozman, 2017-01-13 00:44:07]

All done, PTAL.

[dgozman, 2017-01-13 00:44:11]

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

[commit-bot: I haz the power, 2017-01-13 00:45:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pfeldman, 2017-01-13 02:10:01]

lgtm

[dgozman, 2017-01-13 02:16:51]

The CQ bit was unchecked by dgozman@chromium.org

[dgozman, 2017-01-13 02:16:58]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-01-13 02:17:30]

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-01-13 04:22:48]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484273818604650,
"parent_rev": "02b9d1440e385a6fd768cbff8d8cf8f2443fceef", "commit_rev":
"497dcf0daf5f6e6bc8b644d85a75a6816305b93a"}

[commit-bot: I haz the power, 2017-01-13 04:23:35]

Description was changed from

==========
[DevTools] Tweak ListControl API.

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

to

==========
[DevTools] Tweak ListControl API.

BUG=none

Review-Url: https://codereview.chromium.org/2609363002
Cr-Commit-Position: refs/heads/master@{#443487}
Committed:
https://chromium.googlesource.com/chromium/src/+/497dcf0daf5f6e6bc8b644d85a75...
==========

[commit-bot: I haz the power, 2017-01-13 04:23:36]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/497dcf0daf5f6e6bc8b644d85a75...