Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c5c294b9 authored by Remi NGUYEN VAN's avatar Remi NGUYEN VAN Committed by Android (Google) Code Review
Browse files

Merge "Use FLAG_IMMUTABLE on NetworkStackNotifier intents" into rvc-dev

parents 4e34267c 1acc0030

src/com/android/networkstack/NetworkStackNotifier.java

+2 −2
Original line number Diff line number Diff line
@@ -239,7 +239,7 @@ public class NetworkStackNotifier { 
                    .setContentText(res.getString(R.string.tap_for_info))

                    .setContentIntent(mDependencies.getActivityPendingIntent(

                            getContextAsUser(mContext, UserHandle.CURRENT),

                            infoIntent, PendingIntent.FLAG_UPDATE_CURRENT));

                            infoIntent, PendingIntent.FLAG_IMMUTABLE));



            networkStatus.mShownNotification = NOTE_VENUE_INFO;

        } else if (showValidated) {
@@ -252,7 +252,7 @@ public class NetworkStackNotifier { 
                    .setContentIntent(mDependencies.getActivityPendingIntent(

                            getContextAsUser(mContext, UserHandle.CURRENT),

                            new Intent(Settings.ACTION_WIFI_SETTINGS),

                            PendingIntent.FLAG_UPDATE_CURRENT));

                            PendingIntent.FLAG_IMMUTABLE));



            networkStatus.mShownNotification = NOTE_CONNECTED;

        } else {

tests/unit/src/com/android/networkstack/NetworkStackNotifierTest.kt

+6 −3
Original line number Diff line number Diff line
@@ -22,7 +22,7 @@ import android.app.NotificationManager 
import android.app.NotificationManager.IMPORTANCE_DEFAULT

import android.app.NotificationManager.IMPORTANCE_NONE

import android.app.PendingIntent

import android.app.PendingIntent.FLAG_UPDATE_CURRENT

import android.app.PendingIntent.FLAG_IMMUTABLE

import android.content.Context

import android.content.Intent

import android.content.res.Resources
@@ -57,6 +57,7 @@ import org.junit.runner.RunWith 
import org.mockito.ArgumentCaptor

import org.mockito.ArgumentMatchers.anyInt

import org.mockito.ArgumentMatchers.eq

import org.mockito.ArgumentMatchers.intThat

import org.mockito.Captor

import org.mockito.Mock

import org.mockito.Mockito.any
@@ -188,7 +189,8 @@ class NetworkStackNotifierTest { 
        assertEquals(CHANNEL_CONNECTED, note.channelId)

        assertEquals(timeout, note.timeoutAfter)

        verify(mDependencies).getActivityPendingIntent(

                eq(mCurrentUserContext), mIntentCaptor.capture(), eq(FLAG_UPDATE_CURRENT))

                eq(mCurrentUserContext), mIntentCaptor.capture(),

                intThat { it or FLAG_IMMUTABLE != 0 })

    }



    private fun verifyCanceledNotificationAfterNetworkLost() {
@@ -279,7 +281,8 @@ class NetworkStackNotifierTest { 


        verify(mNm).notify(eq(TEST_NETWORK_TAG), mNoteIdCaptor.capture(), mNoteCaptor.capture())

        verify(mDependencies).getActivityPendingIntent(

                eq(mCurrentUserContext), mIntentCaptor.capture(), eq(FLAG_UPDATE_CURRENT))

                eq(mCurrentUserContext), mIntentCaptor.capture(),

                intThat { it or FLAG_IMMUTABLE != 0 })

        verifyVenueInfoIntent(mIntentCaptor.value)

        verifyCanceledNotificationAfterDefaultNetworkLost()

    }