[Merge] lp:~uriboni/webbrowser-app/bookmark-folders-from-contextual-menu into lp:webbrowser-app
Olivier Tilloy
olivier.tilloy at canonical.com
Wed Sep 2 10:28:52 UTC 2015
test_bookmark_options_from_contextual_menu can now be updated and unskipped, since the context menu implementation has custom delegates (and the emulator has a get_context_menu() method, which returns an object that has a click_action() method).
The _invoke_contextual_menu_on_item() could be made an emulator method, since it duplicates code in test_contextmenu.py (have a look at the open_context_menu() method there, which takes into account the height of the chrome to determine where to invoke the context menu).
In UrlUtils.js, the naming of the 'authority' variable in the new removeScheme function is confusing, since it’s not actually extracting the authority, only removing the scheme. Call it 'remainder', or 'tail', or something more accurate please.
--
https://code.launchpad.net/~uriboni/webbrowser-app/bookmark-folders-from-contextual-menu/+merge/267934
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list