Skip to content

Commit 9090589

Browse files
alancutterChromium LUCI CQ
authored andcommitted
Add "const" to the getters in web app LaunchProcess
Change-Id: I0831ecc4634b0e8f02f8057058206f2ec17aef6e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3216573 Auto-Submit: Alan Cutter <[email protected]> Reviewed-by: Eric Willigers <[email protected]> Commit-Queue: Eric Willigers <[email protected]> Cr-Commit-Position: refs/heads/main@{#930073}
1 parent 29caebc commit 9090589

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

chrome/browser/ui/web_applications/web_app_launch_manager.cc

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -174,13 +174,13 @@ class LaunchProcess {
174174
content::WebContents* Run();
175175

176176
private:
177-
const apps::ShareTarget* MaybeGetShareTarget();
177+
const apps::ShareTarget* MaybeGetShareTarget() const;
178178
std::tuple<GURL, bool /*is_file_handling*/> GetLaunchUrl(
179-
const apps::ShareTarget* share_target);
180-
WindowOpenDisposition GetNavigationDisposition();
179+
const apps::ShareTarget* share_target) const;
180+
WindowOpenDisposition GetNavigationDisposition() const;
181181
content::WebContents* MaybeLaunchSystemWebApp(const GURL& launch_url);
182182
std::tuple<Browser*, WindowOpenDisposition> EnsureBrowser();
183-
Browser* MaybeFindBrowserForLaunch();
183+
Browser* MaybeFindBrowserForLaunch() const;
184184
Browser* CreateBrowserForLaunch();
185185
content::WebContents* NavigateBrowser(
186186
Browser* browser,
@@ -243,7 +243,7 @@ content::WebContents* LaunchProcess::Run() {
243243
return web_contents;
244244
}
245245

246-
const apps::ShareTarget* LaunchProcess::MaybeGetShareTarget() {
246+
const apps::ShareTarget* LaunchProcess::MaybeGetShareTarget() const {
247247
bool is_share_intent =
248248
params_.intent &&
249249
(params_.intent->action == apps_util::kIntentActionSend ||
@@ -254,7 +254,7 @@ const apps::ShareTarget* LaunchProcess::MaybeGetShareTarget() {
254254
}
255255

256256
std::tuple<GURL, bool /*is_file_handling*/> LaunchProcess::GetLaunchUrl(
257-
const apps::ShareTarget* share_target) {
257+
const apps::ShareTarget* share_target) const {
258258
GURL launch_url;
259259
bool is_file_handling = false;
260260
bool is_note_taking_intent =
@@ -295,7 +295,7 @@ std::tuple<GURL, bool /*is_file_handling*/> LaunchProcess::GetLaunchUrl(
295295
return {launch_url, is_file_handling};
296296
}
297297

298-
WindowOpenDisposition LaunchProcess::GetNavigationDisposition() {
298+
WindowOpenDisposition LaunchProcess::GetNavigationDisposition() const {
299299
// Only CURRENT_TAB and NEW_FOREGROUND_TAB dispositions are supported for web
300300
// app launches.
301301
return params_.disposition == WindowOpenDisposition::CURRENT_TAB
@@ -337,7 +337,7 @@ std::tuple<Browser*, WindowOpenDisposition> LaunchProcess::EnsureBrowser() {
337337
return {browser, navigation_disposition};
338338
}
339339

340-
Browser* LaunchProcess::MaybeFindBrowserForLaunch() {
340+
Browser* LaunchProcess::MaybeFindBrowserForLaunch() const {
341341
if (params_.container == apps::mojom::LaunchContainer::kLaunchContainerTab) {
342342
return chrome::FindTabbedBrowser(
343343
&profile_, /*match_original_profiles=*/false,

0 commit comments

Comments
 (0)