<div dir="ltr"><div><div><div><div><div><div><div><div><div><div>Things that would help me help with this endeavour:<br><br></div>1: A bug to file patches against<br></div>2: A method for detecting if our fix actually fixes the problem<br><br></div>I presume a skeleton of what we're looking for is:<br></div>1) Use DXR/ls -r/whatever to find the test file in the tree<br></div>2) On the line number(s) mentioned, replace the existing use of a CPOW with something better. This may involve writing things in terms of ContentTask (see documentation here[X]), or by simply finding a non-CPOW-using alternative (like using browser.webNavigation.sessionHistory instead of browser.sessionHistory)<br></div>3) Run the test to make sure it still passes using ./mach test path/to/test.js<br></div><div>4) Write an informative commit message linking back to bug XXXX<br></div>5) Based on what kind of test it is, send it to try to make sure it isn't broken on other platforms<br></div><div>6) <INSERT VALIDATION OF CPOW REMOVAL STEP HERE><br></div>7) Get the patch reviewed on bug XXXX<br><br></div>Is this correct?<br><br></div>:chutten<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 19, 2016 at 5:00 AM, Gabriele Svelto <span dir="ltr"><<a href="mailto:gsvelto@mozilla.com" target="_blank">gsvelto@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Hi Blake,<br>
<span class=""><br>
On 19/10/2016 00:28, Blake Kaplan wrote:<br>
> I've been seeing a pattern of "unsafe" CPOWs causing our browser-chrome<br>
> mochitests to go intermittently orange. Generally, it seems that a test<br>
> randomly turns orange, gets starred and ignored until RyanVM or another one<br>
> of our sheriffs gets tired of seeing it and needinfos somebody in the hopes<br>
> of fixing it.<br>
<br>
</span>I remember we did a big push in the War on Orange maybe three (or four?)<br>
years ago. We could do it again; calling out to everybody to take<br>
intermittent tests in the modules they're familiar with and start fixing<br>
them. Personally I'd happily dedicate a chunk of my time to doing it for<br>
a couple of weeks; having to re-trigger stuff every single time I make a<br>
try run drives me nuts.<br>
<span class="HOEnZb"><font color="#888888"><br>
 Gabriele<br>
<br>
</font></span><br>______________________________<wbr>_________________<br>
dev-platform mailing list<br>
<a href="mailto:dev-platform@lists.mozilla.org">dev-platform@lists.mozilla.org</a><br>
<a href="https://lists.mozilla.org/listinfo/dev-platform" rel="noreferrer" target="_blank">https://lists.mozilla.org/<wbr>listinfo/dev-platform</a><br>
<br></blockquote></div><br></div>