822655 - "Show controls " option should have S and C as capital letters in context menu

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Ioana Chiorean]

Attachment: Screenshot "Show controls"

Firefox 20.0a1 (2012-12-18)
Device: Galaxy Nexus
OS: Android 4.1.1
Video : Example 1 from http://www.w3schools.com/html/html5_video.asp (bit.ly/HidC) 

Steps to reproduce:
1. Go to http://www.w3schools.com/html/html5_video.asp (bit.ly/HidC) 
2. Scroll down to Example 1 video - long tap on it
3. Observe the Options from the context Menu

Expected results: 
- "Show controls " should have both S and C as capital letters

Actual result:
- "Show controls" has only S as capital letter - see attached screenshot

Comment 1

[Wesley Johnston (:wesj)]

Attachment: Patch

Not sure if finkle is around. I think this counts as a spelling error and we're ok not bumping the string id. Right?

Comment 2

[:Margaret Leibovic]

Comment on attachment 693428 [details] [diff] [review]
Patch

Did you upload the wrong patch? It looks like this code is already landed, and it doesn't actually fix the string.

Comment 3

[Wesley Johnston (:wesj)]

Attachment: Patch

Its still early ;)

Comment 4

[Wesley Johnston (:wesj)]

Attachment: Patch

Margaret suggested bumping the id to be safe and I think she's right.

Comment 5

[Wesley Johnston (:wesj)]

Pushed with DONTBUILD since this is a fairly trivial change. I'm not sure if thats ok or not, but I think I've been told to do it before.

https://hg.mozilla.org/integration/mozilla-inbound/rev/b645cf8f8796

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/b645cf8f8796

Comment 7

[Ioana Chiorean]

Firefox 20.0a1 (2013-01-02)
Device: Galaxy Nexus 
OS: Android 4.1.1 

"Show controls " has now both S and C as capital letters. Marking bug as Verified Fixed.