Note: There are a few cases of duplicates in user autocompletion which are being worked on.

GCLI build should use the latest dryice build

RESOLVED FIXED in Firefox 11

Status

()

Firefox
Developer Tools: Console
P1
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jwalker, Assigned: jwalker)

Tracking

unspecified
Firefox 11
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(1 attachment)

Version 0.3.0 fixed a bug where text! inclusions had newlines stripped and slashes mangled.
Upgrading to the latest inserts many \n into gcli.jsm, however the functionallity isn't changed.
Priority: -- → P1
Created attachment 575166 [details] [diff] [review]
upload 1

2 changes
- insertion of \n
- s/void 0//g (i.e. implied undefined)
Attachment #575166 - Flags: review?(dcamp)

Comment 2

6 years ago
(In reply to Joe Walker from comment #1)

> - s/void 0//g (i.e. implied undefined)

I don't see that second change in this patch.
(In reply to Dave Camp (:dcamp) from comment #2)
> (In reply to Joe Walker from comment #1)
> 
> > - s/void 0//g (i.e. implied undefined)
> 
> I don't see that second change in this patch.

You're right - I was mis-reading the patch.
https://tbpl.mozilla.org/?tree=Try&rev=47353dce1691
Moving GCLI bugs to Developer Tools: Console. Filter on 'baked beans are off'.
Component: Developer Tools → Developer Tools: Console

Comment 6

6 years ago
Comment on attachment 575166 [details] [diff] [review]
upload 1

Review of attachment 575166 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good as long as it's the patch you were looking to submit :)
Attachment #575166 - Flags: review?(dcamp) → review+
Depends on: 699411
Whiteboard: [land-in-fx-team]
Blocks: 699001
https://hg.mozilla.org/integration/fx-team/rev/0b44926dd34a
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/0b44926dd34a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
You need to log in before you can comment on or make changes to this bug.