WebView for Android Nougat relies on code that is not yet part of upstream Chromium releases #420

Closed
thestinger opened this Issue Sep 6, 2016 · 6 comments

Comments

Projects
None yet
2 participants
@thestinger
Contributor

thestinger commented Sep 6, 2016

CopperheadOS will need to reuse Google WebView builds rather than our own builds for the time being. This is quite annoying since it means we're stuck with their baseline compiler hardening rather than being able to enable some trapping UBSan sanitizers and using our -fsanitize=local-init sanitizer to zero all uninitialized local variables. Among other things that were previously possible thanks to building from source...

Can keep checking each stable release to figure out when the new code becomes public. Hopefully, they've already upstreamed enough for us to switch with the next stable release...

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Sep 6, 2016

Contributor

Note: standalone Chromium still works fine.

Contributor

thestinger commented Sep 6, 2016

Note: standalone Chromium still works fine.

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Sep 6, 2016

Contributor

Related: #415.

Contributor

thestinger commented Sep 6, 2016

Related: #415.

@rsesek

This comment has been minimized.

Show comment Hide comment
@rsesek

rsesek Sep 12, 2016

Chromium's issue for upstreaming this code is here: https://bugs.chromium.org/p/chromium/issues/detail?id=643660

rsesek commented Sep 12, 2016

Chromium's issue for upstreaming this code is here: https://bugs.chromium.org/p/chromium/issues/detail?id=643660

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Sep 12, 2016

Contributor

@rsesek Any idea about #415? That seems like it might just be a bit of build integration that's missing now. Most of it seems to be there.

Contributor

thestinger commented Sep 12, 2016

@rsesek Any idea about #415? That seems like it might just be a bit of build integration that's missing now. Most of it seems to be there.

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Sep 12, 2016

Contributor

(I'd be really happy if I had to do 2 builds instead of 4 :P)

Contributor

thestinger commented Sep 12, 2016

(I'd be really happy if I had to do 2 builds instead of 4 :P)

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Nov 16, 2016

Contributor

Have now switched to the Beta channel to build this from source again. The new issue is #489.

Contributor

thestinger commented Nov 16, 2016

Have now switched to the Beta channel to build this from source again. The new issue is #489.

@thestinger thestinger closed this Nov 16, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment