This is needed for bug 812435 where testPreferredLanguage.js at this moment uses Google page, which causes failures whenever the page gets changed. It would have to read the HTTP header for the accepted language.

Added a php file that checks the HTTP headers and generates the page

Comment on attachment 688894 [details] [diff] [review] patch v1.0 Tested this on en-US and italian build as well, works as expected, just two things: >+ "pt" => array( This would need to be "pl" from Polish. >+ <div> This div is not closed later. Thanks.

Comment on attachment 688894 [details] [diff] [review] patch v1.0 Review of attachment 688894 [details] [diff] [review]: ----------------------------------------------------------------- This is way too much code. The only content this patch has to display is the value of $_SERVER["HTTP_ACCEPT_LANGUAGE"].

(In reply to Henrik Skupin (:whimboo) from comment #3) > This is way too much code. The only content this patch has to display is the > value of $_SERVER["HTTP_ACCEPT_LANGUAGE"]. s/patch/page

simplified patch, just echoing the language

Comment on attachment 689745 [details] [diff] [review] patch v2.0 Review of attachment 689745 [details] [diff] [review]: ----------------------------------------------------------------- Oh my. Something I have mentioned a couple of times in the past hit us now. We really haven't checked for existing unit tests or mochitests which probably would exercise this path. So I did a quick check and found: http://mxr.mozilla.org/mozilla-central/source/netwerk/test/unit/test_header_Accept-Language.js That means testing the HTTP header is already done and we do not have to do it again! Instead our test simply has to check if we correctly set the appropriate preference via the preferences UI. There is no real need to have an even external testcase hosted. We should get better on it checking existing tests before re-inventing the wheel.