1607092 - Add new quote from Book of Mozilla for GeckoView

Status: RESOLVED FIXED

(Firefox :: General, enhancement)

Description

[Jonathan Almeida [:jonalmeida]]

Originally filed in fenix#1398 it seems like GeckoView has it's own about:mozilla page from cpeterson's comment.

Although it would be better if the quote included all our GV browsers and not just one (Fenix, FxR, Focus/R-B?).

Seburo had interest in providing a quote as well.

Comment 1

[Seburo]

Attachment: The Book of Mozilla, 6 27.xhtml

Comment 2

[Seburo]

Please find attached my proposed wording.

It has been two years since the writers of the Book convened to publish another chapter so as to bring forth the wisdom of the Beast unto the masses. In that time, much study and meditation has gone into understanding the messages that the Beast passes unto the writers of the book, leading to the attached file being drafted. As tradition dictates, it maintains the form, length and style of those that have written chapters previously.

This text brings forth the Beasts teachings in respect of GeckoView in its many forms. Whilst the intention to land this chapter in Desktop (before the acolytes assist in its transition to the mobile platforms) may be seen as a bit odd, this is not without historical precedent and is in line with the Beasts teachings in respect of cross platform unity.

Feedback is welcome, for the Beast teaches us that teamwork and collaboration is vital to success, but there are one or two elements that should be seen as if they have come from the Beast itself. In contemplating this text it would be appreciated if this could be landed soon so as to spread the word of the Beast to as many as possible as soon as is practical.

Thank you.

Comment 3

[Seburo]

Attachment: The Book of Mozilla, 6 27.xhtml

Comment 4

[Seburo]

This second file corrects a small typo.

Comment 5

[Seburo]

Moving this bug from Fenix to Firefox, as the aim is to land this in desktop before it then lands in the GeckoView products, thus spreading these new teachings among all of the followers of the Beast.

Comment 6

[Seburo]

I believe that the file that needs to be altered is at toolkit/locales/en-US/chrome/global/mozilla.dtd. I have tried to create an updated .dtd file, but am not having much luck.

Comment 7

[Seburo]

Comment on attachment 9118961 [details]
The Book of Mozilla, 6 27.xhtml

Text to be replaced to remove the repeated use of "forth" in the second sentence.

Comment 8

[Seburo]

Attachment: The Book of Mozilla, 6 27.xhtml

Corrected text attached.

Comment 9

[Jonathan Almeida [:jonalmeida]]

Attachment: Bug 1607092 - Add new quote from Book of Mozilla for GeckoView r=Pike,agi

Comment 10

[Jonathan Almeida [:jonalmeida]]

Ehsan, I see that you helped submit the last update, would you be interested in reviewing this as well?

Comment 11

[Jonathan Almeida [:jonalmeida]]

flod, could you take another look at the patch? I updated the page to use Fluent, but I may have made mistakes since I'm not quite familiar with it yet.

Comment 12

[Francesco Lodolo [:flod]]

No need to flag on Bugzilla, Phabricator is enough. I'll take a look at the Fluent file, but you'll need an actual dev to check the rest of the code.

Comment 13

[Jonathan Almeida [:jonalmeida]]

It's a bit unclear to me how to make this test pass: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b2dfd2ae3019347e35839ab4c4362dcb073346b3&selectedTaskRun=cHuI2zbFROm043cVhCS5Ew.0

TEST-UNEXPECTED-FAIL | browser/components/sessionstore/test/browser_capabilities.js | Found a tab after previous test timed out: about:mozilla -

Aside from this, we just need to get a final approval on the patch.

We're probably not going to make it for the 79 release, but we can try to uplift this so that Fenix can pick it up in the next GV beta upgrade.

Comment 14

[Jonathan Almeida [:jonalmeida]]

Fixed the test failures, so the patch should be fine to land: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6d88e67099742d3234e83a01d95893493d76913a

A final +1 from comms would be ideal.

Comment 15

[Mike Hoye [:mhoye]]

(it's not an easter egg if you need to ask permission, also we never spoke, and I was never here.)

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc1e5e4d5ed8
Add new quote from Book of Mozilla for GeckoView r=geckoview-reviewers,Pike,agi

Comment 17

[Jonathan Almeida [:jonalmeida]]

removing NI and disappears in a cloud of smoke

Comment 18

[Narcis Beleuzu [:NarcisB]]

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

Comment 19

[Jonathan Almeida [:jonalmeida]]

Ignore this comment, made an error in the uplift request.

Comment 20

[Jonathan Almeida [:jonalmeida]]

Comment on attachment 9158395 [details]
Bug 1607092 - Add new quote from Book of Mozilla for GeckoView r=Pike,agi

Beta/Release Uplift Approval Request

• User impact if declined: None, this is an easter egg for the Fenix release that will ship with the beta version.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: Open about:mozilla, verify a new quote reference "6:27" is seen.
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Changes to tests that rely on loading the about:mozilla page may be flaky.
• String changes made/needed: None, GeckoView does not inherit the translation files from desktop.

Comment 21

[Francesco Lodolo [:flod]]

(In reply to Jonathan Almeida [:jonalmeida] from comment #20)

Comment on attachment 9158395 [details]
Bug 1607092 - Add new quote from Book of Mozilla for GeckoView r=Pike,agi

Beta/Release Uplift Approval Request

Merge day is on Jul 27, so this patch is already going to Beta in 48 hours.

String changes made/needed: None, GeckoView does not inherit the translation files from desktop.

This is not correct. This patch has new strings that impact localization, they're just not localized in GeckoView (they're in Gecko, currently in Nightly). This should ride the trains with 80.

Comment 22

[Pascal Chevrel:pascalc]

Comment on attachment 9158395 [details]
Bug 1607092 - Add new quote from Book of Mozilla for GeckoView r=Pike,agi

Today is merge day so the uplift is is not necessary to have this patch in 80 beta, I don't think this should be uplifted before the merge.