1235909 - Add optional props for expand/collapse event handlers to the Tree component

Status: RESOLVED FIXED

(DevTools :: General, defect)

Comment 1

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Attachment: Add optional props for expand/collapse event handlers to the Tree component

Comment 2

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Need onExpand in particular for bug 1229229 to know when to start incrementally fetching children in the dominator tree.

Comment 3

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=449144acc118

Comment 4

[Jordan Santell [:jsantell] [@jsantell]]

Comment on attachment 8703004 [details] [diff] [review]
Add optional props for expand/collapse event handlers to the Tree component

Review of attachment 8703004 [details] [diff] [review]:
-----------------------------------------------------------------

Anyway to crudely test expandAllChildren in this context? If not, can we make a small bug for it?

Comment 5

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Filed bug 1237345

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/b1a0a9378c64

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b1a0a9378c64

Comment 8

[Mayur Patil]

[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel          beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS			Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected