Revert 189134 "Due to privacy concerns about the data contained ..."

trunk/src/chrome/browser/extensions/activity_log_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 "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/synchronization/waitable_event.h"
 #include "chrome/browser/extensions/activity_log.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/test_extension_system.h"
@@ skipping 32 matching lines @@
   virtual ~ActivityLogTest() {
     MessageLoop::current()->PostTask(FROM_HERE, MessageLoop::QuitClosure());
     MessageLoop::current()->Run();
   }
 
   static void RetrieveActions_LogAndFetchActions(
       scoped_ptr<std::vector<scoped_refptr<Action> > > i) {
     ASSERT_EQ(2, static_cast<int>(i->size()));
   }
 
-  static void Arguments_Missing(
-      scoped_ptr<std::vector<scoped_refptr<Action> > > i) {
-    scoped_refptr<Action> last = i->front();
-    std::string noargs = "ID: odlameecjipmbmbejkplpemijjgpljce, CATEGORY: "
-      "CALL, VERB: UNKNOWN_VERB, TARGET: TABS, API: tabs.testMethod, ARGS: ";
-    ASSERT_EQ(noargs, last->PrettyPrintForDebug());
-  }
-
-  static void Arguments_Present(
-      scoped_ptr<std::vector<scoped_refptr<Action> > > i) {
-    scoped_refptr<Action> last = i->front();
-    std::string args = "ID: odlameecjipmbmbejkplpemijjgpljce, CATEGORY: "
-      "CALL, VERB: UNKNOWN_VERB, TARGET: UNKNOWN_TARGET, API: "
-      "extension.connect, ARGS: \"hello\", \"world\"";
-    ASSERT_EQ(args, last->PrettyPrintForDebug());
-  }
-
  protected:
   ExtensionService* extension_service_;
   Profile* profile_;
 
  private:
   content::TestBrowserThread ui_thread_;
   content::TestBrowserThread db_thread_;
   content::TestBrowserThread file_thread_;
 };
 
@@ skipping 42 matching lines @@
                              string16(),
                              std::string("document.write"),
                              args.get(),
                              std::string("extra"));
   activity_log->GetActions(
       extension->id(),
       0,
       base::Bind(ActivityLogTest::RetrieveActions_LogAndFetchActions));
 }
 
-TEST_F(ActivityLogTest, LogWithoutArguments) {
-  ActivityLog* activity_log = ActivityLog::GetInstance(profile_);
-  scoped_refptr<const Extension> extension =
-      ExtensionBuilder()
-          .SetManifest(DictionaryBuilder()
-                       .Set("name", "Test extension")
-                       .Set("version", "1.0.0")
-                       .Set("manifest_version", 2))
-          .Build();
-  extension_service_->AddExtension(extension);
-  ASSERT_TRUE(ActivityLog::IsLogEnabled());
-
-  scoped_ptr<ListValue> args(new ListValue());
-  args->Set(0, new base::StringValue("hello"));
-  args->Set(1, new base::StringValue("world"));
-  activity_log->LogAPIAction(extension,
-                             std::string("tabs.testMethod"),
-                             args.get(),
-                             "");
-  activity_log->GetActions(
-      extension->id(),
-      0,
-      base::Bind(ActivityLogTest::Arguments_Missing));
-}
-
-TEST_F(ActivityLogTest, LogWithArguments) {
-  ActivityLog* activity_log = ActivityLog::GetInstance(profile_);
-  scoped_refptr<const Extension> extension =
-      ExtensionBuilder()
-          .SetManifest(DictionaryBuilder()
-                       .Set("name", "Test extension")
-                       .Set("version", "1.0.0")
-                       .Set("manifest_version", 2))
-          .Build();
-  extension_service_->AddExtension(extension);
-  ASSERT_TRUE(ActivityLog::IsLogEnabled());
-
-  scoped_ptr<ListValue> args(new ListValue());
-  args->Set(0, new base::StringValue("hello"));
-  args->Set(1, new base::StringValue("world"));
-  activity_log->LogAPIAction(extension,
-                            std::string("extension.connect"),
-                            args.get(),
-                            "");
-  activity_log->GetActions(
-      extension->id(),
-      0,
-      base::Bind(ActivityLogTest::Arguments_Present));
-}
-
 }  // namespace extensions