1199093 - Refactor FxOS Geo Stumbling (follow-up work of Bug 1154435)

Status: RESOLVED WONTFIX

(Core :: DOM: Geolocation, defect)

Description

[Alphan Chen [:alchen]]

This is follow-up bug for FxOS Geo Stumbling.
In this bug, we would like to complete the following items.
1. Refactoring to a high layer. (a function such as nsGeolocationService::geostumble(nsGeoPosition))
2. Upload Stumble as gzip format
3. Creating an XPCOM component to replace most of UploadStumbleRunnable::Run

Comment 1

[:garvan]

> 2. Upload Stumble as gzip format

I created bug 1199395 for this item

Comment 2

[:garvan]

> 3. Creating an XPCOM component to replace most of UploadStumbleRunnable::Run

Josh made the valid point that XHR in JS is cleaner/simpler code. 
When you add js + idl file + manifest file + mozbuild changes, I think the handful of lines of cpp code to do this becomes about the same number of lines in this approach. I am not sure this is needed because this is such a small piece.

Comment 3

[Alphan Chen [:alchen]]

(In reply to Garvan from comment #2)
> > 3. Creating an XPCOM component to replace most of UploadStumbleRunnable::Run
> 
> Josh made the valid point that XHR in JS is cleaner/simpler code. 
> When you add js + idl file + manifest file + mozbuild changes, I think the
> handful of lines of cpp code to do this becomes about the same number of
> lines in this approach. I am not sure this is needed because this is such a
> small piece.

Agree.
Josh, what do you think?

Comment 4

[Josh Matthews [:jdm]]

Yeah, I think there are better ways we can use our time than doing the JS rewrite.

Comment 5

[Sylvestre Ledru [:Sylvestre]]

I don't think we are going to work on this anymore. Closing.