1413812 - Add Memory tool in the Web Developer menu

Status: NEW

(DevTools :: Memory, enhancement, P3)

Description

[magicp]

Adding Memory tool in the Web Developer menu, because it is default in the Developer Tools.

Comment 1

[Abhinav Koppula]

Attachment: Bug 1413812 - Add 'Memory' tool in the Web Developer menu.

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

Comment 2

[Abhinav Koppula]

Hi Greg,
I have added the `inMenu` flag for Memory tool in definitions.js which seems to have fixed the above issue.
I didn't know whom to assign for review, hence added the Triage Owner for review.

Comment 3

[Greg Tatum [:gregtatum]]

Not all of the tools are listed there as it is. I'm not sure of the policy of what tools should go in there so I don't feel comfortable merging this patch in. Abhinav: thanks for taking the time for writing a patch, but I'm going to clear the review flag for now.

magicp: If you are interested in having a discussion on what tools should be displayed, I would suggest filing an RFC [1]. The DevTools team will review it, and come up with a decision. If there is some existing bug with discussion about this topic, feel free to link me to it.

[1]: https://github.com/devtools-html/rfcs

Comment 4

[Greg Tatum [:gregtatum]]

Comment on attachment 8925325 [details]
Bug 1413812 - Add 'Memory' tool in the Web Developer menu.

https://reviewboard.mozilla.org/r/196490/#review202286

Clearing the review for now, until there is follow-up discussion about the bug.