* Luxemburgish (lb) was accidentially mapped to "lu" for all of the
  non-AOSP projects.
* Since the next sync will download "lb" now, just remove "lu"

Change-Id: I4437b4f0832415f47d823d65e4288c3b30e0ef04

Theme app according to our palette

See merge request !92

Bug: 331528424
Test: m Gallery2
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:2fda5c59abc1742f0848311dfdc24a946388251e)
Merged-In: Ib6c23c487a57644c0b4873350f84e043d5ad1528
Change-Id: Ib6c23c487a57644c0b4873350f84e043d5ad1528

This reduces final apk size from 12MB to 8.4MB.

Change-Id: I9799988456e51625b4e72e837aadaa82a9fc5ba4

Another instantiation issue, but this one wasn't causing a crash,
just breakage; exposure, contrast, etc. sliders were not functional,
because ParametricEditor instantiates its controls using reflection
and proguard is too aggressive about optimizing them out.

Issue: calyxos#2329
Change-Id: I8b9a68878e3a812557ddb915c03efe176e0a17a5

Test: Try to edit an image
Change-Id: Ib3b0526b973dca3a8d48e0f90c84f79e5d102454

RenderScript usage is now deprecated. This removes all usage in Gallery2
where currently it is essentially unused.

Fixes: 211520324
Test: m Gallery2
Change-Id: I6d9584ab5eeda0e22b0b7c9b816c91266a4003f6

Add proguard rules to keep ctors for ActivityState subclasses

Test: mm packages/apps/Gallery2

Change-Id: Ib3daa5515ab63ee37f855bcaf183e32fdc897412
Signed-off-by: Vitalii Dmitriev <vitalii.dmitriev@unikie.com>

* Until Glimpse has an editor
* There's many ways to do this, gutting stuff from
  the manifest just seemed the simplest and fastest
  for a hopefully temporary solution

Change-Id: I42d50463d99fd838fd15e4fd5473fd535f0d9b82

Use encodedAuthority instead of authority to match behavior in other
areas of AOSP and prevent potential URI-related shenanigans.

Change-Id: Ia9ce58130ca080a9912a21e94dbfa81fcca94399

https://github.com/LineageOS/android_frameworks_base/commit/0236d1c5cf434af7f0b1cd71b47316496d1faeca

 started
adding userId in the URIs of screenshots.

Check and only proceed if the user can access the URI. This fixes edit
action as well that is broken due to mentioned change.

Issue: calyxos#1942
Change-Id: I42833a6c61ed5296fbfeda3564400581c2b29b8a
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>

Change-Id: I3a0305b3c2252dd577cb4968c9c131c1998e1a2e
Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>

Fix a path traversal bug that allowed a malicious launcher app to access files in the private space of the Gallery2 app by injecting a bad path.
Fix: Check URI before passing the file out from SharedImageProvider and raise exception if it's not pointing to the relevant folder.
Bug: 198174170
Test: manual by following the reporters instructions

Change-Id: I74dce6c4794b3eafb922bb9a2f2516677cc5f4d8

Although sizeof(short) and sizeof(unsigned short)
should have the same value; use the correct type
to suppress the warning.

Bug: 261919363
Test: make tidy-packages-apps-Gallery2-jni_subset
Change-Id: I8244fe20e613d0dea2a74753deec64346d5e9f5b

This commit is part of a large scale change to fix errorprone
errors that have been downgraded to warnings in the android
source tree, so that they can be promoted to errors again.
The full list of changes include the following, but not all
will be present in any one individual commit:

BadAnnotationImplementation
BadShiftAmount
BanJNDI
BoxedPrimitiveEquality
ComparableType
ComplexBooleanConstant
CollectionToArraySafeParameter
ConditionalExpressionNumericPromotion
DangerousLiteralNull
DoubleBraceInitialization
DurationFrom
DurationTemporalUnit
EmptyTopLevelDeclaration
EqualsNull
EqualsReference
FormatString
FromTemporalAccessor
GetClassOnAnnotation
GetClassOnClass
HashtableContains
IdentityBinaryExpression
IdentityHashMapBoxing
InstantTemporalUnit
InvalidTimeZoneID
InvalidZoneId
IsInstanceIncompatibleType
JUnitParameterMethodNotFound
LockOnBoxedPrimitive
MathRoundIntLong
MislabeledAndroidString
MisusedDayOfYear
MissingSuperCall
MisusedWeekYear
ModifyingCollectionWithItself
NoCanIgnoreReturnValueOnClasses
NonRuntimeAnnotation
NullableOnContainingClass
NullTernary
OverridesJavaxInjectableMethod
ParcelableCreator
PeriodFrom
PreconditionsInvalidPlaceholder
ProtoBuilderReturnValueIgnored
ProtoFieldNullComparison
RandomModInteger
RectIntersectReturnValueIgnored
ReturnValueIgnored
SelfAssignment
SelfComparison
SelfEquals
SizeGreaterThanOrEqualsZero
StringBuilderInitWithChar
TreeToString
TryFailThrowable
UnnecessaryCheckNotNull
UnusedCollectionModifiedInPlace
XorPower

