Issue metadata
Sign in to add a comment
|
'blackbox script' on right button menu is misleading
Reported by
davidmax...@gmail.com,
Oct 28 2017
|
||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36 Steps to reproduce the problem: 1. open a js file in dev tool eg this page 'wizard.js' 2. find function 'goStep()' and select 'showStep()' call 3. right click 4. select 'blackbox script' What is the expected behavior? 'showStep()' is blackboxed What went wrong? wizard.js is blackboxed Did this work before? N/A Chrome version: 61.0.3163.100 Channel: n/a OS Version: 10.0 Flash Version: I realise (now) that blackboxing is currently a file-level thing, and that function-level blackboxing is being worked on, but it is very misleading to have that option in the menu when a function is selected. TBH, I imagine I could say the same for a lot (all?) of things on that menu - it looks like they are in the context of the selection, but they're not.
,
Oct 30 2017
I think I did that already...using this page as an example.
,
Oct 30 2017
Thank you for providing more feedback. Adding requester "brajkumar@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 30 2017
,
Dec 5 2017
Thanks for your report! Context menu said "Blackbox script". We can be better in naming here. We are not going to support function level blackboxing in near future. As work around you can evaluate your function using eval with dedicated sourceURL and use this source url to blackbox it.
,
Dec 6 2017
I'm not really after a work around, since the problem is simply a matter of learning/comprehension - this issue isn't about having a function level blackboxing facility, but is about the menu being misleading. As such: > We can be better in naming here. about sums it up. (Having said that, I guess the comments on function level blackboxing might help people who arrive here via such a search, so no problem) |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by brajkumar@chromium.org
, Oct 30 2017Labels: Needs-Feedback Needs-Milestone