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

Unified Diff: chrome/browser/chromeos/app_mode/kiosk_app_update_service_browsertest.cc

Issue 16844020: app_mode: Add runtime.onRestartRequired event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: restore onBrowserUpdateAvailable and address comments in #2 Created 7 years, 6 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/chromeos/app_mode/kiosk_app_update_service_browsertest.cc
diff --git a/chrome/browser/chromeos/app_mode/kiosk_app_update_service_browsertest.cc b/chrome/browser/chromeos/app_mode/kiosk_app_update_service_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..99efb6c10e1e841327f365dbbce4159b4b8b7bcd
--- /dev/null
+++ b/chrome/browser/chromeos/app_mode/kiosk_app_update_service_browsertest.cc
@@ -0,0 +1,117 @@
+// 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/basictypes.h"
+#include "base/command_line.h"
+#include "base/compiler_specific.h"
+#include "base/file_util.h"
+#include "base/files/file_path.h"
+#include "base/files/scoped_temp_dir.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/path_service.h"
+#include "base/prefs/pref_service.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/test/scoped_path_override.h"
+#include "base/threading/sequenced_worker_pool.h"
+#include "base/time.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/browser_process_platform_part.h"
+#include "chrome/browser/chromeos/app_mode/kiosk_app_update_service.h"
+#include "chrome/browser/chromeos/system/automatic_reboot_manager.h"
+#include "chrome/browser/extensions/extension_test_message_listener.h"
+#include "chrome/browser/extensions/platform_app_browsertest_util.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/common/extensions/extension.h"
+#include "chrome/common/pref_names.h"
+#include "chromeos/chromeos_paths.h"
+#include "chromeos/dbus/update_engine_client.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/test/test_utils.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace chromeos {
+
+class KioskAppUpdateServiceTest : public extensions::PlatformAppBrowserTest {
+ public:
+ KioskAppUpdateServiceTest() : app_(NULL), update_service_(NULL) {}
+ virtual ~KioskAppUpdateServiceTest() {}
+
+ // extensions::PlatformAppBrowserTest overrides:
+ virtual void SetUpOnMainThread() OVERRIDE {
+ extensions::PlatformAppBrowserTest::SetUpOnMainThread();
+
+ ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
+ const base::FilePath& temp_dir = temp_dir_.path();
+
+ const base::TimeDelta uptime = base::TimeDelta::FromHours(1);
+ const std::string uptime_seconds =
+ base::DoubleToString(uptime.InSecondsF());
+ const base::FilePath uptime_file = temp_dir.Append("uptime");
+ ASSERT_EQ(static_cast<int>(uptime_seconds.size()),
+ file_util::WriteFile(
+ uptime_file, uptime_seconds.c_str(), uptime_seconds.size()));
+ uptime_file_override_.reset(
+ new base::ScopedPathOverride(chromeos::FILE_UPTIME, uptime_file));
+
+ app_ = LoadExtension(
+ test_data_dir_.AppendASCII("api_test/runtime/on_restart_required"));
+
+ // Fake app mode command line.
+ CommandLine* command = CommandLine::ForCurrentProcess();
+ command->AppendSwitch(switches::kForceAppMode);
+ command->AppendSwitchASCII(switches::kAppId, app_->id());
+
+ update_service_ = KioskAppUpdateServiceFactory::GetForProfile(profile());
+ update_service_->set_app_id(app_->id());
+
+ content::BrowserThread::GetBlockingPool()->FlushForTesting();
+ content::RunAllPendingInMessageLoop();
+ }
+
+ void FireAppUpdateAvailable() {
+ update_service_->OnAppUpdateAvailable(app_->id());
+ }
+
+ void FireUpdatedNeedReboot() {
+ UpdateEngineClient::Status status;
+ status.status = UpdateEngineClient::UPDATE_STATUS_UPDATED_NEED_REBOOT;
+ g_browser_process->platform_part()->automatic_reboot_manager()->
+ UpdateStatusChanged(status);
+ }
+
+ private:
+ base::ScopedTempDir temp_dir_;
+ scoped_ptr<base::ScopedPathOverride> uptime_file_override_;
+ const extensions::Extension* app_; // Not owned
bartfab (slow) 2013/06/21 18:18:13 Nit: Full stop at end of comment.
xiyuan 2013/06/21 18:33:18 Done.
+ KioskAppUpdateService* update_service_; // Not owned
bartfab (slow) 2013/06/21 18:18:13 Nit: Full stop at end of comment.
xiyuan 2013/06/21 18:33:18 Done.
+
+ DISALLOW_COPY_AND_ASSIGN(KioskAppUpdateServiceTest);
+};
+
+IN_PROC_BROWSER_TEST_F(KioskAppUpdateServiceTest, AppUpdate) {
+ FireAppUpdateAvailable();
+
+ ExtensionTestMessageListener listener("app_update", false);
+ listener.WaitUntilSatisfied();
+}
+
+IN_PROC_BROWSER_TEST_F(KioskAppUpdateServiceTest, OsUpdate) {
+ g_browser_process->local_state()->SetBoolean(prefs::kRebootAfterUpdate, true);
+ FireUpdatedNeedReboot();
+
+ ExtensionTestMessageListener listener("os_update", false);
+ listener.WaitUntilSatisfied();
+}
+
+IN_PROC_BROWSER_TEST_F(KioskAppUpdateServiceTest, Periodic) {
+ g_browser_process->local_state()->SetInteger(
+ prefs::kUptimeLimit, base::TimeDelta::FromMinutes(30).InSeconds());
+
+ ExtensionTestMessageListener listener("periodic", false);
+ listener.WaitUntilSatisfied();
+}
+
+} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698