Skip to content
Snippets Groups Projects
  1. May 13, 2020
    • Jooyung Han's avatar
      Set min_sdk_version for updatable mainline modules · 5f9f31c1
      Jooyung Han authored
      Modules contributing mainline modules (APK/APEX) should set
      min_sdk_version as well as apex_available.
      
      For now setting min_sdk_version doesn't change build outputs.
      But build-time checks will be added soon.
      
      Bug: 145796956
      Bug: 150999716
      Test: m
      Merged-In: Ifaecb49a47a1f43edea3ea06e1cf704a177d1044
      Change-Id: Ifaecb49a47a1f43edea3ea06e1cf704a177d1044
      (cherry picked from commit 33aa294e96f13906f596e427b96652fe80cf199b)
      5f9f31c1
  2. May 08, 2020
  3. May 07, 2020
    • Mark Chien's avatar
    • Mark Chien's avatar
      Refactor the EntitlementManager · 92afd387
      Mark Chien authored
      1. Change ArraySet usage to BitSet
      2. Change mCellularUpstreamPermitted to mLastCellularUpstreamPermitted.
      
      Before this change:
      a member variable(mCellularUpstreamPermitted) is
      used to check whether cellular upstream is permitted, the code must
      ensure to update this variable once entitlement result is changed or the
      entitlement check is triggered but does not have a result yet.
      
      In this change:
      Instead of storing the information about whether cellular is permitted in
      a member variable. The information is recalculated every time when user
      call isCellularUpstreamPermitted(). Now isCellularUpstreamPermitted() is
      always be used to check whether cellular upstream is permitted no matter
      inside or outside EntitlementManager.
      This make the code be easier to maintain that we do not need to care
      when mCellularUpstreamPermitted need to be updated because the
      information would be recalculated every time. And the recalculation is
      lock free because this is only used inside tethering while running in
      the same thread.
      
      Bug: 141256482
      Test: atest TetheringTests
      
      Merged-In: Ic83f42ff4eec38adf039d55d80fcb9b0f16373cc
      Change-Id: Ic83f42ff4eec38adf039d55d80fcb9b0f16373cc
      92afd387
    • Jeongik Cha's avatar
      Use stable networkstack-aidl-interfaces · 1aba7987
      Jeongik Cha authored
      Test: m nothing
      Bug: 133526962
      Original-Change: https://android-review.googlesource.com/1301313
      Merged-In: I507f40866d04db5ed3361831e01eaa4dfaf20bed
      Change-Id: I507f40866d04db5ed3361831e01eaa4dfaf20bed
      1aba7987
    • Mark Chien's avatar
      Test tethering log dump · c96fbe0a
      Mark Chien authored
      Bug: 145490751
      Test: atest TetheringTests
      Merged-In: I01fc6969041711f7a15880144ee5eac591086ecd
      Change-Id: I01fc6969041711f7a15880144ee5eac591086ecd
      c96fbe0a
    • markchien's avatar
      Override tethering module APK-in-APEX for Go variant · 9548c858
      markchien authored
      Bug: 155604224
      Test: build
      Change-Id: I4147173b5f3668491ff9cb7f1f86715b036d6d4b
      9548c858
  4. May 06, 2020
  5. May 05, 2020
    • Anton Hansson's avatar
      Enable api lint and check_last_api for modules · 5b0be161
      Anton Hansson authored
      This adds checking of module api compatibility to the individual module
      api rules. Until now, this checking has been done via the monolithic
      metalava runs which we are aiming to get rid of.
      
      Now is a good time to do this because we can compare them to the just
      finalized version 30 API, which we have no diffs with. Baseline the
      existing wifi failures that metalava fails to find in the previous API.
      
      Bug: 144149403
      Test: m checkapi
      Change-Id: Id222895daa3a769c265965b052a17d5a1ca18462
      5b0be161
    • Junyu Lai's avatar
      [SP18.6] Make offload controller poll interval configurable · e1a04c04
      Junyu Lai authored
      Test: atest TetheringTests
      Bug: 149467454
      
      Change-Id: I0b07a0b520dedb479bf863fbfe898ae85b84b0f3
      Merged-In: I0b07a0b520dedb479bf863fbfe898ae85b84b0f3
      (cherry picked from commit 8371fa281839d5b1a1027936f2f83e59bf5301a3)
      e1a04c04
    • junyulai's avatar
      [SP18.5] Create offload controller poll interval to resource · 2a5f2142
      junyulai authored
      Test: atest TetheringConfigurationTest
      Bug: 149467454
      Change-Id: I8b4ad920a4945504914d3741a9fba5c096fbf452
      Merged-In: I8b4ad920a4945504914d3741a9fba5c096fbf452
      (cherry-picked from aosp/1299413)
      2a5f2142
    • junyulai's avatar
      [SP18.4] Add unit test for polling network stats in OffloadController · 14722baf
      junyulai authored
      Test: atest OffloadControllerTest
      Bug: 149467454
      Change-Id: I9b9c9c096a2366aaf383d5c2d567db6682f02dad
      Merged-In: I9b9c9c096a2366aaf383d5c2d567db6682f02dad
      (cherry-picked from aosp/1295347)
      14722baf
    • junyulai's avatar
      [SP18.3] Adapt TestableNetworkStatsProviderCbBinder · b7011d22
      junyulai authored
      This is a no-op refactoring to adapt new test provider callback.
      Also this patch adapts TestLooper to allow better control on
      delay messages that will be verified in subsequent tests.
      
      Test: atest OffloadControllerTest
      Bug: 149467454
      Change-Id: Icfd6ff289d6689ae2d5753d3fe472516c808dc7a
      Merged-In: Icfd6ff289d6689ae2d5753d3fe472516c808dc7a
      (cherry-picked from aosp/1295346)
      b7011d22
  6. May 02, 2020
  7. May 01, 2020
  8. Apr 30, 2020
    • Anton Hansson's avatar
      Rename module dist files · d7f1fabc
      Anton Hansson authored
      This makes the filenames of the disted artifacts (api txts and stubs)
      match the module name of the modules they're from. This matches the
      naming scheme used by java_sdk_library, which should make the future
      transition to this build rule easier.
      
      Bug: 149293194
      Test: lunch sdk_phone_armv7 && m sdk dist && find out/dist/apistubs
      Change-Id: I076f30931bf2524d57703873cd7de25b3f23b457
      d7f1fabc
    • Anton Hansson's avatar
      Fix tethering module lib stub default · dc8e0fc1
      Anton Hansson authored
      It was using the systemapi stub defaults, but should be using the
      module_lib default.
      
      Bug: 144149403
      Test: m
      Change-Id: Iaab154d9d71900284d92d518a086fc1227c00d5c
      dc8e0fc1
    • Junyu Lai's avatar
      Merge changes I38628dad,I2467b647 into rvc-dev · 027c80d2
      Junyu Lai authored
      * changes:
        [SP18.1] add dependency object to OffloadController
        [SP18] Poll network stats in OffloadController to support data warning
      027c80d2
    • Treehugger Robot's avatar
      [SP18.1] add dependency object to OffloadController · 40092b1f
      Treehugger Robot authored
      In order to mock constant in unit test, a dependency object is
      introduced with minimum code change to achieve this.
      
      Test: atest TetheringTests
      Bug: 149467454
      Change-Id: I38628daddcb7be7c74846e78d36dc88f065b97d9
      Merged-In: I38628daddcb7be7c74846e78d36dc88f065b97d9
      (cherry picked from commit 29aee20b)
      40092b1f
    • Junyu Lai's avatar
      [SP18] Poll network stats in OffloadController to support data warning · bf6fa99b
      Junyu Lai authored
      The OEM implemented tether offload does not support
      data warning since the HAL only tells the hardware about data limit
      but not warning. However, to add such interface in HAL needs OEM to
      comply and implement in hardware.
      
      Thus, as a short-term solution, polls network statistics from HAL
      and notify upper layer when it reaches the alert quota set by
      NetworkStatsService.
      
      Note that when CPU is sleeping, the data warning of tethering offload
      will not work since the polling is also suspended.
      
      Test: manual
      Test: atest OffloadControllerTest
      Bug: 149467454
      Change-Id: I2467b64779b74cd5fec73b42fb303584f52cb1cb
      Merged-In: I2467b64779b74cd5fec73b42fb303584f52cb1cb
      (cherry picked from commit 93660e38)
      bf6fa99b
  9. Apr 28, 2020
  10. Apr 27, 2020
    • TreeHugger Robot's avatar
    • Mark Chien's avatar
      Remove Preconditions usage to stop dependecy with non-updatble class · ed50636b
      Mark Chien authored
      Stop depending on Preconditions that is not released on the same cadence
      as the module, and is maintained as part of the framework.
      
      Bug: 148636687
      Test: atest TetheringTests NetworkStackNextTests
      
      Merged-In: Id0dcec44f362f79bc8c046d722635687a7388aa2
      Change-Id: Id0dcec44f362f79bc8c046d722635687a7388aa2
      ed50636b
    • Mark Chien's avatar
      Unbreak tethering for no offload supported devices · 27cdaf67
      Mark Chien authored
      Catch NoSuchElementException to unbreak no offload devices.
      
      To consistent with fetching offload config service, retry fetcheing
      offload control service.
      b/152430668#comment4 assert that the fetch will be retried only
      if the service is installed on the device.
      
      Bug: 155026033
      Test: run TetheringCoverageTests in virtual devices(do not support
      offload)
      
      Merged-In: Ie0a32a9062c722327a27c6de13e3bb8d9588bebb
      Change-Id: Ie0a32a9062c722327a27c6de13e3bb8d9588bebb
      27cdaf67
    • Bill Yi's avatar
      Import translations. DO NOT MERGE · 8a35765b
      Bill Yi authored
      Auto-generated-cl: translation import
      Change-Id: I08496ebb9bad7c95a6e9df43ef57c55a9956c9d8
      8a35765b
  11. Apr 23, 2020
    • Paul Hu's avatar
      [TNU09] Adjust restricted notification · 0439c2d2
      Paul Hu authored
      1. Let restricted notification that can be dismissed.
      2. Only put up restricted notification when any of tethering is
         activating.
      
      Bug: 154214549
      Test: atest TetheringTests
      Change-Id: Ib980aca154036828abdab35e3bb11d42f85ff610
      Merged-In: Ib980aca154036828abdab35e3bb11d42f85ff610
      (cherry picked from commit 2eb66bdb, aosp/1290334)
      0439c2d2
    • Paul Hu's avatar
      Add TetheredClient tests · 86152672
      Paul Hu authored
      Test APIs below:
      getAddresses()
      getMacAddress()
      getTetheringType()
      AddressInfo.getAddress()
      AddressInfo.getHostname()
      AddressInfo.writeToParcel(android.os.Parcel, int)
      
      Bug: 153614365
      Bug: 153613717
      Test: atest TetheringTests
      Change-Id: Ic7cbebe54a38af5b5c4639eb7641a20de6864a26
      Merged-In: Ic7cbebe54a38af5b5c4639eb7641a20de6864a26
      (cherry picked from commit fa882fdb, aosp/1292438)
      86152672
    • Paul Hu's avatar
      Merge "Add TetheringCommonTests" into rvc-dev · 1277763d
      Paul Hu authored
      1277763d
  12. Apr 22, 2020
    • Bill Yi's avatar
      Import translations. DO NOT MERGE · 70521ef0
      Bill Yi authored
      Change-Id: I4b3a8cf8d738bf6f680258f1ebd77d894dc986cd
      Auto-generated-cl: translation import
      70521ef0
    • Paul Hu's avatar
      Add TetheringCommonTests · 1cae88f5
      Paul Hu authored
      Bug: 153614365
      Bug: 153613717
      Test: atest TetheringTests TetheringCoverageTests
      Change-Id: If7c933ec0c72943312cd37bfc66918f10a5504a9
      Merged-In: If7c933ec0c72943312cd37bfc66918f10a5504a9
      (cherry picked from commit f340f6fd, aosp/1290553)
      1cae88f5
    • Mark Chien's avatar
    • Paul Hu's avatar
    • Mark Chien's avatar
      Include NetworkStaticLibTestsLib into tethering coverage test · d0ed7dd8
      Mark Chien authored
      Bug: 148636687
      Test: atest NetworkStaticLibTests
      Test: atest TetheringCoverageTests
      Merged-In: I8cd9dbc9fe163583ff1d016c9262546949105b80
      Change-Id: I8cd9dbc9fe163583ff1d016c9262546949105b80
      d0ed7dd8
    • Paul Hu's avatar
      a7c5d570
    • Paul Hu's avatar
      [TNU08] Remove the "tethering is on" notification · 127ee592
      Paul Hu authored
      All carriers discarded the requirement to put up a standing
      notification when tethering is on. Thus, remove the "tethering
      is on" notification.
      
      Bug: 154438388
      Test: atest TetheringTests
      Change-Id: Ife3915837b6b7b83d3eaaa84b71b6409ff37b71c
      Merged-In: Ife3915837b6b7b83d3eaaa84b71b6409ff37b71c
      (cherry picked from commit 0171c07d05dd2625c6dcfd47977a701ddc2d5d36, aosp/1289107)
      127ee592
    • Treehugger Robot's avatar
      Retry the call that fetches the tethering offload HAL. · 83a814eb
      Treehugger Robot authored
      The CL that moved the initialization of the tethering offload
      config HAL from C++ to Java caused the code not to retry
      fetching the service if it is not ready when tethering is
      started. This is because the C++ version of getService() retries,
      but the Java version only retries if getService(true) is called.
      
      Make the new code retry as well.
      
      b/152430668#comment4 asserts that the fetch will be retried only
      if the service is installed on the device, so the retries should
      be attempted (and thus should not have any startup time impact)
      on devices that do not support tethering offload.
      
      Bug: 152430668
      Test: builds, boots, tethering offload works
      Merged-In: I093f127d90b2aa1b13eb0748378a24726d419472
      Change-Id: I093f127d90b2aa1b13eb0748378a24726d419472
      83a814eb
    • Paul Hu's avatar
      [TNU07] Remove unused resources · ef341599
      Paul Hu authored
      Some strings are not used, so just remove them.
      
      Bug: 154445061
      Test: atest TetheringTests
      Change-Id: Ie67f7f4d0863d58c4c5206e445c1dbdfe3f65f60
      Merged-In: Ie67f7f4d0863d58c4c5206e445c1dbdfe3f65f60
      (cherry picked from commit 59eeedc045aaaa0c7d5b97277d25f64cce254464, aosp/1289100)
      ef341599
Loading