Bug 52369 – Incorrect href=”Tel” fields are shows as active links.

• Status: RESOLVED
• Resolution: INVALID
• Component: DOM

[Prasad]

Incorrect telephone number href's are handled differently in cWRT / widget and in S60 browser.

<a href="tel:[phone]">Dial Nokia PBX</a><br>
<a href="tel:">Invalid tel: 1</a>
<a href="tel:+1234567890123456789012345">invalid tel 2</a>link -->
<a href="tel:[phone].6789.01234">Tel with Punctuation</a>

In cWRT both invalid phone numbers (2nd and 3rd ones) are shown as active links where as in S60 browser shows those two phone numbers as static text.

Steps for testing (cWRT):

* install attached WGT widget (Signed_tel.wgt)
* start "Tel Widget" from application shell

2nd and 3rd "phone numbers" are shown as active links.


Steps for testing in S60 browser:

* copy attached tel.html into c:
* open S60 browser
* open URL: file:///c:/data/tel.html

2nd and 3rd "phone numbers" are shown as normal text.


Looks like QtWebkit doesn't handle this and looking into the S60 browser source codes shows that there are some "3rd party" modified code that is done for
Symbian platform to allow the invalid telephone tags to be disabled.

http://trac.webkit.org/browser/trunk/WebCore/html/HTMLAnchorElement.cpp

In S60 browser code following method is used for the filtering: bool HTMLAnchorElement::checkTelWtaiNumber(const String& number, bool aCti)


Note:
-----

Retested using below mentioned Builds and the error is REPRODUCIBLE:

Symbian: Latest Release

QtTestBrowser: Webkit 2.1

Linux: Ubuntu 10.04 Lucid Lynx, Qt v.4.7.0, Webkit -2.1 (Trunk:-73634)

Safari: Version: 5.0.3 (7533.19.4)

[Prasad]

Nokia Internal Bugzilla ID: 
http://bugs.nokia-boston.com/bugzilla/show_bug.cgi?id=3951

[Alexey Proskuryakov]

I think that this is correct behavior. There is nothing invalid about these URLs - and the engine can't really know what result opening a link gives. As an example, no browser checks if a domain exists when styling an http link like <a href="http://does.no.exist/index.html">.

Please don't prefix cross-platform bugs with [Qt]. This significantly limits the audience, as many people don't read platform specific bugs and patches.