-
Notifications
You must be signed in to change notification settings - Fork 2.3k
1022767 use json l10n manifests #23549
1022767 use json l10n manifests #23549
Conversation
Zibi Braniecki (zbraniecki) started tests. Results |
1952d6b
to
a6b3c7c
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
a6b3c7c
to
2a8aa0e
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
2a8aa0e
to
1035a74
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
1035a74
to
501eeae
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
501eeae
to
e226905
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
e226905
to
4574b2b
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
4574b2b
to
cb8bbb6
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
self.official === '1' ? | ||
'official' : 'unofficial', | ||
utils.basename(resURL)); | ||
isBranding = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isBranding
needs to be reset on the next iteration of the loop; otherwise it stays true
for all subsequent files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, you could call this isOfficialBranding
and avoid checking self.official
again in line 171.
cb8bbb6
to
3f7a9e8
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
3f7a9e8
to
bc48c57
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
L10n.onManifestInjected.call(this, node.getAttribute('href')); | ||
break; | ||
case 'localization': | ||
if (!('noFetch' in nodes[i].dataset)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: use node
in lieu of nodes[i]
here.
bc48c57
to
8bbf1ea
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
8bbf1ea
to
e440ff2
Compare
}, | ||
"zh-TW": { | ||
"name": "Smart Collections", | ||
"description": "Smart Collections Manager" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Collections are en-US only in the gaia repo, no additional locales.
804f49d
to
9f8d2eb
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
9f8d2eb
to
893258b
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
893258b
to
c2aecca
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
c2aecca
to
b69640b
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
Zibi Braniecki (zbraniecki) started tests. Results |
73ae306
to
edfd1e3
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
Zibi Braniecki (zbraniecki) started tests. Results |
25b39b5
to
54b1186
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
54b1186
to
70a1444
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
70a1444
to
de5a5e7
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
de5a5e7
to
5015012
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
…adata. r=pike, yuren, stas
5015012
to
02ca679
Compare
Zibi Braniecki (zbraniecki) started tests. Results |
…ests Bug 1022767 - Use manifest.webapp instead of locales.ini for l10n metadata. r=pike, yuren, stas
https://bugzilla.mozilla.org/show_bug.cgi?id=1022767