[SystemInfo API] Finish TODOs in SystemInfoProvider

chrome/browser/extensions/api/system_info_cpu/system_info_cpu_apitest.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 "base/command_line.h"
 #include "chrome/browser/extensions/api/system_info_cpu/cpu_info_provider.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/test/base/ui_test_utils.h"
 
 namespace extensions {
 
 using api::system_info_cpu::CpuInfo;
 
 const char kExtensionId[] = "lfakdgdkbaleijdcpbfbngfphpmgfdfn";
 
 class MockCpuInfoProviderImpl : public CpuInfoProvider {
  public:
   MockCpuInfoProviderImpl() {}
 
-  virtual bool QueryInfo(CpuInfo* info) OVERRIDE {
-    if (!info) return false;
+  virtual void QueryInfo() OVERRIDE {
+    CpuInfo* info = &info_;
 
     info->num_of_processors = 4;
     info->arch_name = "x86";
     info->model_name = "unknown";
 
-    return true;
+    success_ = true;
   }
 
  private:
   virtual ~MockCpuInfoProviderImpl() {}
 };
 
 class SystemInfoCpuApiTest: public ExtensionApiTest {
  public:
   SystemInfoCpuApiTest() {}
   virtual ~SystemInfoCpuApiTest() {}
@@ skipping 10 matching lines @@
 };
 
 IN_PROC_BROWSER_TEST_F(SystemInfoCpuApiTest, Cpu) {
   CpuInfoProvider* provider = new MockCpuInfoProviderImpl();
   // The provider is owned by the single CpuInfoProvider instance.
   CpuInfoProvider::InitializeForTesting(provider);
   ASSERT_TRUE(RunExtensionTest("systeminfo/cpu")) << message_;
 }
 
 }  // namespace extensions