• Rayan Kanso's avatar
    Revert "[mojo] Enable default InstalledAppProvider binder for Android" · fbd962fe
    Rayan Kanso authored
    This reverts commit 100226ae.
    
    Reason for revert: Broke the feature on Android. Also, the feature has been disabled on webview in crrev.com/c/2156991
    
    Bug: 1086686
    
    Original change's description:
    > [mojo] Enable default InstalledAppProvider binder for Android
    > 
    > Since only Chrome binds InstalledAppProvider, but a default no-op
    > behaviour is expected for other embedders (tests, WebView, WebLayer),
    > this change removed the 'non-Android' condition for the default
    > InstalledAppProvider binder to avoid renderer termination for the
    > 'no binder found' case.
    > 
    > Bug: 1064644, 1064827
    > Test: No crash when opening reddit.com in webview shell on an emulator
    > Change-Id: Ia9e6d98f0abff94929bf10dcd778166c90907f40
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124872
    > Reviewed-by: Daniel Cheng <dcheng@chromium.org>
    > Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#755083}
    
    TBR=dcheng@chromium.org,oksamyt@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 1064644, 1064827
    Change-Id: Ie2ccdb3724abe7b28dcc461dedebb286c667f595
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2218028Reviewed-by: default avatarRayan Kanso <rayankans@chromium.org>
    Commit-Queue: Rayan Kanso <rayankans@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#772297}
    fbd962fe
browser_interface_binders.cc 46.4 KB