Joe Walker [:jwalker] Assignee
	Description • 13 years ago

This will help while converting data for output display.

	Joe Walker [:jwalker] Assignee
	Comment 1 • 13 years ago

Moving GCLI bugs to Developer Tools: Console. Filter on 'baked beans are off'.

	Joe Walker [:jwalker] Assignee
	Comment 2 • 12 years ago

Triage. Filter on PEGASUS.

	Joe Walker [:jwalker] Assignee
	Comment 3 • 12 years ago

GCLI Triage.

	Joe Walker [:jwalker] Assignee
	Comment 4 • 12 years ago

Triage, filter on TEABAGS

	Joe Walker [:jwalker] Assignee
	Comment 5 • 12 years ago

New component triage. Filter on "Lobster Thermidor aux crevettes with a Mornay sauce"

	Joe Walker [:jwalker] Assignee
	Comment 6 • 11 years ago

https://github.com/MikeRatcliffe/gcli/pull/1

	Joe Walker [:jwalker] Assignee
	Comment 7 • 11 years ago

The last pull request got closed: https://github.com/MikeRatcliffe/gcli/pull/2

	Joe Walker [:jwalker] Assignee
	Comment 8 • 11 years ago

The changes to gcli.jsm should be familiar to you.

	Joe Walker [:jwalker] Assignee
	Comment 9 • 11 years ago

https://tbpl.mozilla.org/?tree=Try&rev=3e7f38b6ccfc

	Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]
	Comment 10 • 11 years ago

Comment on attachment 727626 [details] [diff] [review]
v1

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

Looks good to me, r+.

::: browser/devtools/commandline/gcli.jsm
@@ +10903,5 @@
>  
>      promisedDirectTabText.resolve(directTabText);
>      promisedArrowTabText.resolve(arrowTabText);
>      promisedEmptyParameters.resolve(emptyParameters);
> +  }.bind(this), onError);

Should this not be:
}.bind(this), util.errorHandler);
?

	Joe Walker [:jwalker] Assignee
	Comment 11 • 11 years ago

(In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #10)
> Comment on attachment 727626 [details] [diff] [review]
> v1
> 
> Review of attachment 727626 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks good to me, r+.
> 
> ::: browser/devtools/commandline/gcli.jsm
> @@ +10903,5 @@
> >  
> >      promisedDirectTabText.resolve(directTabText);
> >      promisedArrowTabText.resolve(arrowTabText);
> >      promisedEmptyParameters.resolve(emptyParameters);
> > +  }.bind(this), onError);
> 
> Should this not be:
> }.bind(this), util.errorHandler);
> ?

Good spot, but in this case no. We don't want to see the error message, we want to pass it on to the promises. onError is a function to do that:

  // If anything goes wrong, we pass the error on to all the child promises
  var onError = function(ex) {
    promisedDirectTabText.reject(ex);
    promisedArrowTabText.reject(ex);
    promisedEmptyParameters.reject(ex);
  };

Thanks

	Joe Walker [:jwalker] Assignee
	Comment 12 • 11 years ago

Try breakage was down to forgetting to copy the helpers.js changes across.

	Joe Walker [:jwalker] Assignee
	Comment 13 • 11 years ago

Comment out a failing test.

That's OK we're fixing it in bug 784790 which is following this one down the landing strip.

	Joe Walker [:jwalker] Assignee
	Comment 14 • 11 years ago

https://tbpl.mozilla.org/?tree=Fx-Team&rev=f952eebd4cbd
https://hg.mozilla.org/integration/fx-team/rev/f952eebd4cbd

	Tim Taubert [:ttaubert] (inactive)
	Comment 15 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/f952eebd4cbd