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

Unified Diff: components/chrome_cleaner/public/interfaces/chrome_prompt.mojom

Issue 2932703006: Chrome Cleaner: Remove indirect base::FilePath mojo dependency. (Closed)
Patch Set: Address review comments, make tests pass Created 3 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: components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
diff --git a/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom b/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
index 07013fe06431897c6d0486f5d135eb8c1df37b3d..38f724414a2d35b5e6c8dcbaf12b062381e84353 100644
--- a/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
+++ b/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
@@ -4,7 +4,10 @@
module chrome_cleaner.mojom;
-import "mojo/common/file_path.mojom";
+// IMPORTANT NOTE: Avoid adding dependencies to other .mojom files.
+// Since the Chrome Cleaner is built as a binary independent of Chrome, the
+// mojo interface between the two should contain only primitive types to avoid
+// type incompabilities as the binaries' versions diverge.
[Extensible]
enum PromptAcceptance {
@@ -21,6 +24,10 @@ enum PromptAcceptance {
IGNORED = 4,
};
+struct FilePath {
+ array<uint16> value;
+};
+
// Service provided by Chrome to prompt the user to start a cleanup if the
// Chrome Cleanup Tool detects unwanted software on the system.
interface ChromePrompt {
@@ -29,6 +36,6 @@ interface ChromePrompt {
// will be deleted by the Chrome Cleanup Tool.
// Returns:
// - prompt_acceptance: indicates if the user accepted the prompt.
- PromptUser(array<mojo.common.mojom.FilePath> files_to_delete)
+ PromptUser(array<FilePath> files_to_delete)
=> (PromptAcceptance prompt_acceptance);
};

Powered by Google App Engine
This is Rietveld 408576698