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

Side by Side Diff: chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.h

Issue 2449213006: Factor out ShouldIgnoreNavigation and add tests for the function (Closed)
Patch Set: 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 unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_EXTERNAL_PROTOCOL_DIALOG_H _ 5 #ifndef CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_EXTERNAL_PROTOCOL_DIALOG_H _
6 #define CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_EXTERNAL_PROTOCOL_DIALOG_H _ 6 #define CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_EXTERNAL_PROTOCOL_DIALOG_H _
7 7
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 13 matching lines...) Expand all
24 }; 24 };
25 25
26 // Shows ARC version of the dialog. Returns true if ARC is supported, running, 26 // Shows ARC version of the dialog. Returns true if ARC is supported, running,
27 // and in a context where it is allowed to handle external protocol. 27 // and in a context where it is allowed to handle external protocol.
28 bool RunArcExternalProtocolDialog(const GURL& url, 28 bool RunArcExternalProtocolDialog(const GURL& url,
29 int render_process_host_id, 29 int render_process_host_id,
30 int routing_id, 30 int routing_id,
31 ui::PageTransition page_transition, 31 ui::PageTransition page_transition,
32 bool has_user_gesture); 32 bool has_user_gesture);
33 33
34 bool ShouldIgnoreNavigationForTesting(ui::PageTransition page_transition);
35
34 GetActionResult GetActionForTesting( 36 GetActionResult GetActionForTesting(
35 const GURL& original_url, 37 const GURL& original_url,
36 const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers, 38 const mojo::Array<mojom::IntentHandlerInfoPtr>& handlers,
37 size_t selected_app_index, 39 size_t selected_app_index,
38 std::pair<GURL, std::string>* out_url_and_package); 40 std::pair<GURL, std::string>* out_url_and_package);
39 41
40 } // namespace arc 42 } // namespace arc
41 43
42 #endif // CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_EXTERNAL_PROTOCOL_DIALO G_H_ 44 #endif // CHROME_BROWSER_CHROMEOS_ARC_INTENT_HELPER_ARC_EXTERNAL_PROTOCOL_DIALO G_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/intent_helper/arc_external_protocol_dialog.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698