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)

12 Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: vporof, Unassigned)

References

Details

(Whiteboard: [gclicommands])

Attachments

(1 file)

      No description provided.
Summary: GCLI commant to refresh the Tilt visualization mesh → GCLI command to refresh the Tilt visualization mesh
Attached patch v1Splinter Review
Attachment #636659 - Flags: review?(jwalker)
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)
Blocks: GCLICMD
Filter on teabags
Whiteboard: [gclicommands]
I will create tests for this
Assignee: vporof → mratcliffe
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.
Okay, my apologies. I will update the spreadsheet.
Assignee: mratcliffe → vporof
Depends on: 734038
Priority: -- → P3
New component triage. Filter on "Lobster Thermidor aux crevettes with a Mornay sauce"
Component: Developer Tools: Console → Developer Tools: Graphic Commandline and Toolbar
Filter on TILT QUODLIBET.
Assignee: vporof → nobody
Status: ASSIGNED → NEW
Resetting priorities because these P* levels are very out of date.
Sorry for the bug spam. Filter on Lobster Thermidor
Priority: P3 → --
Triage. Filter on Lobster Thermidor.
Tilt being killed by e10s.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: