DevTools: add expand/collapse-subtree to context menu for DOM elements

DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633
Committed: https://crrev.com/6611f953abf0bbd3d92032b1adc3ca54380b845f
Cr-Commit-Position: refs/heads/master@{#395226}

[luoe, 2016-05-20 19:25:13]

Description was changed from

==========
DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633
==========

to

==========
DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633
==========

[luoe, 2016-05-20 19:25:13]

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

[luoe, 2016-05-20 19:28:35]

Do you think the labels "Expand subtree"/"Collapse subtree" make sense for
recursive actions?

Would it make sense for:
- Expand can always visible, even if node's subtree is partially or fully
expanded
- Collapse to be visible only when node is partially or fully expanded

[paulirish, 2016-05-20 20:29:31]

"Expand all" / "Collapse all"


I'm not sure its worth it to be smart about toggling visibility or disabled
state for those. I'd just keep both there always.

[luoe, 2016-05-20 20:57:36]

Sound good to me

[lushnikov, 2016-05-20 21:23:10]

lgtm

[luoe, 2016-05-20 21:51:02]

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

[commit-bot: I haz the power, 2016-05-20 21:51:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1998723003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1998723003/20001

[commit-bot: I haz the power, 2016-05-20 23:26:50]

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

[commit-bot: I haz the power, 2016-05-20 23:26:51]

Dry run: This issue passed the CQ dry run.

[luoe, 2016-05-21 01:51:20]

The CQ bit was checked by luoe@chromium.org

[commit-bot: I haz the power, 2016-05-21 01:51:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1998723003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1998723003/20001

[commit-bot: I haz the power, 2016-05-21 01:55:28]

Description was changed from

==========
DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633
==========

to

==========
DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633
==========

[commit-bot: I haz the power, 2016-05-21 01:55:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-21 01:56:43]

Description was changed from

==========
DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633
==========

to

==========
DevTools: add expand/collapse-subtree to context menu for DOM elements

BUG=590633

Committed: https://crrev.com/6611f953abf0bbd3d92032b1adc3ca54380b845f
Cr-Commit-Position: refs/heads/master@{#395226}
==========

[commit-bot: I haz the power, 2016-05-21 01:56:44]

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