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

Unified Diff: chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.h

Issue 2480203002: ui: Cleanup class/struct forward declarations (Closed)
Patch Set: Sync CL to position 430550 Created 4 years, 1 month 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/arc/intent_helper/arc_external_protocol_dialog.h
diff --git a/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.h b/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.h
index f2de5b8cc6e2776f8f19a6e281ab5f047571f0fc..987ad5de0647e037341c11fd79072154d3e82231 100644
--- a/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.h
+++ b/chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.h
@@ -16,10 +16,22 @@ class GURL;
namespace arc {
+// An enum returned from GetAction function. This is visible for testing.
enum class GetActionResult {
+ // ARC cannot handle the |original_url|, and the URL does not have a fallback
+ // http(s) URL. Chrome should show the "Google Chrome OS can't open the page"
+ // dialog now.
SHOW_CHROME_OS_DIALOG,
+ // ARC cannot handle the |original_url|, but the URL did have a fallback URL
+ // which Chrome can handle. Chrome should show the fallback URL now.
OPEN_URL_IN_CHROME,
+ // ARC can handle the |original_url|, and one of the ARC activities is a
+ // preferred one. ARC should handle the URL now.
HANDLE_URL_IN_ARC,
+ // Chrome should show the disambig UI because 1) ARC can handle the
+ // |original_url| but none of the ARC activities is a preferred one, or
+ // 2) there are two or more browsers (e.g. Chrome and a browser app in ARC)
+ // that can handle a fallback URL.
ASK_USER,
};

Powered by Google App Engine
This is Rietveld 408576698