Merge lp:~osomon/webbrowser-app/drawer-menu-actions-visible into lp:webbrowser-app

[Olivier Tilloy]

Set the 'visible' property of actions instead of 'enabled' to ensure the visibility of the corresponding entries in the drawer menu matches.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:1177
http://jenkins.qa.ubuntu.com/job/webbrowser-app-ci/2227/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/4146
    SUCCESS: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-amd64-ci/981
    SUCCESS: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-armhf-ci/981
        deb: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-armhf-ci/981/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-i386-ci/981
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/3382
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/4143
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/4143/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/23277

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/webbrowser-app-ci/2227/rebuild

review: Approve (continuous-integration)

=== modified file 'src/app/webbrowser/Browser.qml'
--- src/app/webbrowser/Browser.qml	2015-09-03 09:46:29 +0000
+++ src/app/webbrowser/Browser.qml	2015-09-13 21:23:23 +0000
@@ -367,21 +367,21 @@
                     objectName: "share"
                     text: i18n.tr("Share")
                     iconName: "share"
-                    enabled: (formFactor == "mobile") && browser.currentWebview && browser.currentWebview.url.toString()
+                    visible: (formFactor == "mobile") && browser.currentWebview && browser.currentWebview.url.toString()
                     onTriggered: internal.shareLink(browser.currentWebview.url, browser.currentWebview.title)
                 },
                 Action {
                     objectName: "history"
                     text: i18n.tr("History")
                     iconName: "history"
-                    enabled: browser.historyModel
+                    visible: browser.historyModel
                     onTriggered: historyViewLoader.active = true
                 },
                 Action {
                     objectName: "tabs"
                     text: i18n.tr("Open tabs")
                     iconName: "browser-tabs"
-                    enabled: (formFactor != "mobile") && !browser.wide
+                    visible: (formFactor != "mobile") && !browser.wide
                     onTriggered: {
                         recentView.state = "shown"
                         recentToolbar.state = "shown"
@@ -391,14 +391,14 @@
                     objectName: "newtab"
                     text: i18n.tr("New tab")
                     iconName: browser.incognito ? "private-tab-new" : "tab-new"
-                    enabled: (formFactor != "mobile") && !browser.wide
+                    visible: (formFactor != "mobile") && !browser.wide
                     onTriggered: browser.openUrlInNewTab("", true)
                 },
                 Action {
                     objectName: "findinpage"
                     text: i18n.tr("Find in page")
                     iconName: "search"
-                    enabled: !chrome.findInPageMode && !newTabViewLoader.active
+                    visible: !chrome.findInPageMode && !newTabViewLoader.active
                     onTriggered: {
                         chrome.findInPageMode = true
                         chrome.focus = true