t-artistik / browserscope

Automatically exported from code.google.com/p/browserscope
Apache License 2.0
0 stars 0 forks source link

"We think you're using [...]" is missing js_user_agent_string and document_mode #188

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Load browserscope with the IE 9 preview, or ChromeFrame.
2. Look at the browser reported after "We think you're using"

What is the expected output? What do you see instead?
It should give IE Platform Preview or ChromeFrame as the browser instead of 
something like IE.

The code that looks up the browser is missing inputs that we use for the 
beacon:
  js_user_agent_string
  document_mode

It will either need to make a second request to lookup the browser,
or fix it on the javascript-side (duplicates code, but avoids an extra 
round trip).

I am thinking that making a second request is the way to go.

Original issue reported on code.google.com by steve.lamm on 24 Mar 2010 at 7:03

GoogleCodeExporter commented 9 years ago
Fixed in r553.

Original comment by steve.lamm on 24 Mar 2010 at 10:27