Skip to content
This repository was archived by the owner on May 10, 2019. It is now read-only.

Conversation

@fmarier
Copy link
Contributor

@fmarier fmarier commented Nov 28, 2012

When the browser is idle, it will start fetching and caching the dialog's Javascript, CSS and images. This works on IE9 and Firefox
(it's currently disabled in Chromium).

The trick is to make sure that the communication iframe is open while the user is busy reading the RP's site. When using the
Observer API, this happens automatically at the end of the navigator.id.watch() call.

More background on this feature here: http://feeding.cloud.geek.nz/2012/11/prefetching-resources-to-prime-browser.html

When the browser is idle, it will start fetching and caching the
dialog's Javascript, CSS and images. This works on IE9 and Firefox
(it's currently disabled in Chromium).

The trick is to make sure that the communication iframe is open
while the user is busy reading the RP's site. When using the
Observer API, this happens automatically at the end of the
navigator.id.watch() call.
@fmarier
Copy link
Contributor Author

fmarier commented Nov 28, 2012

I also thought it might be a good idea to add a new helper function to connect-cachify. In which case we could make this patch a bit more readable: fmarier@e78a8a6

@ghost ghost assigned shane-tomlinson Nov 29, 2012
@jaredhirsch
Copy link
Member

do we want to pull in this optimization? @shane-tomlinson, @fmarier, thoughts?

let's close this if we don't want to pull this code in right now, so that we can get to PR inbox zero (theoretically at least).

@shane-tomlinson
Copy link

@6a68 - not yet, not until we slim down the overall bundle.

@fmarier
Copy link
Contributor Author

fmarier commented Jan 25, 2013

I'll submit a new pull request once we're ready to merge. I have to simplify this one anyways since Austin merged my connect-cachify PR.

@fmarier fmarier closed this Jan 25, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants