Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] Unable to open links from other applications in Chromium #68

Closed
aeiouaeiouaeiouaeiouaeiouaeiou opened this issue Jun 30, 2022 · 8 comments
Closed

[BUG] Unable to open links from other applications in Chromium #68

aeiouaeiouaeiouaeiouaeiouaeiou opened this issue Jun 30, 2022 · 8 comments
Labels
bug Something isn't working

Comments

@aeiouaeiouaeiouaeiouaeiouaeiou
Copy link

To Reproduce
Steps to reproduce the behavior:

  1. Set Chromium as the default browser in the Safari settings.
  2. Open the PDF file in Preview.app that contains the HTTP links and click them.
  3. Nothing happens. The console shows an error lsboxd[7079]: @AE relay 4755524c:4755524c.

Desktop (please complete the following information):

  • OS: OS X Mountain Lion
  • Build 1016494
@aeiouaeiouaeiouaeiouaeiouaeiou aeiouaeiouaeiouaeiouaeiouaeiou added the bug Something isn't working label Jun 30, 2022
@pjpreilly
Copy link

pjpreilly commented Jun 30, 2022 •
edited
Loading

Probably related on OS X Lion....
The external links ( GoogleSearch & GoogleMaps ) do not open any link. No error msg is apparent in the console. This works on the current stable.

EDIT: I forgot to mention it was the 3rd party app PopClip having this issue.....

@Wowfunhappy
Copy link

