1014367 - API key support for translation

Status: VERIFIED FIXED

(Firefox :: Translations, defect)

Description

[:Felipe Gomes (needinfo for replies!)]

Add the support for the Bing API clientid/key tokens.

There are 3 parts to it:
 - add the build time support for including the keys
 - use the keys in the code
 - add the keyfile/mozconfig changes to production

This bug is about part 1.

Comment 1

[:Felipe Gomes (needinfo for replies!)]

Attachment: API key build patch

Gavin, what do you think about using the same structure as the other keys, going through nsURLFormatter like this?

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 8426793 [details] [diff] [review]
API key build patch

Sounds good to me.

Comment 3

[(not currently active) Ted Mielczarek]

Comment on attachment 8426793 [details] [diff] [review]
API key build patch

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

::: configure.in
@@ +4038,5 @@
>      MOZ_GOOGLE_API_KEY=no-google-api-key
>  fi
>  AC_SUBST(MOZ_GOOGLE_API_KEY)
>  
> +# Allow to specify a Bing API key file that contains the client ID and the

Grammar nit: "Allow specifying"

::: toolkit/components/urlformatter/Makefile.in
@@ +16,5 @@
>  	@echo '#define MOZ_GOOGLE_API_KEY $(MOZ_GOOGLE_API_KEY)' > $@
>  
> +bing_api_key:
> +	@echo '#define MOZ_BING_API_KEY $(MOZ_BING_API_KEY)' > $@
> +	@echo '#define MOZ_BING_API_CLIENTID $(MOZ_BING_API_CLIENTID)' >> $@

We should probably just create these files in configure instead. I won't make you change that since you're just following precedent.

Comment 4

[:Felipe Gomes (needinfo for replies!)]

Thanks!

> ::: configure.in
> @@ +4038,5 @@
> >      MOZ_GOOGLE_API_KEY=no-google-api-key
> >  fi
> >  AC_SUBST(MOZ_GOOGLE_API_KEY)
> >  
> > +# Allow to specify a Bing API key file that contains the client ID and the
> 
> Grammar nit: "Allow specifying"

This was just copy&paste from the two comment blocks above. I will fix the comment on the others too when landing, ok?

Comment 5

[(not currently active) Ted Mielczarek]

Thanks!

Comment 6

[:Felipe Gomes (needinfo for replies!)]

I actually left the other blocks untouched because it was such a small thing that I just avoided messing with the hg blame on them. I fixed the phrase on my new block.

https://hg.mozilla.org/integration/fx-team/rev/459040b37060

Comment 7

[Marco Mucci [:MarcoM]]

Added to Iteration 32.2

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/459040b37060