See https://errorprone.info/bugpatterns for more
information on the checks.

Bug: 253827323
Test: m RUN_ERROR_PRONE=true javac-check
Change-Id: If22f7e827ef4d3e5766d6cc9f6a78353fbfaa60f

Android T allows apps to declare a runtime receiver as not exported
by invoking registerReceiver with a new RECEIVER_NOT_EXPORTED flag;
receivers registered with this flag will only receive broadcasts from
the platform and the app itself. However to ensure developers can
properly protect their receivers, all apps targeting U or later
registering a receiver for non-system broadcasts must specify either
the exported or not exported flag when invoking #registerReceiver;
if one of these flags is not provided, the platform will throw a
SecurityException. This commit updates all the exposed receivers
with a new RECEIVER_EXPORTED_UNAUDITED flag to maintain the existing
behavior of exporting the receiver while also flagging the receiver
for audit before the U release.

Bug: 234659204
Test: Build
Change-Id: I202a6837a0496cf6761f4b8b3d92a0904af68741

The Nfc Beam feature is now removed.

Test: m Gallery2
Bug: 236980291
Change-Id: I0de0904e74a73471a785b87cafebccbe25e7450f
(cherry picked from commit 617969afea0cca94aa2e76e23edc068453383389)

Bug: 236980291
Test: build pass
Change-Id: Ia92f4afe8f07e21d962456ab7841d46545a9bb7a

After calling the function AndroidBitmap_lockPixels(), neither the
return value nor the returned pointers were checked for the possible
failure. So the pointers "source" & "destination" could have invalid
value.

Along with changes above, we also have some compile warnings and code
format cleaned.

Test: mmm packages/apps/Gallery2, presubmit check.

Change-Id: I28a5d678311cb963bacd6f7b96ca2e12d498d965
Signed-off-by: Jing Mike <jingyangliu@eswincomputing.com>

More apps should be aware of this intent.

Change-Id: Ib35f56806d943945dc9594959c5f4b9290b49eba

mFullRect is the rect where seek gesture is allowed.

Change-Id: Ic59f06c6e35c41da3202e129ab0fb179b6c95db2

Make the top 1/4 of the screen not respond to brightness/volume/seek
gestures, to allow dragging down statusbar without changing them.
Also increase the gesture treshold.

Change-Id: Ibc4241071dc3bf6702cec191c72751829c779ea8

To properly handle the case that a calling app's use of the crop intent
provides an output URI that already exists, e.g. one that is the same
as the input URI, ensure that writes are truncated.

If any apps are affected by this in practice, it's not clear which;
the generic crop intent does not appear to be widely-used, and calls
from AOSP apps or from the system use temporary files for this.

Change-Id: I2a468d9527b679b6aaef6af1521ec0467f58bf60

This serves no purpose whatsoever.

Change-Id: Ie2cf1ca622e6c6395e0aec9068a8a42ed8583a77

Change-Id: Ie58dc18680ea13373bfdd800c6397b33da5501b9

* Part of the new iconset made by Asher Simonds

Change-Id: I6a9817c40b5b0a9d897ae4512c127fa96672e11b

so the package can send notifications on T+

Bug: 194833441
Change-Id: Ie10df35cd9d07e0012eaf286a2a9b947c68b1864

MediaMetadataRetriever.release now declares a thrown IOException.

Bug: 200173116
Test: Should be a non-functional change (only relevant when using MediaDataSource).
Change-Id: I53665e192051ccc5103e9a61014fbed596a86891

Edit action expects content URI in data otherwise leads to crash due to NPE.

This is similar to what I7a1254e579cea38fd70fc13da81e1dae98f5f968 is doing with
screenshot activity in fw/base.

Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
Change-Id: Ifdded8862e4e08dfcb05391c1cb40cf1f4f7dbd8

Change-Id: I1a800b87b624c36dcd0d951ef79942929237b5ed

Change-Id: I3e9a1c46785d77f445c70292082926079fb27baf

Turns out nothing really uses platform APIs?

Change-Id: If0c7e168187f1e80457b82691e4f2557320c660a

This reverts Change-Id I4bf198d20bb2a341e459553d2b519b426fb5d538
"Gallery: use platform cert"

This reverts Change-Id I65aed83b3cea951202c001e556d9e4200f2bc81f
"Gallery2: set module privileged in Android.mk to delete photos on SDcard"

Change-Id: I4d55c49d0c022aab6358b79c50894658b7caf5d5

(×) is multiplication sign. It looks like some Camera apps specify
this as part of resolution.

Bug: 216176283
Test: make and Presubmit
Change-Id: I2836f7121293f1d594afaf76ef46d4e68190e355

Fixes Gallery2 ATTACH_DATA Intent Action Could be Intercepted to Grant
Access of Its Own Non-picture Private Files to OutSide.

Bug: 201535148
Test: manual
+ Will attach the test results to the above bug.

Change-Id: Ie47945cc3519099cb685a26eee81130c2766be80
(cherry picked from commit 35ab9cd99173079886c3cd636b0606f55fefd846)