Suggest upgrading to SP3 or later for invalid certificate errors.

chrome/browser/ssl/ssl_blocking_page_unittest.cc

+// Copyright (c) 2012 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
+
+#include "chrome/browser/ssl/ssl_blocking_page.h"
+#include "chrome/test/base/browser_with_test_window_test.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/test/web_contents_tester.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+#if defined(OS_WIN)
+#include "base/win/windows_version.h"
+#endif
+
+TEST(SSLBlockingPageTest, SSLBlockingPageWindowsVersionTest) {

[felt, 2014/06/30 18:36:15]

This test won't get coverage because we don't have bots that will cover all the
cases.

Instead, it would make more sense to use OSInfo from
https://code.google.com/p/chromium/codesearch#chromium/src/base/win/windows_v...
and then mock out the version() and service_pack() calls using EXPECT_CALL. See
https://code.google.com/p/googlemock/wiki/ForDummies. That way you can make it
act like various different versions and see if the code works as expected.

[radhikabhar, 2014/07/01 23:56:49]

Should I change it to get the OS info from OSInfo. I did not do that because I
would be using the same logic so I am not sure how much that will make sense.

+  bool os_version_win = false;
+#if defined(OS_WIN)
+  os_version_win = true;
+  OSVERSIONINFOEX version_info = { sizeof version_info };
+  GetVersionEx(reinterpret_cast<OSVERSIONINFO*>(&version_info));
+  if ((version_info.dwMajorVersion == 5) && (version_info.dwMinorVersion == 1)
+      && (version_info.wServicePackMajor < 3))
+    EXPECT_TRUE(SSLBlockingPage::WindowsVersionSP3Lower());
+  else
+    EXPECT_FALSE(SSLBlockingPage::WindowsVersionSP3Lower());
+#endif
+  if (!os_version_win)
+    EXPECT_FALSE(SSLBlockingPage::WindowsVersionSP3Lower());
+}