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

Unified Diff: extensions/browser/preload_check_test_util.h

Issue 2693373003: PreloadCheck class for extension pre-install checks (Closed)
Patch Set: review ready Created 3 years, 9 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: extensions/browser/preload_check_test_util.h
diff --git a/extensions/browser/preload_check_test_util.h b/extensions/browser/preload_check_test_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..60e82441acad578d6f3631b0268ad4a3b79fa90a
--- /dev/null
+++ b/extensions/browser/preload_check_test_util.h
@@ -0,0 +1,38 @@
+// Copyright 2017 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.
+
+#ifndef EXTENSIONS_BROWSER_PRELOAD_CHECK_TEST_UTIL_H_
+#define EXTENSIONS_BROWSER_PRELOAD_CHECK_TEST_UTIL_H_
+
+#include "base/run_loop.h"
+#include "extensions/browser/preload_check.h"
+
+namespace extensions {
+
+// Provides a callback method for a PreloadCheck that stores its results.
+class PreloadCheckObserver {
+ public:
+ PreloadCheckObserver();
+ virtual ~PreloadCheckObserver();
+
+ virtual void OnCheckComplete(PreloadCheck::Errors errors);
+
+ void Wait() {
+ if (call_count_)
+ return;
+
+ base::RunLoop().RunUntilIdle();
Devlin 2017/03/08 03:01:48 We already have the object file, so may as well mo
michaelpg 2017/03/09 01:53:30 Doh.
+ }
+
+ const PreloadCheck::Errors& errors() const { return errors_; }
+ int call_count() const { return call_count_; }
+
+ private:
+ PreloadCheck::Errors errors_;
+ int call_count_;
Devlin 2017/03/08 03:01:49 DISALLOW_COPY_AND_ASSIGN
michaelpg 2017/03/09 01:53:30 Done.
+};
+
+} // namespace extensions
+
+#endif // EXTENSIONS_BROWSER_PRELOAD_CHECK_TEST_UTIL_H_

Powered by Google App Engine
This is Rietveld 408576698