android component + release build broken due to VR |
||||
Issue description
Warning: com.google.vr.cardboard.AndroidNCompat: can't find referenced method 'void setVrModeEnabled(boolean,android.content.ComponentName)' in library class android.app.Activity
Warning: com.google.vr.cardboard.AndroidNCompat: can't find referenced method 'boolean isSustainedPerformanceModeSupported()' in library class android.os.PowerManager
Warning: com.google.vr.cardboard.AndroidNCompat: can't find referenced method 'void setSustainedPerformanceMode(boolean)' in library class android.view.Window
Warning: there were 3 unresolved references to library class members.
You probably need to update the library versions.
(http://proguard.sourceforge.net/manual/troubleshooting.html#unresolvedlibraryclassmember)
Error: Please correct the above warnings first
Repro by building chrome_public_apk with is_component_build = true, is_debug = false, is_java_debug = false. Apparently no bot does this combination of flags..
Note this has nothing to do with the VrShell that's currently being written. VrShell is still off by default.
,
Sep 26 2016
short term workaround is "is_java_debug = true"
,
Sep 26 2016
Thanks :)
,
Sep 29 2016
https://codereview.chromium.org/2377763003/
,
Sep 29 2016
bshe: why is vrshell stuff affecting the build if enable_vr_shell is false?
,
Sep 29 2016
I think it is because webvr also use gvr library(https://cs.chromium.org/chromium/src/device/vr/BUILD.gn?rcl=0&l=97). And gvr_common internally depends on gvr_base, which results the proguard issue. The reason it doesn't trigger the same issue for previous version is because gvr library was using reflection to check if these functions exist or not. I have verified locally that once we have N sdk available in chromium, we can then safely reland gvr 1.0 to chromium.
,
Nov 30 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by csharrison@chromium.org
, Sep 26 2016