Cleanup: LFH only needs to be enabled on XP/2003

base/process/memory_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.
 
 #define _CRT_SECURE_NO_WARNINGS
 
 #include "base/process/memory.h"
 
 #include <stddef.h>
 
@@ skipping 49 matching lines @@
                 const_cast<int*>(&kConstantInModule)));
 
   // Any address within the kernel32 module should return
   // kernel32's HMODULE.  Our only assumption here is that
   // kernel32 is larger than 4 bytes.
   HMODULE kernel32 = ::GetModuleHandle(L"kernel32.dll");
   HMODULE kernel32_from_address =
       base::GetModuleFromAddress(reinterpret_cast<DWORD*>(kernel32) + 1);
   EXPECT_EQ(kernel32, kernel32_from_address);
 }
-
-TEST(ProcessMemoryTest, EnableLFH) {
-  ASSERT_TRUE(base::EnableLowFragmentationHeap());
-  if (IsDebuggerPresent()) {
-    // Under these conditions, LFH can't be enabled. There's no point to test
-    // anything.
-    const char* no_debug_env = getenv("_NO_DEBUG_HEAP");
-    if (!no_debug_env || strcmp(no_debug_env, "1"))
-      return;
-  }
-  HMODULE kernel32 = GetModuleHandle(L"kernel32.dll");
-  ASSERT_TRUE(kernel32 != NULL);
-  HeapQueryFn heap_query = reinterpret_cast<HeapQueryFn>(GetProcAddress(
-      kernel32,
-      "HeapQueryInformation"));
-
-  // On Windows 2000, the function is not exported. This is not a reason to
-  // fail but we won't be able to retrieves information about the heap, so we
-  // should stop here.
-  if (heap_query == NULL)
-    return;
-
-  HANDLE heaps[1024] = { 0 };
-  unsigned number_heaps = GetProcessHeaps(1024, heaps);
-  EXPECT_GT(number_heaps, 0u);
-  for (unsigned i = 0; i < number_heaps; ++i) {
-    ULONG flag = 0;
-    SIZE_T length;
-    ASSERT_NE(0, heap_query(heaps[i],
-                            HeapCompatibilityInformation,
-                            &flag,
-                            sizeof(flag),
-                            &length));
-    // If flag is 0, the heap is a standard heap that does not support
-    // look-asides. If flag is 1, the heap supports look-asides. If flag is 2,
-    // the heap is a low-fragmentation heap (LFH). Note that look-asides are not
-    // supported on the LFH.
-
-    // We don't have any documented way of querying the HEAP_NO_SERIALIZE flag.
-    EXPECT_LE(flag, 2u);
-    EXPECT_NE(flag, 1u);
-  }
-}
 #endif  // defined(OS_WIN)
 
 #if defined(OS_MACOSX)
 
 // For the following Mac tests:
 // Note that base::EnableTerminationOnHeapCorruption() is called as part of
 // test suite setup and does not need to be done again, else mach_override
 // will fail.
 
 TEST(ProcessMemoryTest, MacTerminateOnHeapCorruption) {
@@ skipping 346 matching lines @@
   for (size_t i = 0; i < (kSafeCallocItems * kSafeCallocSize); ++i)
     EXPECT_EQ(0, bytes[i]);
   free(value_);
 
   EXPECT_FALSE(base::UncheckedCalloc(1, test_size_, &value_));
   EXPECT_TRUE(value_ == NULL);
 }
 #endif  // !defined(MEMORY_TOOL_REPLACES_ALLOCATOR)
 #endif  // !defined(OS_ANDROID) && !defined(OS_OPENBSD) && !(defined(OS_WIN) &&
         // !defined(ALLOCATOR_SHIM)) && !defined(MEMORY_TOOL_REPLACES_ALLOCATOR)