android: add screenSize to AndroidManifest's configChanges (thanks, Daniel!).
authorRyan C. Gordon <icculus@icculus.org>
Fri, 19 May 2017 15:06:05 -0400
changeset 110181f151426c2ca
parent 11017 411bf9f7f908
child 11019 97839b6bbd46
android: add screenSize to AndroidManifest's configChanges (thanks, Daniel!).

Fixes Bugzilla #3448.

"Starting with Android API 13, another configuration change must be declared
to be handled by the app if it is desired to not let the system handle
rotation itself:

https://developer.android.com/guide/topics/resources/runtime-changes.html#HandlingTheChange

This will have effect if either a developer or SDL per default will set target
API > 12. Currently it is set to 12, but this might change in the future, like
when applying this patch: https://bugzilla.libsdl.org/show_bug.cgi?id=3445

The effect of not having this change applied is that the SDL app is destroyed
upon rotation. Even when the manifest has an additional entry to e.g. always
stay in landscape mode, the onDestroy() call will happen on devices that are
portrait per default, when switching off screen due to power save. The device
will then (at least try to) rotate into portrait to show the portrait screen
lock after resume.

I believe it is safe to apply this patch even with target API still set to 12,
the additional parameter is simply ignored."
android-project/AndroidManifest.xml
     1.1 --- a/android-project/AndroidManifest.xml	Fri May 19 14:51:03 2017 -0400
     1.2 +++ b/android-project/AndroidManifest.xml	Fri May 19 15:06:05 2017 -0400
     1.3 @@ -36,7 +36,7 @@
     1.4                   android:hardwareAccelerated="true" >
     1.5          <activity android:name="SDLActivity"
     1.6                    android:label="@string/app_name"
     1.7 -                  android:configChanges="keyboardHidden|orientation"
     1.8 +                  android:configChanges="keyboardHidden|orientation|screenSize"
     1.9                    >
    1.10              <intent-filter>
    1.11                  <action android:name="android.intent.action.MAIN" />