Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| index 5ca863096b36d819eac0d67cdd363465992cb0c1..7d2c2d4404894bb08b2c8375a55cbba09b951769 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| @@ -405,10 +405,15 @@ public class WebappActivity extends FullScreenActivity { |
| // app is not directly launched from the home screen, as this interferes |
| // with the heuristic. |
| if (mWebappInfo.isLaunchedFromHomescreen()) { |
| + onWillUpdateLastUsedTime(storage); |
|
dominickn
2017/06/08 05:15:27
It would be good if you could add a quick test tha
pkotwicz
2017/06/12 20:34:46
How do you suggest that I add such a test? I don't
dominickn
2017/06/14 00:19:39
Ah yes, the integration test is sufficient with an
|
| + storage.setWasLaunched(); |
|
dominickn
2017/06/08 05:15:27
I think you need to setWasLaunched() before onWill
pkotwicz
2017/06/12 20:34:46
Actually, it is important for the WebApkActivity m
|
| storage.updateLastUsedTime(); |
| } |
| } |
| + /** Called prior to updating the "last used" time in WebappDataStorage. */ |
| + protected void onWillUpdateLastUsedTime(WebappDataStorage storage) {} |
| + |
| protected void initializeSplashScreenWidgets(int backgroundColor, Bitmap splashImage) { |
| Bitmap displayIcon = splashImage == null ? mWebappInfo.icon() : splashImage; |
| int minimiumSizeThreshold = getResources().getDimensionPixelSize( |