[Merge] lp:~rpadovani/webbrowser-app/updateUrlManagementFunctions into lp:webbrowser-app

Riccardo Padovani riccardo at rpadovani.com
Tue Apr 14 20:54:27 UTC 2015


Riccardo Padovani has proposed merging lp:~rpadovani/webbrowser-app/updateUrlManagementFunctions into lp:webbrowser-app.

Commit message:
Add support for data: uri in the UI, remove length limitation to tdl in website addresses

Requested reviews:
  Ubuntu Phablet Team (phablet-team)
Related bugs:
  Bug #1377953 in webbrowser-app: "data: URIs don't work"
  https://bugs.launchpad.net/webbrowser-app/+bug/1377953

For more details, see:
https://code.launchpad.net/~rpadovani/webbrowser-app/updateUrlManagementFunctions/+merge/256213

Fix bug #1377953 - data: URIs don't work

You can find some tests here: http://www-archive.mozilla.org/quality/networking/testing/datatests.html

They don't work all as expected (as described in the page), but they work as Chromium on vivid, so I think it's acceptable

Fix bug #1441281 - update looksLikeAUrl().

Remove .tdl regex length limitation (min 2, max *)
-- 
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~rpadovani/webbrowser-app/updateUrlManagementFunctions into lp:webbrowser-app.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 799 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20150414/d0947203/attachment.diff>


More information about the Ubuntu-reviews mailing list