<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I agree with Dave that it seems like it's related to focus due to the screenshot but I also don't see how that would have changed with your patch. I notice that there isn't any waitForFocus calls in that directory so you could try wrapping the goNext call in the test function with one.<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Matthew<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 29, 2015 at 11:01 AM, Dave Townsend <span dir="ltr"><<a href="mailto:dtownsend@mozilla.com" target="_blank">dtownsend@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>When you say we're initializing the add-on manager what do you mean specifically? The add-on manager is always initialized before profile-after-change (<a href="http://mxr.mozilla.org/mozilla-central/source/toolkit/xre/nsXREDirProvider.cpp#838" target="_blank">http://mxr.mozilla.org/mozilla-central/source/toolkit/xre/nsXREDirProvider.cpp#838</a>).<br><br></div>I noticed that in both of the failures the test harness window has focus rather than the browser window. That would explain why the popup notifications aren't showing. I don't know why the focus would have changed though.<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 29, 2015 at 10:36 AM, Benjamin Smedberg <span dir="ltr"><<a href="mailto:bsmedberg@mozilla.com" target="_blank">bsmedberg@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>MattN and Mossop:<br><br>This push is causing timeout failures in browser_popupNotification.js (for non-e10s, disabled in e10s) or browser_addons.js (for e10s):<br><br><a href="https://hg.mozilla.org/integration/fx-team/rev/40520b2ca6b8" target="_blank">https://hg.mozilla.org/integration/fx-team/rev/40520b2ca6b8</a><br><br></div>Failure log:<br><a href="https://treeherder.mozilla.org/logviewer.html#?job_id=2866587&repo=fx-team" target="_blank">https://treeherder.mozilla.org/logviewer.html#?job_id=2866587&repo=fx-team</a><br><br></div>I suspect that this has something to do with the fact that this patch changes the ordering of initializing the addon manager. We are now reliably initializing the addon manager during profile-after-change, instead of at first-use sometime later.<br><br></div>But I can't figure out how this would be affecting the PopupNotifications code. Matt, you're the expert in PopupNotifications and Dave you're my addon manager expert. Can one of you help me figure out how to start debugging this? I can't reproduce it locally, and I expect it may be timing-dependent but I'm really not sure what to do next.<br><br></div>--BDS<span><font color="#888888"><br clear="all"><div><div><div><div><div><div><br>-- <br><div><div dir="ltr"><div><div dir="ltr">Benjamin Smedberg<br>Engineering Manager, Firefox<br><br></div></div></div></div>
</div></div></div></div></div></div></font></span></div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" target="_blank">https://mail.mozilla.org/listinfo/firefox-dev</a><br>
<br></blockquote></div><br></div>