• Brian Sheedy's avatar
    Revert "android: Reintroduce and rework ModernLinker." · 5e961649
    Brian Sheedy authored
    This reverts commit ac8fe0e9.
    
    Reason for revert: crbug.com/986879
    
    Original change's description:
    > android: Reintroduce and rework ModernLinker.
    > 
    > This is partially a revert of two commits:
    > - 53a10113: "Delete unused file:
    >    base/android/linker/android_dlext.h"
    > - fdb31883: "Use the LegacyLinker instead of the
    >    ModernLinker"
    > 
    > ModernLinker has been reworked, especially the native side. Notable changes:
    > - The library is only loaded once, not twice in the browser process
    > - Relocations are always shared between the browser process and the other ones.
    > 
    > This is enabled only on N+ for Chrome.apk, neither for Monochrome nor for TriChrome.
    > 
    >              on Q, further commit will trim it as well.
    > 
    > Binary-Size: Increase due to bringing back ModernLinker. Required to save memory
    > Bug: 979638
    > Change-Id: Ia76773b58a71854a36ea49ab82daef1af717b606
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1683245
    > Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
    > Reviewed-by: Andrew Grieve <agrieve@chromium.org>
    > Reviewed-by: Egor Pasko <pasko@chromium.org>
    > Commit-Queue: Benoit L <lizeb@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#679535}
    
    TBR=pasko@chromium.org,yfriedman@chromium.org,agrieve@chromium.org,lizeb@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 979638
    Change-Id: I431aa2011c4a0c4b23b725099e414a52443778f0
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715745Reviewed-by: default avatarBrian Sheedy <bsheedy@chromium.org>
    Reviewed-by: default avatarEgor Pasko <pasko@chromium.org>
    Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#680141}
    5e961649
linker_jni.h 6.94 KB