Skip to content
  1. May 13, 2020
  2. May 12, 2020
  3. May 11, 2020
    • Treehugger Robot's avatar
      963c11e3
    • Shuo Qian's avatar
    • Treehugger Robot's avatar
    • Tyler Gunn's avatar
      Ensure conference state is posted to Telecom for new conferences. · 1036237b
      Tyler Gunn authored
      There is an edge-case where an IMS conference fails to merge two
      participants into a conference, AND the conference event package comes in
      before the ImsCall indicates it is multiparty.  In this case, the CEP
      will have a single participant which means we'll treat the call as a
      standalone call.  ImsConference correctly sets the conference state, but
      this was previously just passed along the ConnectionService adapter
      to Telecom.  In this case the conference isn't even added to Telecom so
      the method call in effect does nothing.
      
      Changed to cache the conference state and ensure when adding the conference
      to Telecom that we call the setConferenceState method (yeah, could have
      put this in the ParcelableConference, but this is an edge case).
      
      Test: Wrote unit test to verify property is set.
      Bug: 134047008
      Fixes: 155097934
      Merged-In: I203880c593f6bac471a06eb9dcbb529d0bb75ba5
      Change-Id: Ib48437abb1295d147e227727ce4ff2996f28b5de
      1036237b
    • Hai Zhang's avatar
      Grant ACCESS_SHARED_LIBRARIES permission to Shell for testing. · 68cc951b
      Hai Zhang authored
      Bug: 154348597
      Test: presubmit
      Change-Id: I75f07b88c6eca9d2296d3bcdbcc43ad9ff382023
      Merged-In: Ic721288e8f1e1a1f255eca95b2efb7b6e0c7c83b
      68cc951b
    • Tyler Gunn's avatar
    • Shuo Qian's avatar
      Modify the default values to avoid 2 DB hysteresis DB limitation · e2faa8ff
      Shuo Qian authored
      Test: Treehugger
      Bug: 155026280
      Change-Id: Ic1af774b28629c7ab0e105f6ce198bfce8d160a8
      Merged-In: Ic1af774b28629c7ab0e105f6ce198bfce8d160a8
      (cherry picked from commit 66c51cda)
      e2faa8ff
    • Rambo Wang's avatar
    • Tyler Gunn's avatar
      Fix parceling and unparceling for URIs in Telecom. · 04e1fd15
      Tyler Gunn authored
      When testing parceling and unparceling of numbers with postdial digits
      it appears in parcelling to the dialer there were some cases where
      the commas were being URL encoded during parceling.
      
      Elsewhere in the platform the Uri.writeToParcel and
      Uri.CREATOR.createFromParcel methods are used.  Switching to those methods
      ensured that the parceling did not uri-encode on parcel/unparcel.
      GatewayInfo already used the right method to unparcel.
      
      Test: Unit/CTS tests.
      Bug: 152172598
      Fixes: 155098148
      Merged-In: I87b881e55a749e0ea46ba3b3c7d297856a744f37
      Change-Id: I32b7d049107cb3901fd934dc609541d1d2622a5a
      04e1fd15
    • Kweku Adams's avatar
      Add day-night-mode permission to Shell. · 6adfa79d
      Kweku Adams authored
      Without the permission "adb shell cmd uimode night <mode>" doesn't work.
      Adding the permission to the AndroidManifest file so the shell command
      will work.
      
      Bug: 155584597
      Test: run `adb shell cmd uimode night <no|yes|auto>`
      Change-Id: Ibfb145d762c73bba9de639d4050b9b26e4778cc9
      6adfa79d
    • Rambo Wang's avatar
      LTE+ indicator not shown when attached to CA network · 35c8eb19
      Rambo Wang authored
      NetworkTypeController (or DcTracker in aosp/master) will set lte
      display type to LTE_CA when current ServiceState fulfills one of the
      following conditions:
      - getDataNetworkType return NETWORK_TYPE_LTE_CA
      - isUsingCarrierAggregation return true
      
      NetworkRegistrationInfo#getAccessNetworkTechnology() always returns LTE
      when data rat is LTE CA. So the first condition will never get
      fulfilled.
      
      isUsingCarrierAggregation flag will only get updated when
      ServiceStateTracker calls RatRatcheter.ratchet without location change.
      This makes isUsingCarrierAggregation flag not always reliable to
      determine if carrier aggregation is on.
      
      The CL fix the issue by changing ServiceState#isUsingCarrierAggregation
      to return true when ServiceState#getCellBandwidths return more than one.
      The same condition is already used by RatRatcheter when set
      isUsingCarrierAggregation flag in RatRatcheter#ratchet.
      
      Bug: 154060784
      Test: atest com.android.internal.telephony.ServiceStateTest
      Test: atest android.telephony.cts.ServiceStateTest
      Merged-In: Ie6c07cb58d2a40eeb10e8f0131c98212ba463517
      Change-Id: Ie6c07cb58d2a40eeb10e8f0131c98212ba463517
      (cherry picked from commit 8a41bbf4)
      35c8eb19
    • Treehugger Robot's avatar
    • andrewlewis's avatar
      Fix 'associated' javadoc typos · 5b075b8f
      andrewlewis authored
      Change-Id: I4ff16ec3d6bd1d98bb04b647a38f12af87286a5e
      5b075b8f
    • Treehugger Robot's avatar
    • Treehugger Robot's avatar
      932b0b27
    • Chalard Jean's avatar
    • Jooyung Han's avatar
      InProcessTethering: set min_sdk_version · ace605ef
      Jooyung Han authored
      Modules contributing mainline modules (APK/APEX) should set
      min_sdk_version as well as apex_available.
      
      InProcessTethering is a part of com.android.tethering.inprocess,
      therefore min_sdk_version is set.
      
      Bug: 145796956
      Bug: 150999716
      Test: m
      Change-Id: I655211e0b9aa4bd7cb3718b6f567f0aa1a0176a5
      ace605ef
    • Shawn Lien's avatar
      97e6f46c
    • Chalard Jean's avatar
      Add the NOT_SUSPENDED capability to Ethernet - doc part · 65b75375
      Chalard Jean authored
      Test: ConnectivityServiceTest, EthernetManagerTest
      Bug: 150570873
      Change-Id: I151dfaa90b245dfc67d83516081b965284386d6a
      65b75375
    • Chalard Jean's avatar
      Fix a bug where VPNs start out suspended on cellular · bd13c96a
      Chalard Jean authored
      As NetworkAgent is in a transition where all agents need
      to include the NOT_SUSPENDED capability as part of their
      migration to the system API, ConnectivityService adds it
      forcefully to all agents that don't have the CELLULAR
      transport. This doesn't include VPNs when VPNs have some
      cellular network as their underlying network.
      
      The best way to solve this is to make sure the VPN
      capabilities reflect those of the underlying networks as
      far as the NOT_SUSPENDED capability is concerned. This
      is how they work for other similar capabilities.
      
      This also happens to contain a drive-by fix for an issue
      with a spurious capabilities callback is triggered when
      a VPN connects and it has any underlying network (which
      means almost always, because it will take the default
      network if it doesn't declare any). Fixing this was
      necessary to have a cogent test of this issue, but it
      could be moved to another patch or it could stay unfixed
      with some minor ajustment to the tests if judged too
      dangerous to include in R at this point.
      
      Test: New tests in this patch. Also manually tested with
            tcpdump as described in b/150570873.
      Bug: 150570873
      Change-Id: I3e4ff990c0d4825b21c7679be29a482a2d1324ec
      bd13c96a
  4. May 10, 2020
    • Paul Duffin's avatar
      Add lint-baseline.txt files for test base/runner/mock · ee5a4af4
      Paul Duffin authored
      In preparation for enabling API lint checking on java_sdk_library
      adds lint-baseline.txt containing all the existing issues for these
      APIs.
      
      The test-base and test-runner APIs are old and have been deprecated
      so there is little benefit in fixing those issues.
      
      The test-mock API while old has not been deprecated and so should be
      cleaned up. Bug 156175891 tracks that cleanup work.
      
      Bug: 156126315
      Bug: 156175891
      Test: m checkapi
      Change-Id: Ifb4669fcf7f2a36e5cb4d593bfc5d03330901ee9
      ee5a4af4
    • Remi NGUYEN VAN's avatar
      Send full LinkProperties to NetworkMonitor · f2945b5d
      Remi NGUYEN VAN authored
      Set the parcelSensitiveFields bit when sending LinkProperties to
      NetworkMonitor, so that the captive portal API URL is not lost.
      
      Test: atest ConnectivityServiceIntegrationTest (see followup change)
      Bug: 156062304
      Change-Id: Ifd4e9c02a6b9a2b2b8b254fc4da7bfb9e0a84550
      f2945b5d
  5. May 09, 2020
Loading