<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 23, 2015 at 10:34 PM, Nicholas Alexander <span dir="ltr"><<a href="mailto:nalexander@mozilla.com" target="_blank">nalexander@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Wed, Sep 23, 2015 at 5:59 PM, Ehsan Akhgari <span dir="ltr"><<a href="mailto:ehsan.akhgari@gmail.com" target="_blank">ehsan.akhgari@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Sigh...<br><br></div>Do you know what's different between the initial request that we make for the page and the one that we make when we request the desktop mode?  Is the only difference the User-Agent header?<br></div></blockquote><div><br></div></span><div>I think this is correct, based on reading the test at<br><br><a href="https://hg.mozilla.org/mozilla-central/rev/c4a1f1188677#l4.57" target="_blank">https://hg.mozilla.org/mozilla-central/rev/c4a1f1188677#l4.57</a><br><br></div><div>and the code at<br></div><div><br><a href="https://dxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#3210" target="_blank">https://dxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#3210</a><br></div></div></div></div></blockquote></div><br></div><div class="gmail_extra">Ah, of course.  I am not surprised that nobody is using Vary: User-Agent <<a href="http://www.rimmkaufman.com/blog/vary-user-agent/30112012/">http://www.rimmkaufman.com/blog/vary-user-agent/30112012/</a>>.<br><br></div><div class="gmail_extra">I will backout that part of the patch, as I now believe not bypassing the cache in that case is incorrect.  Thanks for catching this!<br clear="all"></div><div class="gmail_extra"><br>-- <br><div class="gmail_signature"><div dir="ltr">Ehsan<br></div></div>
</div></div>