Closed
Bug 768392
Opened 12 years ago
Closed 8 years ago
GCLI command to refresh the Tilt visualization mesh
Categories
(DevTools Graveyard :: Graphic Commandline and Toolbar, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: vporof, Unassigned)
References
Details
(Whiteboard: [gclicommands])
Attachments
(1 file)
3.99 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Updated•12 years ago
|
Summary: GCLI commant to refresh the Tilt visualization mesh → GCLI command to refresh the Tilt visualization mesh
Reporter | ||
Comment 1•12 years ago
|
||
Attachment #636659 -
Flags: review?(jwalker)
Comment 2•12 years ago
|
||
Comment on attachment 636659 [details] [diff] [review] v1 Review of attachment 636659 [details] [diff] [review]: ----------------------------------------------------------------- (Cut and paste alert!) I do think we need to add some tests for these commands. We've skipped it in the past and got bitten. It's not hard, because there is a framework to help. Examples: - browser/devtools/commandline/test/browser_gcli_commands.js - browser/devtools/commandline/test/browser_gcli_edit.js There is even documentation (which I should perhaps move to MDN): - browser/devtools/commandline/test/head.js See the doc comments for DeveloperToolbarTest.checkInputStatus() and DeveloperToolbarTest.exec() Thanks,
Attachment #636659 -
Flags: review?(jwalker)
Reporter | ||
Comment 5•12 years ago
|
||
Thanks Mike, but the reasons I'm delaying this a bit is that the commad relies on a pretty uncomfortable hack, which exposes some private APIs in Tilt. There are a few bugs on file that plan to fix this, all of them already have patches, so it's likely that I'll change this in the almost-near future. Theoretically this works ok as it is, but I'm not feeling quite positive about it. Let's postpone this for a short while, until Tilt settles.
Comment 6•12 years ago
|
||
Okay, my apologies. I will update the spreadsheet.
Assignee: mratcliffe → vporof
Comment 7•12 years ago
|
||
New component triage. Filter on "Lobster Thermidor aux crevettes with a Mornay sauce"
Component: Developer Tools: Console → Developer Tools: Graphic Commandline and Toolbar
Reporter | ||
Comment 8•11 years ago
|
||
Filter on TILT QUODLIBET.
Assignee: vporof → nobody
Status: ASSIGNED → NEW
Comment 9•10 years ago
|
||
Resetting priorities because these P* levels are very out of date. Sorry for the bug spam. Filter on Lobster Thermidor
Priority: P3 → --
Comment 10•8 years ago
|
||
Triage. Filter on Lobster Thermidor. Tilt being killed by e10s.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•6 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•