New issue
Advanced search Search tips

Issue 619137 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Jan 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Cache android intent information to prevent roundtrip to android when chrome should handle url

Project Member Reported by zentaro@chromium.org, Jun 10 2016

Issue description

Optimization to prevent an IPC roundtrip in the cases that chrome should handle a clicked url.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7a610a093d5bd05845a4468a73e04f0e1a7e32f8

commit 7a610a093d5bd05845a4468a73e04f0e1a7e32f8
Author: zentaro <zentaro@chromium.org>
Date: Fri Jun 10 22:30:56 2016

Add initial stub implementation of LocalActivityResolver.

ShouldChromeHandleUrl just returns true for now. In future it
will return true only if Chrome should handle this URL. Part of b\29248657

BUG= 619137 

Review-Url: https://codereview.chromium.org/2059573002
Cr-Commit-Position: refs/heads/master@{#399299}

[modify] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc.gypi
[modify] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc/BUILD.gn
[add] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc/intent_helper/local_activity_resolver.cc
[add] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc/intent_helper/local_activity_resolver.h

Project Member

Comment 2 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7a610a093d5bd05845a4468a73e04f0e1a7e32f8

commit 7a610a093d5bd05845a4468a73e04f0e1a7e32f8
Author: zentaro <zentaro@chromium.org>
Date: Fri Jun 10 22:30:56 2016

Add initial stub implementation of LocalActivityResolver.

ShouldChromeHandleUrl just returns true for now. In future it
will return true only if Chrome should handle this URL. Part of b\29248657

BUG= 619137 

Review-Url: https://codereview.chromium.org/2059573002
Cr-Commit-Position: refs/heads/master@{#399299}

[modify] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc.gypi
[modify] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc/BUILD.gn
[add] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc/intent_helper/local_activity_resolver.cc
[add] https://crrev.com/7a610a093d5bd05845a4468a73e04f0e1a7e32f8/components/arc/intent_helper/local_activity_resolver.h

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a69ff524cf24d96b99f1b0185c40a0202e3838e0

commit a69ff524cf24d96b99f1b0185c40a0202e3838e0
Author: zentaro <zentaro@chromium.org>
Date: Mon Jun 27 16:59:26 2016

Add handler and mojo interface to accept android intent filters.

This adds the interface to recieve intent filters from android. Follow
up CL will store these and implement matching logic.

BUG= 619137 

Review-Url: https://codereview.chromium.org/2078683002
Cr-Commit-Position: refs/heads/master@{#402203}

[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/chrome/browser/chromeos/arc/arc_service_launcher.cc
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/arc_service_manager.cc
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/arc_service_manager.h
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/common/intent_helper.mojom
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/intent_helper/arc_intent_helper_bridge.cc
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/intent_helper/arc_intent_helper_bridge.h
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/intent_helper/local_activity_resolver.cc
[modify] https://crrev.com/a69ff524cf24d96b99f1b0185c40a0202e3838e0/components/arc/intent_helper/local_activity_resolver.h

Status: WontFix (was: Untriaged)
I think this is obsolete now.

Sign in to add a comment