[FlyWeb] Add back l10n support for FlyWeb system addon

NEW
Unassigned

Status

()

Core
DOM: Flyweb
11 months ago
5 months ago

People

(Reporter: djvj, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(firefox52-, firefox-esr45-, fennec+, thunderbird_esr45?)

Details

(Reporter)

Description

11 months ago
We had to remove l10n usage from the FlyWeb system addon because it was causing oranges on OSX 10.10 debug M-e10s(1) and M-e10s(2) tests.  It did not seem to be due to issues in the code itself, and suggested issues in the underlying platform.

For reference, here is the delta-patch (and try run) that adds l10n files (but doesn't reference them in any way) on top of the current system addon, and replicates the oranges:

https://hg.mozilla.org/try/rev/33254b9d36ca326c2af3fa6fd8b639f3db90ca18
https://treeherder.mozilla.org/#/jobs?repo=try&revision=33254b9d36ca

We should figure out what's going on and add a proper l10n impl for the addon.
Firefox::Translation is for the Automatic Translation feature. I agree there's no good component for this bug (should Flyweb have its own?), so moving to Core::Networking as most Flyweb bugs are there.
Component: Translation → Networking
Product: Firefox → Core
Version: unspecified → Trunk
(In reply to :Felipe Gomes (needinfo me!) from comment #1)
> so moving to Core::Networking as most Flyweb bugs are there.

But they shouldn't be in Core::Networking unless necko subtree is somehow involved.
Component: Networking → Untriaged

Comment 3

9 months ago
Michal, can you please suggest a proper component for this bug. I also search in bugzilla and I saw that bugs with [FlyWeb] are in Core: Networking.
Flags: needinfo?(michal.novotny)
(In reply to ovidiu boca[:Ovidiu] from comment #3)
> Michal, can you please suggest a proper component for this bug. I also

Maybe localization? But I really don't know.

> search in bugzilla and I saw that bugs with [FlyWeb] are in Core: Networking.

This is a common mistake. The code lives in DOM and IMO a new component "DOM: Flyweb" should be created.
Flags: needinfo?(michal.novotny)

Comment 5

9 months ago
Hi Pike, Ryan VanderMeulen suggest me to ping you about this issue. Can you help us finding the right place for this issue?

Thanks
Flags: needinfo?(pike-bugzilla)
Flags: needinfo?(pike-bugzilla) → needinfo?(l10n)
(In reply to ovidiu boca[:Ovidiu] from comment #5)
> Hi Pike, Ryan VanderMeulen suggest me to ping you about this issue. Can you
> help us finding the right place for this issue?
> 
> Thanks

I'll defer the answer to that to flod.
Flags: needinfo?(l10n) → needinfo?(francesco.lodolo)
I believe this bug doesn't belong to the Mozilla Localization component as it stands, because it's not going to be fixed on the localization side of things. It belongs to the component where other FlyWeb bugs belong, and that still seems unclear based on previous comments (its own component sounds like a good idea to me).

I also went back to bug 1272107 to check the discussion, and it's still unclear what are the expectations: is the development of FlyWeb going to follow the trains (i.e. aurora and beta are string frozen) or need to go faster than that? Is it going to need more than the two current strings?
Flags: needinfo?(francesco.lodolo)
One more note: if this needs to live in m-c, in order to get managed by compare-locales the folder needs to be listed in browser/locales/l10n.ini (see for example Pocket)
http://hg.mozilla.org/mozilla-central/file/default/browser/locales/l10n.ini#l9
Kannan, this bug is and old stuck in untriaged. Could you please move it to the correct location so that it stops alerting and we know who's responsible for it? If you need help making a Core: Flyweb component, please reach out to Emma or myself.
Flags: needinfo?(kvijayan)
(Reporter)

Comment 10

9 months ago
(In reply to Benjamin Smedberg [:bsmedberg] from comment #9)
> Kannan, this bug is and old stuck in untriaged. Could you please move it to
> the correct location so that it stops alerting and we know who's responsible
> for it? If you need help making a Core: Flyweb component, please reach out
> to Emma or myself.

Thanks for the suggestion, I think it's appropriate to make a FlyWeb component for this.  How do we get that started?
Flags: needinfo?(kvijayan) → needinfo?(benjamin)

Comment 11

9 months ago
Filed bug 1294191, requesting Toolkit->Flyweb.
Component: Untriaged → Flyweb
Flags: needinfo?(benjamin)
Product: Core → Toolkit
(Assignee)

Updated

6 months ago
Product: Toolkit → Core

Comment 12

6 months ago
[Tracking Requested - why for this release]:

Release Note Request (optional, but appreciated)
[Why is this notable]:
[Affects Firefox for Android]:
[Suggested wording]:
[Links (documentation, blog post, etc)]:
tracking-fennec: --- → ?
blocking-fx: --- → ?
status-firefox-esr45: --- → ?
tracking-firefox-esr45: --- → ?
tracking-thunderbird_esr45: --- → ?
Flags: needinfo?(kvijayan)

Comment 13

6 months ago
[Tracking Requested - why for this release]:
status-firefox50: --- → ?
status-firefox51: --- → ?
tracking-firefox52: --- → ?
Flags: needinfo?(kvijayan)
tracking-fennec: ? → +
not tracking for 52, no justification given in comment 12.  and resetting status flags.
blocking-fx: ? → ---
status-firefox50: ? → ---
status-firefox51: ? → ---
status-firefox-esr45: ? → ---
tracking-firefox52: ? → -
tracking-firefox-esr45: ? → -
You need to log in before you can comment on or make changes to this bug.