Bug 1565010 - Fix intermittent on browser_webconsole_warn_about_replaced_api.js .r=Honza.

marco-c · marco-c · commit cc3c3c47d89f · 2020-04-25T19:11:47.000Z
Log the console replaced message only after the current batch of messages are consumed. We also enable the timestamp in the test, so in case of failure we can diagnose what's going on better. Differential Revision: https://phabricator.services.mozilla.com/D72121 UltraBlame original commit: d321839de1603d8bf02752ebaa40434015149320
diff --git a/devtools/client/webconsole/test/browser/browser_webconsole_warn_about_replaced_api.js b/devtools/client/webconsole/test/browser/browser_webconsole_warn_about_replaced_api.js
@@ -75,15 +75,21 @@ function
 )
 {
 await
-SpecialPowers
+pushPref
+(
+"
+devtools
 .
-pushPrefEnv
+webconsole
+.
+timestampMessages
+"
+true
+)
+;
+await
+pushPref
 (
-{
-set
-:
-[
-[
 "
 devtools
 .
@@ -92,9 +98,6 @@ webconsole
 persistlog
 "
 true
-]
-]
-}
 )
 ;
 let
diff --git a/devtools/client/webconsole/webconsole-ui.js b/devtools/client/webconsole/webconsole-ui.js
@@ -2249,6 +2249,15 @@ handleTabNavigated
 packet
 )
 {
+await
+this
+.
+wrapper
+.
+waitAsyncDispatches
+(
+)
+;
 if
 (
 !
@@ -2264,15 +2273,6 @@ logWarningAboutReplacedAPI
 )
 ;
 }
-await
-this
-.
-wrapper
-.
-waitAsyncDispatches
-(
-)
-;
 this
 .
 emit

Original file line number	Diff line number	Diff line change
`@@ -75,15 +75,21 @@ function`
`75`	`75`	`)`
`76`	`76`	`{`
`77`	`77`	`await`
`78`		`-SpecialPowers`
	`78`	`+pushPref`
	`79`	`+(`
	`80`	`+"`
	`81`	`+devtools`
`79`	`82`	`.`
`80`		`-pushPrefEnv`
	`83`	`+webconsole`
	`84`	`+.`
	`85`	`+timestampMessages`
	`86`	`+"`
	`87`	`+true`
	`88`	`+)`
	`89`	`+;`
	`90`	`+await`
	`91`	`+pushPref`
`81`	`92`	`(`
`82`		`-{`
`83`		`-set`
`84`		`-:`
`85`		`-[`
`86`		`-[`
`87`	`93`	`"`
`88`	`94`	`devtools`
`89`	`95`	`.`
`@@ -92,9 +98,6 @@ webconsole`
`92`	`98`	`persistlog`
`93`	`99`	`"`
`94`	`100`	`true`
`95`		`-]`
`96`		`-]`
`97`		`-}`
`98`	`101`	`)`
`99`	`102`	`;`
`100`	`103`	`let`
Original file line number	Diff line number	Diff line change
`@@ -2249,6 +2249,15 @@ handleTabNavigated`
`2249`	`2249`	`packet`
`2250`	`2250`	`)`
`2251`	`2251`	`{`
	`2252`	`+await`
	`2253`	`+this`
	`2254`	`+.`
	`2255`	`+wrapper`
	`2256`	`+.`
	`2257`	`+waitAsyncDispatches`
	`2258`	`+(`
	`2259`	`+)`
	`2260`	`+;`
`2252`	`2261`	`if`
`2253`	`2262`	`(`
`2254`	`2263`	`!`
`@@ -2264,15 +2273,6 @@ logWarningAboutReplacedAPI`
`2264`	`2273`	`)`
`2265`	`2274`	`;`
`2266`	`2275`	`}`
`2267`		`-await`
`2268`		`-this`
`2269`		`-.`
`2270`		`-wrapper`
`2271`		`-.`
`2272`		`-waitAsyncDispatches`
`2273`		`-(`
`2274`		`-)`
`2275`		`-;`
`2276`	`2276`	`this`
`2277`	`2277`	`.`
`2278`	`2278`	`emit`