Skip to content

GitLab

Switch branch/tag
  1. 05 Dec, 2015 2 commits
  3. 03 Dec, 2015 2 commits
    • Christopher Wiley's avatar
      Merge "libbinder: Handle transaction failures correctly" · 823fa55a
      Christopher Wiley authored
      823fa55a
    • Christopher Wiley's avatar
      libbinder: Handle transaction failures correctly · cff7f175
      Christopher Wiley authored 
      Java code expects status_t != OK to be caught at the JNI level in
android_util_Binder.cpp (see signalExceptionForError).  We were
incorrectly mapping this kind of failure to a special exception type
and writing that exception type to parcels.

Instead, refuse to write EX_TRANSACTION_FAILED to a parcel and return
the status value instead.

While here, remove non-trivial constructors to push authors toward the
more explicit factory methods.  Remove getException() and push authors
toward using the simpler getter methods. Fix minor camelCase issues.

Bug: 25615695
Test: system/tools/aidl integration tests still pass

Change-Id: I7cad3ac8ae8300b5ac0b466606f4934d01e503c5
      cff7f175
  5. 02 Dec, 2015 2 commits
    • Dan Stoza's avatar
      Merge "bug#460672 framebuffer surface can't keep triple buffer if define... · b2e3095a
      Dan Stoza authored 
      Merge "bug#460672 framebuffer surface can't keep triple buffer if define NUM_FRAMEBUFFER_SURFACE_BUFFERS := 3"
      b2e3095a
    • james.zhang's avatar
      bug#460672 framebuffer surface can't keep triple buffer if define... · 5e8eb5ea
      james.zhang authored 
      bug#460672 framebuffer surface can't keep triple buffer if define NUM_FRAMEBUFFER_SURFACE_BUFFERS := 3

[root cause  ] framebuffer surface is in the same process of surface flinger
[changes     ] call IGraphicBufferProducer::allocateBuffers on producer inside the DisplayDevice constructor to force-allocate all three buffers.
[side effects] none
[self test   ]<tshark, enter and exit mainmenu>: ok
[reviewers   ] zhongjun.chen
[change_type ] AOB --> google_original
[tag_product ] common

Change-Id: Ideb75d3aa1ed23b2fa4d263a120ea3ca39d01361
      5e8eb5ea
  7. 30 Nov, 2015 2 commits
  9. 23 Nov, 2015 3 commits
  11. 20 Nov, 2015 1 commit
  13. 19 Nov, 2015 4 commits
  15. 18 Nov, 2015 5 commits
  17. 17 Nov, 2015 2 commits
    • Christopher Wiley's avatar
      libbinder: Add binder::Status type · 09eb7497
      Christopher Wiley authored 
      This object implements equivalent functionality to the Java logic which
serializes and re-throws exceptions from services.

Bug: 25615695
Test: Integration test for generated AIDL code reveals this to work
      correctly.

Change-Id: Ic80c9def0dd232582ea9c49717ec50894af6bfc2
      09eb7497
    • Todd Kennedy's avatar
      Merge "use negative mask" · 4e427f0a
      Todd Kennedy authored
      4e427f0a
  19. 16 Nov, 2015 2 commits
  21. 13 Nov, 2015 4 commits
  23. 12 Nov, 2015 1 commit
  25. 11 Nov, 2015 1 commit
    • Christopher Wiley's avatar
      libbinder: Return UNEXPECTED_NULL when appropriate · 4db672dd
      Christopher Wiley authored 
      Return a meaningful and consistent error code when libbinder encounters
a null reference that it didn't expect.  This allows us to detect these
situations and cause a corresponding NullPointerException to be thrown
on the remote callsite.

While here, consolidate places we read vector<T> from a parcel into a
single templated version.  This dramatically reduces code duplication.

Bug: 25617251
Test: Integration tests asserting that vectors serialize correctly
      passes.  Integration test asserting that we throw
      NullPointerException on seeing a null reference also passes.

Change-Id: Ia59c264884ec89539c1b99cd7bd28ac201b5b7ac
      4db672dd
  27. 09 Nov, 2015 1 commit
  29. 08 Nov, 2015 1 commit
  31. 05 Nov, 2015 1 commit
  33. 04 Nov, 2015 2 commits
  35. 03 Nov, 2015 2 commits
  37. 02 Nov, 2015 2 commits