Closed Bug 796157 Opened 12 years ago Closed 12 years ago

Review b2g/locales/en-US for string freeze


(Firefox OS Graveyard :: General, defect, P1)



(blocking-basecamp:+, firefox18 fixed, firefox19 fixed, firefox20 fixed)

B2G C2 (20nov-10dec)
blocking-basecamp +
Tracking Status
firefox18 --- fixed
firefox19 --- fixed
firefox20 --- fixed


(Reporter: Pike, Assigned: Pike)



We'll need a string freeze for b2g on mozilla-central in time for uplift to aurora.

I think most of the files there aren't needed, actually, just netError.dtd and make sense to me right now.

Also, the branding files need review, those potentially show up in network error messages generated by gecko, and I'm pretty sure we don't want "B2G" to show up there? Those are currently under b2g/branding/.../locales/en-US
Axel: I assume you are the right assignee? If not, can you find the right person to do this?
Assignee: nobody → l10n
blocking-basecamp: ? → +
This needs a functional review of someone that knows what b2g does with those strings.

I don't think that we can help here from the l10n side, the questions is more about:

If we trigger a netError.dtd message that includes brand.dtd's brandShortName, what should that name be?

Folks from the l10n side won't be able to answer that, I'm afraid.

Also, I don't know why many of the files are actually there, I'd remove all but the two I mentioned in the initial comment, plus the branding.
Assignee: l10n → nobody
Axel: Can you chat with Vivien about what's needed to be done here. I think the combination of you two is the best to figure out what needs to be done and who can do it.
I've talked to stas a bit, he's going to figure out what to do.

Thinking about it, if we want to use the mobile-cloned netError.xhtml, we probably need to use the mobile-derived netError.dtd.

This is unfortunate, as we may want to use non-branded error messages. Stas is gonna hack a bit to see how non-browser web apps may react to errors in remote content loading. If the errors show up there easily, we may not want to use "Firefox" branded UI. Stas is also going to talk to UX about this.
Assignee: nobody → stas
I have this on my list for next week.
Stas, now that we're string-frozen, can this be closed?
Sadly, not. with the patch in bug 796051 we get a better grip on how many files aren't actually used at all.

But there's still bits that I find confusing, like the crashreporter.ini.
Priority: -- → P1
(In reply to Axel Hecht [:Pike] from comment #7)

> But there's still bits that I find confusing, like the crashreporter.ini.

I'm not sure why that file is included. It's definitely not used by the gaia crash reporter UI. I'm not familiar with the exact role of xulrunner in gaia, but it seems odd to me that crash reporter directories were included in there.
I'm actually just updating the patch in the chrome target bug to use strings in b2g, and in the course removing anything but neterror and appstrings.
What's next here?
So far, nothing's blew up on us from bug 796051, thus I'll resolve this one as FIXED.

This is in collab with bug 808326, which nails down the branding questions within the remaining files.
Assignee: stas → l10n
Closed: 12 years ago
Depends on: 796051
Resolution: --- → FIXED
Target Milestone: --- → B2G C2 (20nov-10dec)
You need to log in before you can comment on or make changes to this bug.