Fixes detection logic for IE under XP SP2.

chrome_frame/CFInstall.js

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview CFInstall.js provides a set of utilities for managing
  * the Chrome Frame detection and installation process.
  * @author slightlyoff@google.com (Alex Russell)
  */
 
@@ skipping 91 matching lines @@
                    'margin-top: -300px;' +
                  '}' +
                  '.chromeFrameOverlayContent {' +
                    'position: absolute;' +
                    'margin-left: -400px;' +
                    'margin-top: -300px;' +
                    'left: 50%;' +
                    'top: 50%;' +
                    'border: 1px solid #93B4D9;' +
                    'background-color: white;' +
+                   'z-index: 2001;' +
                  '}' +
                  '.chromeFrameOverlayContent iframe {' +
                    'width: 800px;' +
                    'height: 600px;' +
                    'border: none;' +
                  '}' +
                  '.chromeFrameOverlayCloseBar {' +
                    'height: 1em;' +
                    'text-align: right;' +
                    'background-color: #CADEF4;' +
                  '}' +
                  '.chromeFrameOverlayUnderlay {' +
                    'position: absolute;' +
                    'width: 100%;' +
                    'height: 100%;' +
                    'background-color: white;' +
                    'opacity: 0.5;' +
                    '-moz-opacity: 0.5;' +
                    '-webkit-opacity: 0.5;' +
                    '-ms-filter: ' +
                       '"progid:DXImageTransform.Microsoft.Alpha(Opacity=50)";' +
                    'filter: alpha(opacity=50);' +
+                   'z-index: 2000;' +
                  '}';
     injectStyleSheet(rules);
     cfStyleTagInjected = true;
   };
 
   /**
    * Injects style rules to hide the overlay version of the GCF prompt.
    * Multiple calls have no effect.
    */
   var closeOverlay = function() {
@@ skipping 88 matching lines @@
           '<td>' +
             // TODO(slightlyoff): i18n
             '<button id="chromeFrameCloseButton">close</button>' +
           '</td>' +
         '</tr>' +
         '<tr>' +
           '<td id="chromeFrameIframeHolder"></td>' +
         '</tr>' +
       '</table>';
 
-    document.body.appendChild(n);
+    var b = document.body;
+    // Insert underlay nodes into the document in the right order.
+    while (n.firstChild) {
+      b.insertBefore(n.lastChild, b.firstChild);
+    }
     var ifr = makeIframe(args);
     byId('chromeFrameIframeHolder').appendChild(ifr);
     byId('chromeFrameCloseButton').onclick = closeOverlay;
   };
 
   var CFInstall = {};
 
   /**
    * Checks to see if Chrome Frame is available, if not, prompts the user to
    * install. Once installation is begun, a background timer starts,
@@ skipping 11 matching lines @@
 
     // We currently only support CF in IE
     // TODO(slightlyoff): Update this should we support other browsers!
     var ua = navigator.userAgent;
     var ieRe = /MSIE \S+; Windows NT/;
     var bail = false;
     if (ieRe.test(ua)) {
       // We also only support Win2003/XPSP2 or better. See:
       //  http://msdn.microsoft.com/en-us/library/ms537503%28VS.85%29.aspx
       if (parseFloat(ua.split(ieRe)[1]) < 6 &&
-          ua.indexOf('SV1') >= 0) {
+          // 'SV1' indicates SP2, only bail if not SP2 or Win2K3
+          ua.indexOf('SV1') < 0) {
         bail = true;
       }
     } else {
+      // Not IE
       bail = true;
     }
     if (bail) {
       return;
     }
 
     // Inject the default styles
     injectCFStyleTag();
 
     if (document.cookie.indexOf("disableGCFCheck=1") >=0) {
@@ skipping 51 matching lines @@
 
   CFInstall._force = false;
   CFInstall._forceValue = false;
   CFInstall.isAvailable = isAvailable;
 
   // expose CFInstall to the external scope. We've already checked to make
   // sure we're not going to blow existing objects away.
   scope.CFInstall = CFInstall;
 
 })(this['ChromeFrameInstallScope'] || this);