I was not able to indicate this issue on official builds of Chromium 1016505 or 1016484 (the closest available on https://commondatastorage.googleapis.com/chromium-browser-snapshots/index.html?prefix=Mac) on High Sierra, indicating that this is likely a Chromium Legacy issue.

@Wowfunhappy
Copy link

Wowfunhappy commented Aug 7, 2022 •
edited
Loading

I can also confirm this is happening in the latest stable build (104.0.5112.79) on OS X 10.9.5.

In addition, opening an html file directly in Chromium does not work. Double clicking the file launches Chromium, but Chromium will act as though it was launched normally from the dock, so it will not load the file. I assume these are the same issue.

@scrutinizer11
Copy link

In fact, it can't open any external link, e.g. from Mail. In the latter case, clicking on View it on GitHub in the Mail message body opens Chromium but doesn't land me on the page.

@Wowfunhappy
Copy link

I think the issue stems from this commit: 2ac8ad1

@blueboxd
Copy link
Owner

@Wowfunhappy Thanks!
Reverted that commit and resolved in Canary.

But I forgot to do it with stable...will be fixed in next stable.
(We need -legacy specific versioning scheme to distinguish patches/builds within same Chromium version 🤔)

@pjpreilly
Copy link

But I forgot to do it with stable...will be fixed in next stable.

... & now Canary is more stable than....Stable...for now

@scrutinizer11
Copy link

The issue is still present. I use a Mac browser switcher and Open with. Trying to open the link in Chromium with Open with from Firefox opens another FF tab, and the switcher reports "Application Chromium was unable to handle the URL". The build is 104.0.5112.101

@Wowfunhappy Wowfunhappy mentioned this issue Aug 29, 2022
Closed
blueboxd pushed a commit that referenced this issue Aug 31, 2022
[Identifiability Study] Fix condition to run Reid Estimation
073f33f 
Identifiability Study should be enabled with Reid estimation for the estimator to run.

(cherry picked from commit 3d2a6bc)

Bug: 1330244
Change-Id: Iebfba34cbecf1a5e27fda60a2e4b0a48d9b27a1e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3780905
Reviewed-by: Antonio Sartori <antoniosartori@chromium.org>
Commit-Queue: Saeid Eid <saeideid@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1027179}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3789980
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Antonio Sartori <antoniosartori@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5195@{#68}
Cr-Branched-From: 7aa3f07-refs/heads/main@{#1027018}
blueboxd pushed a commit that referenced this issue Sep 29, 2022
@natiahlyi
[Passwords Import] Handle MAX_FILE_SIZE gracefully
efc3dbd 
This CL adds graceful handling of the status, that have been already
implemented on the backend, but was not included in mocks yet.

(cherry picked from commit 0a18357)

Bug: 1355796
Change-Id: Icdcb727193bb4f3a904cc0e1a7ed12133aa686d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3849607
Commit-Queue: Viktor Semeniuk <vsemeniuk@google.com>
Auto-Submit: Andrii Natiahlyi <natiahlyi@google.com>
Reviewed-by: Viktor Semeniuk <vsemeniuk@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1038679}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3852962
Auto-Submit: Elias Khsheibun <eliaskh@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5249@{#68}
Cr-Branched-From: 4f7bea5-refs/heads/main@{#1036826}
blueboxd pushed a commit that referenced this issue Nov 30, 2022
Added new strings for reader mode action chip
f7296ea 
(cherry picked from commit c38a889)

Bug: 1373898, 1375828
Change-Id: I79df365397b41357fd24eedee0ad01bb1952a51c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3950990
Reviewed-by: Shakti Sahu <shaktisahu@chromium.org>
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Commit-Queue: Salvador Guerrero Ramos <salg@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1060166}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3964734
Reviewed-by: Krishna Govind <govind@chromium.org>
Owners-Override: Krishna Govind <govind@chromium.org>
Commit-Queue: Krishna Govind <govind@chromium.org>
Cr-Commit-Position: refs/branch-heads/5359@{#68}
Cr-Branched-From: 27d3765-refs/heads/main@{#1058933}
blueboxd pushed a commit that referenced this issue Feb 8, 2023
@muyao-xu
Iterate on a copy of presentation routes in PresentationFrame::Reset()
a81edbb 
This is a speculative fix for a crash.

(cherry picked from commit 4fa590a)

Bug: 1219904
Change-Id: I4154d98d915f68456d5813703627da71f26278dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4120352
Reviewed-by: Takumi Fujimoto <takumif@chromium.org>
Commit-Queue: Muyao Xu <muyaoxu@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1086167}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4125523
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Muyao Xu <muyaoxu@google.com>
Cr-Commit-Position: refs/branch-heads/5481@{#68}
Cr-Branched-From: 130f3e4-refs/heads/main@{#1084008}
blueboxd pushed a commit that referenced this issue Mar 7, 2023
[Cormorant] add networking code for query web id
d163138 
(cherry picked from commit 9c616cc)

Bug: 1400503
Change-Id: Ic6e45647d07abacbd008bf2c09f180507b2c0e48
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4180024
Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org>
Reviewed-by: Dan H <harringtond@chromium.org>
Reviewed-by: Justin DeWitt <dewittj@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1098207}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4209751
Commit-Queue: Justin DeWitt <dewittj@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Jeffrey Cohen <jeffreycohen@chromium.org>
Cr-Commit-Position: refs/branch-heads/5563@{#68}
Cr-Branched-From: 3ac59a6-refs/heads/main@{#1097615}
blueboxd pushed a commit that referenced this issue Apr 14, 2023
Extend flag ios-popular-sites-improved-suggestions to M118.
0ad8738 
(cherry picked from commit 2bc1808)

Bug: 1420295
Change-Id: I5f6d62629fa735a751dc0a6697cdc8b6d0f2368c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4295140
Reviewed-by: Scott Yoder <scottyoder@google.com>
Commit-Queue: Benjamin Williams <bwwilliams@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1111041}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4298230
Cr-Commit-Position: refs/branch-heads/5615@{#68}
Cr-Branched-From: 9c6408e-refs/heads/main@{#1109224}
blueboxd pushed a commit that referenced this issue May 9, 2023
@mohamedamir
[Passwords] Record authentication time in the management bubble
4a67b15 
This CL is recording the time the user takes to complete the
authentication flow when navigating to the details view in the new
password management bubble.

(cherry picked from commit 1868f29)

Bug: 1427951, 1297513
Change-Id: I66370ed178535dd047b1f8a22fc34b5accd4626d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4370684
Reviewed-by: Viktor Semeniuk <vsemeniuk@google.com>
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1122325}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4373803
Auto-Submit: Mohamed Amir Yosef <mamir@chromium.org>
Commit-Queue: Viktor Semeniuk <vsemeniuk@google.com>
Cr-Commit-Position: refs/branch-heads/5672@{#68}
Cr-Branched-From: 5f2a724-refs/heads/main@{#1121455}
blueboxd pushed a commit that referenced this issue May 27, 2023
@kgrosu
[PCCT-SideSheet] Add UMAs for PCCT type and side-sheet resize
f3128a2 
Histograms for:
- CustomTabs.SideSheetResizeType - logged when a side-sheet is maximized/minimized - the side-sheet equivalent of the bottom-sheet CustomTabs.ResizeType2. Uses the same enum as CustomTabs.ResizeType2 but auto_* is currently not supported for side-sheet.
- CustomTabs.PartialCustomTabType - logs the type of a PCCT when it is created. During the lifetime of a PCCT this can be logged multiple times if there is a transition between the PCCT types (e.g. screen rotation, entering multi-window mode)

(cherry picked from commit 85c98f9)

Bug: 1439968
Change-Id: Icdac98261a5f17d98055d68d1ca9cc5455941e0a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4471896
Reviewed-by: Sinan Sahin <sinansahin@google.com>
Code-Coverage: Findit <findit-for-me@appspot.gserviceaccount.com>
Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org>
Commit-Queue: Kevin Grosu <kgrosu@google.com>
Cr-Original-Commit-Position: refs/heads/main@{#1135592}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4492834
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/5735@{#68}
Cr-Branched-From: 2f562e4-refs/heads/main@{#1135570}
blueboxd pushed a commit that referenced this issue Sep 10, 2023
[M117] Check screen min width to determine if the device is a tablet
faeb52f 
(cherry picked from commit d652810)

Bug: 1471398
Change-Id: I5390cee627742038b44fa8dc98ce82542b923591
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4765221
Commit-Queue: Shu Yang <shuyng@google.com>
Reviewed-by: Sirisha Kavuluru <skavuluru@google.com>
Reviewed-by: Theresa Sullivan <twellington@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1182131}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4774875
Auto-Submit: Shu Yang <shuyng@google.com>
Commit-Queue: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/branch-heads/5938@{#68}
Cr-Branched-From: 2b50cb4-refs/heads/main@{#1181205}
@Wowfunhappy Wowfunhappy mentioned this issue Mar 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Wowfunhappy @pjpreilly @scrutinizer11 @blueboxd @aeiouaeiouaeiouaeiouaeiouaeiou