[PATCH] D51255: [WWW] Use https instead of http to fix page problems

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 27 07:19:58 PDT 2018


jdoerfert added a comment.

In https://reviews.llvm.org/D51255#1214424, @asl wrote:

> I think we need to preserve the protocol. E.g. if the page is loaded via http we need to load css via http as well.


That sounds reasonable and is (afaik) implied by removing the hostname.
As mentioned before, I like the solution but I just don't know if that might cause problems somewhere else.


https://reviews.llvm.org/D51255





More information about the llvm-commits mailing list