450444 - [ru] Add Mail.ru mailto: protocol handler

Status: RESOLVED FIXED

(Mozilla Localizations :: ru / Russian, defect)

Description

[Alexander L. Slovesnik]

Attachment: Add Mail.ru mailto: protocol handler

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)

Comment 1

[Axel Hecht [:Pike]]

We need their permission in Mic's files first.

Comment 2

[Mic Berman]

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

Comment 3

[Alexander Bykov]

It's ok to use mailto link 
http://win.mail.ru/cgi-bin/sentmsg?mailto=%
in firefox

Alexander Bykov,
Mail.Ru Team

Comment 4

[Axel Hecht [:Pike]]

Hi Alexander, thanks.

Does that url pass the testcases on http://people.mozilla.org/~ctalbert/test-protocol-links.html ?

Comment 5

[Alexander Bykov]

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)

Comment 6

[Alexander Bykov]

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 ?

Comment 7

[Axel Hecht [:Pike]]

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

Say, mailto:[e-mail]?cc=[e-mail]&subject=Фото should work.

Re Test 5, what's the outcome of the given url?

Comment 8

[Alexander L. Slovesnik]

Attachment: Add Mail.ru mailto: protocol handler v.2

I forgot to bump gecko.handlerService.defaultHandlersVersion, fixing it.

Comment 9

[Alexander Bykov]

to Axel Hecht:
Test 8 - it's OK
Test 5 - CC field value will be: "[e-mail]?body=foo" because we spliting params by '&'

Comment 10

[Axel Hecht [:Pike]]

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.

Comment 11

[Axel Hecht [:Pike]]

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

Comment 12

[Alexander L. Slovesnik]

Pushed to l10n-central:
http://hg.mozilla.org/l10n-central/index.cgi/ru/rev/c6ced56bc047

Comment 13

[Alexander L. Slovesnik]

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