Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3993)

Unified Diff: chrome/browser/extensions/api/log_private/log_private_apitest_chromeos.cc

Issue 329853010: Additional methods for chrome.logPrivate API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/log_private/log_private_apitest_chromeos.cc
diff --git a/chrome/browser/extensions/api/log_private/log_private_apitest_chromeos.cc b/chrome/browser/extensions/api/log_private/log_private_apitest_chromeos.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d0154eed019ec6fb5327c4d5f3e4fd57eaafaeb8
--- /dev/null
+++ b/chrome/browser/extensions/api/log_private/log_private_apitest_chromeos.cc
@@ -0,0 +1,103 @@
+// Copyright 2013 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 <string>
+
+#include "base/file_util.h"
+#include "base/files/file_path.h"
+#include "chrome/browser/extensions/extension_apitest.h"
+#include "chrome/common/chrome_switches.h"
+#include "chromeos/dbus/fake_dbus_thread_manager.h"
+#include "chromeos/dbus/fake_debug_daemon_client.h"
+#include "content/public/browser/browser_thread.h"
+#include "extensions/common/extension_builder.h"
+#include "net/dns/mock_host_resolver.h"
+#include "net/test/embedded_test_server/embedded_test_server.h"
+#include "net/test/embedded_test_server/http_request.h"
+#include "net/test/embedded_test_server/http_response.h"
+
+using net::test_server::BasicHttpResponse;
+using net::test_server::HttpResponse;
+using net::test_server::HttpRequest;
+
+namespace {
+
+class TestDebugDaemonClient : public chromeos::FakeDebugDaemonClient {
+ public:
+ explicit TestDebugDaemonClient(const base::FilePath& test_file)
+ : test_file_(test_file) {}
+
+ virtual ~TestDebugDaemonClient() {}
+
+ virtual void DumpDebugLogs(bool is_compressed,
+ base::File file,
+ const GetDebugLogsCallback& callback) OVERRIDE {
+ base::File* file_param = new base::File(file.Pass());
+ content::BrowserThread::PostBlockingPoolTaskAndReply(
+ FROM_HERE,
+ base::Bind(
+ &GenerateTestLogDumpFile, test_file_, base::Owned(file_param)),
+ base::Bind(callback, true));
+ }
+
+ static void GenerateTestLogDumpFile(const base::FilePath& test_tar_file,
+ base::File* file) {
+ std::string test_file_content;
+ EXPECT_TRUE(base::ReadFileToString(test_tar_file, &test_file_content))
+ << "Cannot read content of file " << test_tar_file.value();
+ const int data_size = static_cast<int>(test_file_content.size());
+ EXPECT_EQ(data_size, file->Write(0, test_file_content.data(), data_size));
+ EXPECT_TRUE(file->SetLength(data_size));
+ file->Close();
+ }
+
+ private:
+ base::FilePath test_file_;
+};
+
+} // namespace
+
+namespace extensions {
+
+class LogPrivateApiTest : public ExtensionApiTest {
+ public:
+ LogPrivateApiTest() {}
+
+ virtual ~LogPrivateApiTest() {}
+
+ virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+ chromeos::FakeDBusThreadManager* fake_dbus_thread_manager =
+ new chromeos::FakeDBusThreadManager;
+ fake_dbus_thread_manager->SetFakeClients();
+ base::FilePath tar_file_path =
+ test_data_dir_.Append("log_private/dump_logs/system_logs.tar");
+ fake_dbus_thread_manager->SetDebugDaemonClient(
+ scoped_ptr<chromeos::DebugDaemonClient>(
+ new TestDebugDaemonClient(tar_file_path)));
+ chromeos::DBusThreadManager::SetInstanceForTesting(
+ fake_dbus_thread_manager);
+ ExtensionApiTest::SetUpInProcessBrowserTestFixture();
+ }
+
+ scoped_ptr<HttpResponse> HandleRequest(const HttpRequest& request) {
+ scoped_ptr<BasicHttpResponse> response(new BasicHttpResponse);
+ response->set_code(net::HTTP_OK);
+ response->set_content(
+ "<html><head><title>LogPrivateTest</title>"
+ "</head><body>Hello!</body></html>");
+ return response.PassAs<HttpResponse>();
+ }
+};
+
+IN_PROC_BROWSER_TEST_F(LogPrivateApiTest, DumpLogsAndCaptureEvents) {
+ // Setup dummy HTTP server.
+ host_resolver()->AddRule("www.test.com", "127.0.0.1");
+ ASSERT_TRUE(StartEmbeddedTestServer());
+ embedded_test_server()->RegisterRequestHandler(
+ base::Bind(&LogPrivateApiTest::HandleRequest, base::Unretained(this)));
+
+ ASSERT_TRUE(RunExtensionTest("log_private/dump_logs"));
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698