825608 - Access token date is broken for public requests

Status: RESOLVED FIXED

(Core :: DOM: Geolocation, defect)

Description

[Josh Matthews [:jdm]]

>      // check to see if it has expired
>      let accessTokenDate;
>      if (this.lastRequestPrivate) {
>        accessTokenDate = gPrivateAccessTime;
>      } else {
>        Services.prefs.getIntPref(accessTokenPrefName + ".time");
>      }

Comment 1

[Josh Matthews [:jdm]]

In particular, we need to set accessTokenDate to the result of the pref.

Comment 2

[Sankha Narayan Guria [:sankha]]

Attachment: patch v1

Can you review this?

Comment 3

[Josh Matthews [:jdm]]

Comment on attachment 732264 [details] [diff] [review]
patch v1

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

Thanks!

Comment 4

[Josh Matthews [:jdm]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/892800c367ab

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/892800c367ab