Closed Bug 450444 Opened 13 years ago Closed 13 years ago

[ru] Add Mail.ru mailto: protocol handler

Categories

(Mozilla Localizations :: ru / Russian, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: unghost, Assigned: unghost)

Details

(Keywords: fixed1.9.0.2)

Attachments

(1 file, 1 obsolete file)

Mail.ru has informed me that they has implemented mailto: protocol handler for Firefox users and I'd love to include it in Russian build of Firefox.
Mail.ru is the largest free e-mail service in Russia (see http://en.wikipedia.org/wiki/Mail.ru for more info) and it would be large benefit for Russian users to have Mail.ru protocol web-handler in Russian build of Firefox.
Patch is attached (regarding http vs. https issue - Mail.ru has informed me, that they are not ready to provide https web-mail right now)
Attachment #333586 - Flags: review?(l10n)
We need their permission in Mic's files first.
Whiteboard: needs-mic
alexander, can you get the mail.ru guys to email me asap with permission or comment in this bug.
if you have tested the feature and it works we can add it to the current list of 3 mailto for ru: rambler, yandex and google
It's ok to use mailto link 
http://win.mail.ru/cgi-bin/sentmsg?mailto=%
in firefox

Alexander Bykov,
Mail.Ru Team
Hi Alexander, thanks.

Does that url pass the testcases on http://people.mozilla.org/~ctalbert/test-protocol-links.html ?
Test  5 failed (url works)
Test  6 failed
Test  7 failed
Test  8 failed (unfixable due to lack of utf-8 support)
Test 10 failed

thanks for link
we will fix it soon, except utf-8 support (Test 8)
all fixed, except:
Test 5 - we trying to parse incorrect url (common practice)
Test 8 - we support only windows-1251 charset on the web so utf-8 chars not in windows-1251 converted to '?'

its acceptable ?
Test 8 should work as long as the UTF-8 content has an equivalent in windows-1251.

Say, mailto:addr1@foo.com?cc=addr2@foo.com&subject=Фото should work.

Re Test 5, what's the outcome of the given url?
I forgot to bump gecko.handlerService.defaultHandlersVersion, fixing it.
Attachment #333586 - Attachment is obsolete: true
Attachment #333787 - Flags: review?(l10n)
Attachment #333586 - Flags: review?(l10n)
to Axel Hecht:
Test 8 - it's OK
Test 5 - CC field value will be: "addr2@foo.com?body=foo" because we spliting params by '&'
Comment on attachment 333787 [details] [diff] [review]
Add Mail.ru mailto: protocol handler v.2

r=me, code is OK, tests are OK, permission is in comment 3.

There are obviously more advanced ways for error recovery in URLs as the example shows, but that's not a real issue.
Attachment #333787 - Flags: review?(l10n) → review+
Comment on attachment 333787 [details] [diff] [review]
Add Mail.ru mailto: protocol handler v.2

Please land this on hg, and on cvs trunk with my 1.9.0.2 approval.

Please use check-in comments referencing this bug and my review/approval, and use the fixed1.9.0.2 and verified1.9.0.2 keywords to track landing and testing on fx 3.0.x
Attachment #333787 - Flags: approval1.9.0.2+
Pushed to l10n-central:
http://hg.mozilla.org/l10n-central/index.cgi/ru/rev/c6ced56bc047
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: needs-mic
Fixed on CVS trunk:
Checking in chrome/browser-region/region.properties;
/l10n/l10n/ru/browser/chrome/browser-region/region.properties,v  <--  region.properties
new revision: 1.23; previous revision: 1.22
done
Keywords: fixed1.9.0.2
You need to log in before you can comment on or make changes to this bug.