1341680 - this.layoutInspector is undefined when opening inspector

Status: RESOLVED FIXED

(DevTools :: Inspector, defect, P3)

Description

[Julian Descottes [:jdescottes]]

STRs:
- enable layout panel in devtools config
- open inspector, with any other panel than the layout panel selected by default

=> Error is logged
> TypeError: this.layoutInspector is undefined: onSidebarSelect@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/layout/layout.js:451:7
> emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/event-emitter.js:191:13
> handleSelectionChange@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/inspector/toolsidebar.js:234:5
> select/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/components/tabs/tabbar.js:139:9
> notifyAll@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/vendor/react.js:842:9
> close@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/vendor/react.js:16630:5
> closeAll@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/vendor/react.js:19101:11
> perform@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/vendor/react.js:19048:11
> perform@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/vendor/react.js:16660:12
> flushBatchedUpdates@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/client/shared/vendor/react.js:1

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1341680 - wait for layoutInspector to be initialized before attaching events;

Review commit: https://reviewboard.mozilla.org/r/114556/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/114556/

Comment 2

[Gabriel Luong [:gl] (ΦωΦ)]

Comment on attachment 8839998 [details]
Bug 1341680 - wait for layoutInspector to be initialized before attaching events;

https://reviewboard.mozilla.org/r/114556/#review116176

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/882622ce817e
wait for layoutInspector to be initialized before attaching events;r=gl

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/882622ce817e

Comment 5

[Kazi Nuzhat Tasnem]

I have reproduced this bug with Firefox nightly 54.0a1 (build id:20170222030329)on
windows 7(64 bit)

I have verified this bug as fixed with Firefox beta 54.0b3 (build id:20170427091925)
User Agent:Mozilla/5.0 (Windows NT 6.1; WOW64; rv:54.0) Gecko/20100101 Firefox/54.0

[testday-20170428]