Remove the first launch from home screen requirement for notification deep linking.

Remove the first launch from home screen requirement for notification deep linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement. Now sites which have
been added to homescreen OR launched from homescreen within the
last ten days are permitted to be deep-linked from a notification.

BUG=541711
Committed: https://crrev.com/b4abc9d04c20c2d83124a1b9cc3960e8f4e48bdb
Cr-Commit-Position: refs/heads/master@{#386381}

[dominickn, 2016-04-11 00:01:07]

Description was changed from

==========
Remove the first launch from home screen requirement for notification deep
linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement, so that adding a site
to home screen is counted as a 'launch' and permits deep linking.

BUG=541711
==========

to

==========
Remove the first launch from home screen requirement for notification deep
linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement. Now sites which have
been added to homescreen OR launched from homescreen within the
last ten days are permitted to be deep-linked from a notification.

BUG=541711
==========

[dominickn, 2016-04-11 00:08:29]

dominickn@chromium.org changed reviewers:
+ mlamouri@chromium.org, yfriedman@chromium.org

[dominickn, 2016-04-11 00:08:29]

mlamouri: PTAL at chrome/browser/webapps

yfriendman: PTAL at junit/

dfalcantara@ is away this week (he reviewed all the previous patches for this).
I'd like to get this into the M51 branch if at all possible. :)

This CL removes a launched property from the WebappDataStorage shared
preference, and all other uses of it. This is a PM suggested change to deep
linking from notifications (where a notification can open a web app on the
homescreen in the nice standalone app frame). The change will allow
notifications to open the web app in the standalone frame if the app has been
installed - but not launched - within the last ten days. Previously the app had
to be launched for the deep link to work.

[mlamouri (slow - plz ping), 2016-04-11 13:21:56]

lgtm

[Yaron, 2016-04-11 13:57:07]

lgtm

[dominickn, 2016-04-11 14:01:36]

The CQ bit was checked by dominickn@chromium.org

[commit-bot: I haz the power, 2016-04-11 14:01:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1872983002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1872983002/1

[commit-bot: I haz the power, 2016-04-11 14:41:00]

Description was changed from

==========
Remove the first launch from home screen requirement for notification deep
linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement. Now sites which have
been added to homescreen OR launched from homescreen within the
last ten days are permitted to be deep-linked from a notification.

BUG=541711
==========

to

==========
Remove the first launch from home screen requirement for notification deep
linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement. Now sites which have
been added to homescreen OR launched from homescreen within the
last ten days are permitted to be deep-linked from a notification.

BUG=541711
==========

[commit-bot: I haz the power, 2016-04-11 14:41:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-11 14:42:11]

Description was changed from

==========
Remove the first launch from home screen requirement for notification deep
linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement. Now sites which have
been added to homescreen OR launched from homescreen within the
last ten days are permitted to be deep-linked from a notification.

BUG=541711
==========

to

==========
Remove the first launch from home screen requirement for notification deep
linking.

https://crrev.com/1867543002 implemented notification deep linking for
Android, with the requirement that a web app be explicily launched from
home screen before it would be permitted to have deep links from
notifications. This CL removes this requirement. Now sites which have
been added to homescreen OR launched from homescreen within the
last ten days are permitted to be deep-linked from a notification.

BUG=541711

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

[commit-bot: I haz the power, 2016-04-11 14:42:12]

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