User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0 Build ID: 20170201180315 Expected results: Presently in /devtools/client/inspector.js, the attributes submenu and paste submenu are created in separate functions and those functions are called in _openMenu function inorder to create the submenus, but copy submenu is created within the _openMenu function.It would be nice to give copySubmenu its own separate function like the other two. It will reduce the size of the _openMenu function as well as will make the code more consistent.
Created attachment 8838829 [details] [diff] [review] bug1340762.patch Moved copySubmenu to a separate function
Created attachment 8838831 [details] [diff] [review] bug1340762.patch Moved copySubmenu to separate function
Created attachment 8838897 [details] [diff] [review] bug1340762.patch rebased and r+. Thank you for the patch Deepjyoti
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c0d805983cf3 Refactor copySubmenu into its own separate function in inspector.js; r=gl