Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2ms per navigation.

BUG=609122
TEST=None

Committed: https://crrev.com/e3343a3007162a60ecffb8a242eabb9e941c7280
Cr-Commit-Position: refs/heads/master@{#398194}

[pkotwicz, 2016-06-03 23:05:22]

Patchset #1 (id:1) has been deleted

[pkotwicz, 2016-06-03 23:05:40]

Description was changed from

==========
Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2 ms per navigation.

BUG=609122
TEST=None
==========

to

==========
Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2ms per navigation.

BUG=609122
TEST=None
==========

[pkotwicz, 2016-06-03 23:07:06]

pkotwicz@chromium.org changed reviewers:
+ mariakhomenko@chromium.org

[pkotwicz, 2016-06-03 23:07:06]

Maria, can you please take a look?

[Maria, 2016-06-06 18:56:01]

On 2016/06/03 23:07:06, pkotwicz wrote:
> Maria, can you please take a look?

lgtm

This is fine as-is. If you wanted to make it even faster, you could actually
speed it up by adding a cache of size 1 for URL to the set of resolve infos.
Then when we do the next navigation, we would often already have previous
navigation infos since it would probably have the referrer URL in the cache with
its infos.

[pkotwicz, 2016-06-07 01:07:50]

The CQ bit was checked by pkotwicz@chromium.org

[pkotwicz, 2016-06-07 01:07:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
mariakhomenko@chromium.org
Link to the patchset: https://codereview.chromium.org/2035203002/#ps40001
(title: " ")

[pkotwicz, 2016-06-07 01:08:08]

The CQ bit was unchecked by pkotwicz@chromium.org

[pkotwicz, 2016-06-07 01:11:16]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2016-06-07 01:11:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2035203002/40001

[commit-bot: I haz the power, 2016-06-07 01:54:11]

Description was changed from

==========
Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2ms per navigation.

BUG=609122
TEST=None
==========

to

==========
Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2ms per navigation.

BUG=609122
TEST=None
==========

[commit-bot: I haz the power, 2016-06-07 01:54:12]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-07 01:56:24]

Description was changed from

==========
Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2ms per navigation.

BUG=609122
TEST=None
==========

to

==========
Make ExternalNavigationHandler#intentsHaveSameResolvers() faster

ExternalNavigationHandler#intentsHaveSameResolvers() will be called on each
WebAPK navigation. Removing the duplicate call to
PackageManager#queryIntentActivities() saves 2ms per navigation.

BUG=609122
TEST=None

Committed: https://crrev.com/e3343a3007162a60ecffb8a242eabb9e941c7280
Cr-Commit-Position: refs/heads/master@{#398194}
==========

[commit-bot: I haz the power, 2016-06-07 01:56:25]

Patchset 2 (id:??) landed as
https://crrev.com/e3343a3007162a60ecffb8a242eabb9e941c7280
Cr-Commit-Position: refs/heads/master@{#398194}