Bug 1262006 - Swap narrate start/stop button tooltip states. r=Gijs

MozReview-Commit-ID: 4cRneVhQclY

--HG--
extra : rebase_source : 7f680fc81804b89166c6845b28c59056ee3a04c5

toolkit/components/narrate/NarrateControls.jsm

@@ -190,7 +190,7 @@ NarrateControls.prototype = {
     let startStopButton = this._doc.getElementById("narrate-start-stop");
     startStopButton.classList.toggle("speaking", speaking);
     startStopButton.title =
-      gStrings.GetStringFromName(speaking ? "start" : "stop");
+      gStrings.GetStringFromName(speaking ? "stop" : "start");
 
     this._doc.getElementById("narrate-skip-previous").disabled = !speaking;
     this._doc.getElementById("narrate-skip-next").disabled = !speaking;

toolkit/components/narrate/test/NarrateTestUtils.jsm

@@ -47,6 +47,8 @@ this.NarrateTestUtils = {
     ok($(this.FORWARD).disabled, "forward button is disabled");
     ok(!!$(this.START), "start button is showing");
     ok(!$(this.STOP), "stop button is hidden");
+    // This checks for a localized label. Not the best...
+    ok($(this.START).title == "Start", "Button tooltip is correct");
   },
 
   isStartedState: function(window, ok) {
@@ -55,6 +57,8 @@ this.NarrateTestUtils = {
     ok(!$(this.FORWARD).disabled, "forward button is enabled");
     ok(!$(this.START), "start button is hidden");
     ok(!!$(this.STOP), "stop button is showing");
+    // This checks for a localized label. Not the best...
+    ok($(this.STOP).title == "Stop", "Button tooltip is correct");
   },
 
   selectVoice: function(window, voiceUri) {

toolkit/components/narrate/test/browser.ini

@@ -2,6 +2,7 @@
 support-files =
   head.js
   NarrateTestUtils.jsm
+  !/browser/base/content/test/general/readerModeArticle.html
 
 [browser_narrate.js]
 [browser_narrate_disable.js]