Camera app update removed support for v64 |
|||||||
Issue descriptionChromeOS version: 64 or earlier ChromeOS device model: any Case#: 15677819 Description: Camera app hfhhnacclhffhdffklopdkcgdhifgngh update changed requirement to minimum_chrome_version "65.0.0.0" and now v64- couldn't install this application at all, even if they had it before. Maybe there should be at least some "compat" version fork for now? Steps to reproduce: 1. Enroll device with v64 2. Look for Camera app Current Behavior / Reproduction: Couldn't find Camera app nor in apps, nor in Webstore. Going to app page directly shows "Not compatible" Expected Behavior: Being able to use Camera app in some way Drive link to logs:
,
May 5 2018
+ Naveen for PM awareness It's highly unusual for us to restrict a standard app to the very latest version of Chrome OS. Can we understand: 1) why was this restriction put in place to begin with? Are there APIs or issues with 64 and older that require 65 in order for the camera to work? 2) If this is for the camera's video recording capabilities and requires 65, can we instead wrap the record function in a 65 requirement. For your understanding, it is not unusual for our very large and important EDU customers to pin to older versions of Chrome OS for a period of time and not update to the latest stable. This is especially true during testing season when schools are mostly concerned about the stability of the units. Thus we cannot expect all devices to be running the very latest stable release. How quickly can we get this fixed with the Camera app? Can we revert to the older version for now until a version of the app that checks for 65 at runtime in order to enable video recording is developed?
,
May 5 2018
FYR, https://chromium-review.googlesource.com/c/apps/camera/+/990785 Please refer to shenghao@'s and jcliang@'s comments for the possible impacts using either one of the following fixes. The possible fix would be either: 1. Simply update the manifest.json to allow Camera app running on older versions (video recording performance may not be good on earlier Chrome versions on some devices.) 2. Or disable video recording capability to run on Chrome versions earlier than 65.
,
May 6 2018
Can we not serve different versions of the app based on the Chrome browser version? I thought there was the ability to do so in the chrome web store. If that's not possible, I would recommend we go with option (1) which Yuli proposed unless Omri has other suggestions.
,
May 6 2018
I would prefer (1) in #3. I can push a new version for that tomorrow if no one objects.
,
May 6 2018
#1 is fine as long as it's a performance issue and app is still stable.
,
May 7 2018
Uploaded https://chromium-review.googlesource.com/c/apps/camera/+/1045963
,
May 7 2018
The following revision refers to this bug: https://chromium.googlesource.com/apps/camera/+/6f45d84dba53b1831d5f067898a8a5ceeffd6124 commit 6f45d84dba53b1831d5f067898a8a5ceeffd6124 Author: Sheng-Hao Tsao <shenghao@google.com> Date: Mon May 07 08:41:28 2018 Update min supported chrome version EDU customers might pin to older version of ChromeOS and not update to the latest stable version. This CL changes the min supported version back to 59 so that those users can still use the camera app. BUG= 840027 TEST=Verified on cyan R59-9460.60.0 that still capture and recording are working Change-Id: I7e7d4ecb511d447edd185c6766fb0580f8791dfc Reviewed-on: https://chromium-review.googlesource.com/1045963 Reviewed-by: Ricky Liang <jcliang@chromium.org> Reviewed-by: yuli <yuli@chromium.org> Tested-by: Sheng-hao Tsao <shenghao@google.com> [modify] https://crrev.com/6f45d84dba53b1831d5f067898a8a5ceeffd6124/src/manifest-tests.json [modify] https://crrev.com/6f45d84dba53b1831d5f067898a8a5ceeffd6124/src/manifest.json
,
May 7 2018
The following revision refers to this bug: https://chromium.googlesource.com/apps/camera/+/b1e14ffbd4cd8b396bf28ca987fd519f6c785f85 commit b1e14ffbd4cd8b396bf28ca987fd519f6c785f85 Author: Sheng-Hao Tsao <shenghao@google.com> Date: Mon May 07 10:21:11 2018 Update min supported chrome version EDU customers might pin to older version of ChromeOS and not update to the latest stable version. This CL changes the min supported version back to 59 so that those users can still use the camera app. BUG= 840027 TEST=Verified on cyan R59-9460.60.0 that still capture and recording are working Change-Id: I7e7d4ecb511d447edd185c6766fb0580f8791dfc Reviewed-on: https://chromium-review.googlesource.com/1046866 Reviewed-by: Sheng-hao Tsao <shenghao@google.com> Tested-by: Sheng-hao Tsao <shenghao@google.com> [modify] https://crrev.com/b1e14ffbd4cd8b396bf28ca987fd519f6c785f85/src/manifest-tests.json [modify] https://crrev.com/b1e14ffbd4cd8b396bf28ca987fd519f6c785f85/src/manifest.json
,
May 7 2018
5.0.1 has been pushed to 10% now.
,
May 11 2018
Our customer confirmed that the camera is showing now without the error.
,
May 12 2018
1. Makes sense to me to
,
May 14 2018
5.0.1 has been pushed to 100%.
,
Jan 15
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f45d84dba53b1831d5f067898a8a5ceeffd6124 commit 6f45d84dba53b1831d5f067898a8a5ceeffd6124 Author: Sheng-Hao Tsao <shenghao@google.com> Date: Mon May 07 08:41:28 2018 Update min supported chrome version EDU customers might pin to older version of ChromeOS and not update to the latest stable version. This CL changes the min supported version back to 59 so that those users can still use the camera app. BUG= 840027 TEST=Verified on cyan R59-9460.60.0 that still capture and recording are working Change-Id: I7e7d4ecb511d447edd185c6766fb0580f8791dfc Reviewed-on: https://chromium-review.googlesource.com/1045963 Reviewed-by: Ricky Liang <jcliang@chromium.org> Reviewed-by: yuli <yuli@chromium.org> Tested-by: Sheng-hao Tsao <shenghao@google.com> [modify] https://crrev.com/6f45d84dba53b1831d5f067898a8a5ceeffd6124/src/manifest-tests.json [modify] https://crrev.com/6f45d84dba53b1831d5f067898a8a5ceeffd6124/src/manifest.json |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by y...@chromium.org
, May 5 2018