Merge lp:~fboucault/camera-app/resolution_options into lp:camera-app/staging

[Florian Boucault]

New option for the user to choose between the maximum resolution the sensor allows or the resolution that fits the screen.

[Florian Boucault (fboucault)]

Depends on https://code.launchpad.net/~fboucault/qtubuntu-camera/resolution_fixes/+merge/274404

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:586
http://jenkins.qa.ubuntu.com/job/camera-app-ci/435/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/131
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/131
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/131/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/131
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/276
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-mako/930
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/757
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/24192

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:587
http://jenkins.qa.ubuntu.com/job/camera-app-ci/436/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/132
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/132
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/132/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/132
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/277
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-mako/931
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/758
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/24234

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:588
http://jenkins.qa.ubuntu.com/job/camera-app-ci/437/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/133
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/133
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/133/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/133
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/278
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-mako/932
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/759
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/24248

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:589
http://jenkins.qa.ubuntu.com/job/camera-app-ci/448/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/144/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/144/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/144/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/295
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-mako/952
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/780
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/24901

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:592
http://jenkins.qa.ubuntu.com/job/camera-app-ci/457/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/153
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/153
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/153/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/153
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/311
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-touch/971
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/812
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25247

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

review: Needs Fixing (continuous-integration)

[Bill Filler (bfiller)]

Framework version needs to be 15.04.3 as the required qtubuntu-camera change is not in the current 15.04.2 version that is in ota8

review: Needs Fixing

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:597
http://jenkins.qa.ubuntu.com/job/camera-app-ci/463/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/159
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/159
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/159/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/159
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/317
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-touch/978
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/819
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25291

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:600
http://jenkins.qa.ubuntu.com/job/camera-app-ci/465/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/161
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/161
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/161/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/161
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/319
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-touch/980
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/821
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25306

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

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:601
http://jenkins.qa.ubuntu.com/job/camera-app-ci/466/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/162
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/162
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/162/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/162
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/320
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-touch/981
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/822
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25321

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

review: Needs Fixing (continuous-integration)

[Florian Boucault (fboucault)]

https://myapps.developer.ubuntu.com/dev/api/click-framework/ does not have 15.04.3 yet

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:602
http://jenkins.qa.ubuntu.com/job/camera-app-ci/475/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/171
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/171
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/171/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/171
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/334
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-touch/995
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/840
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25499

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

review: Needs Fixing (continuous-integration)

=== modified file 'BottomEdgeIndicators.qml'
--- BottomEdgeIndicators.qml	2015-10-22 12:21:14 +0000
+++ BottomEdgeIndicators.qml	2015-11-25 17:47:21 +0000
@@ -84,7 +84,7 @@
                     id: indicatorIcon
                     anchors.fill: parent
                     color: "white"
-                    name: modelData && modelData.isToggle ? modelData.icon : modelData.get(model.selectedIndex).icon
+                    name: modelData && modelData.isToggle ? modelData.icon : (modelData.get(model.selectedIndex) ? modelData.get(model.selectedIndex).icon : "")
                     source: name ? "image://theme/%1".arg(name) : (modelData.iconSource || "")
                     visible: source != ""
                 }

=== modified file 'CameraApp/advancedcamerasettings.cpp'
--- CameraApp/advancedcamerasettings.cpp	2015-10-27 09:17:27 +0000
+++ CameraApp/advancedcamerasettings.cpp	2015-11-25 17:47:21 +0000
@@ -26,6 +26,10 @@
 #include <QtMultimedia/QVideoDeviceSelectorControl>
 #include <QtMultimedia/QCameraFlashControl>
 #include <QtMultimedia/QCameraExposureControl>
+#include <QGuiApplication>
+#include <QScreen>
+
+#include <cmath>
 
 // Definition of this enum value is duplicated in qtubuntu-camera
 static const QCameraExposure::ExposureMode ExposureHdr = static_cast<QCameraExposure::ExposureMode>(QCameraExposure::ExposureModeVendor + 1);
@@ -222,6 +226,12 @@
         QObject::connect(m_cameraControl,
                          SIGNAL(captureModeChanged(QCamera::CaptureModes)),
                          this, SIGNAL(resolutionChanged()));
+        QObject::connect(m_cameraControl,
+                         SIGNAL(captureModeChanged(QCamera::CaptureModes)),
+                         this, SIGNAL(maximumResolutionChanged()));
+        QObject::connect(m_cameraControl,
+                         SIGNAL(captureModeChanged(QCamera::CaptureModes)),
+                         this, SIGNAL(fittingResolutionChanged()));
     }
 
     m_cameraFlashControl = flashControlFromCamera(m_camera);
@@ -237,6 +247,8 @@
     m_videoEncoderControl = videoEncoderControlFromCamera(m_camera);
 
     Q_EMIT resolutionChanged();
+    Q_EMIT maximumResolutionChanged();
+    Q_EMIT fittingResolutionChanged();
     Q_EMIT hasFlashChanged();
     Q_EMIT hasHdrChanged();
     Q_EMIT hdrEnabledChanged();
@@ -260,6 +272,8 @@
         }
         Q_EMIT activeCameraIndexChanged();
         Q_EMIT resolutionChanged();
+        Q_EMIT maximumResolutionChanged();
+        Q_EMIT fittingResolutionChanged();
         Q_EMIT hasFlashChanged();
         Q_EMIT videoSupportedResolutionsChanged();
     }
@@ -277,6 +291,117 @@
     return QSize();
 }
 
+QSize AdvancedCameraSettings::imageCaptureResolution() const
+{
+    if (m_imageEncoderControl != 0) {
+        return m_imageEncoderControl->imageSettings().resolution();
+    }
+
+    return QSize();
+}
+
+QSize AdvancedCameraSettings::videoRecorderResolution() const
+{
+    if (m_videoEncoderControl != 0) {
+        return m_videoEncoderControl->videoSettings().resolution();
+    }
+
+    return QSize();
+}
+
+QSize AdvancedCameraSettings::maximumResolution() const
+{
+    if (m_imageEncoderControl) {
+        QList<QSize> sizes = m_imageEncoderControl->supportedResolutions(
+                                       m_imageEncoderControl->imageSettings());
+
+        QSize maximumSize;
+        long maximumPixels = 0;
+
+        QList<QSize>::const_iterator it = sizes.begin();
+        while (it != sizes.end()) {
+            const long pixels = ((long)((*it).width())) * ((long)((*it).height()));
+            if (pixels > maximumPixels) {
+                maximumSize = *it;
+                maximumPixels = pixels;
+            }
+            ++it;
+        }
+
+        return maximumSize;
+    }
+
+    return QSize();
+}
+
+float AdvancedCameraSettings::getScreenAspectRatio() const
+{
+    float screenAspectRatio;
+    QScreen *screen = QGuiApplication::primaryScreen();
+    Q_ASSERT(!screen);
+    const int kScreenWidth = screen->geometry().width();
+    const int kScreenHeight = screen->geometry().height();
+    Q_ASSERT(kScreenWidth > 0 && kScreenHeight > 0);
+
+    screenAspectRatio = (kScreenWidth > kScreenHeight) ?
+        ((float)kScreenWidth / (float)kScreenHeight) : ((float)kScreenHeight / (float)kScreenWidth);
+
+    return screenAspectRatio;
+}
+
+QSize AdvancedCameraSettings::fittingResolution() const
+{
+    QList<float> prioritizedAspectRatios;
+    prioritizedAspectRatios.append(getScreenAspectRatio());
+    const float backAspectRatios[4] = { 16.0f/9.0f, 3.0f/2.0f, 4.0f/3.0f, 5.0f/4.0f };
+    for (int i=0; i<4; ++i) {
+        if (!prioritizedAspectRatios.contains(backAspectRatios[i])) {
+            prioritizedAspectRatios.append(backAspectRatios[i]);
+        }
+    }
+
+    if (m_imageEncoderControl) {
+        QList<QSize> sizes = m_imageEncoderControl->supportedResolutions(
+                                       m_imageEncoderControl->imageSettings());
+
+        QSize optimalSize;
+        long optimalPixels = 0;
+
+        if (!sizes.empty()) {
+            float aspectRatio;
+
+            // Loop over all reported camera resolutions until we find the highest
+            // one that matches the current prioritized aspect ratio. If it doesn't
+            // find one on the current aspect ration, it selects the next ratio and
+            // tries again.
+            QList<float>::const_iterator ratioIt = prioritizedAspectRatios.begin();
+            while (ratioIt != prioritizedAspectRatios.end()) {
+                // Don't update the aspect ratio when using this function for finding
+                // the optimal thumbnail size as it will affect the preview window size
+                aspectRatio = (*ratioIt);
+
+                QList<QSize>::const_iterator it = sizes.begin();
+                while (it != sizes.end()) {
+                    const float ratio = (float)(*it).width() / (float)(*it).height();
+                    const long pixels = ((long)((*it).width())) * ((long)((*it).height()));
+                    const float EPSILON = 0.02;
+                    if (fabs(ratio - aspectRatio) < EPSILON && pixels > optimalPixels) {
+                        optimalSize = *it;
+                        optimalPixels = pixels;
+                    }
+                    ++it;
+                }
+                if (optimalPixels > 0) break;
+                ++ratioIt;
+            }
+        }
+
+        return optimalSize;
+    }
+
+    return QSize();
+}
+
 QStringList AdvancedCameraSettings::videoSupportedResolutions() const
 {
     if (m_videoEncoderControl) {

=== modified file 'CameraApp/advancedcamerasettings.h'
--- CameraApp/advancedcamerasettings.h	2014-12-08 19:12:52 +0000
+++ CameraApp/advancedcamerasettings.h	2015-11-25 17:47:21 +0000
@@ -40,6 +40,10 @@
     Q_PROPERTY (int activeCameraIndex READ activeCameraIndex WRITE setActiveCameraIndex
                 NOTIFY activeCameraIndexChanged)
     Q_PROPERTY (QSize resolution READ resolution NOTIFY resolutionChanged)
+    Q_PROPERTY (QSize imageCaptureResolution READ imageCaptureResolution)
+    Q_PROPERTY (QSize videoRecorderResolution READ videoRecorderResolution)
+    Q_PROPERTY (QSize maximumResolution READ maximumResolution NOTIFY maximumResolutionChanged)
+    Q_PROPERTY (QSize fittingResolution READ fittingResolution NOTIFY fittingResolutionChanged)
     Q_PROPERTY (QStringList videoSupportedResolutions READ videoSupportedResolutions NOTIFY videoSupportedResolutionsChanged)
     Q_PROPERTY (bool hasFlash READ hasFlash NOTIFY hasFlashChanged)
     Q_PROPERTY (bool hdrEnabled READ hdrEnabled WRITE setHdrEnabled NOTIFY hdrEnabledChanged)
@@ -53,6 +57,11 @@
     void setCamera(QObject* camera);
     void setActiveCameraIndex(int index);
     QSize resolution() const;
+    QSize imageCaptureResolution() const;
+    QSize videoRecorderResolution() const;
+    QSize maximumResolution() const;
+    QSize fittingResolution() const;
+    float getScreenAspectRatio() const;
     QStringList videoSupportedResolutions() const;
     bool hasFlash() const;
     bool hasHdr() const;
@@ -66,6 +75,8 @@
     void cameraChanged();
     void activeCameraIndexChanged();
     void resolutionChanged();
+    void maximumResolutionChanged();
+    void fittingResolutionChanged();
     void hasFlashChanged();
     void hasHdrChanged();
     void hdrEnabledChanged();

=== modified file 'ViewFinderOverlay.qml'
--- ViewFinderOverlay.qml	2015-11-25 17:03:06 +0000
+++ ViewFinderOverlay.qml	2015-11-25 17:47:21 +0000
@@ -51,6 +51,12 @@
         property bool preferRemovableStorage: false
         property string videoResolution: "1920x1080"
         property bool playShutterSound: true
+        // FIXME: stores the resolution selected for 2 cameras. Instead it should:
+        //  - support any number of cameras
+        //  - not rely on the camera index but on Camera.deviceId
+        // Ref.: http://doc.qt.io/qt-5/qml-qtmultimedia-camera.html#deviceId-prop
+        property string photoResolution0
+        property string photoResolution1
 
         onFlashModeChanged: if (flashMode != Camera.FlashOff) hdrEnabled = false;
         onHdrEnabledChanged: if (hdrEnabled) flashMode = Camera.FlashOff
@@ -88,12 +94,83 @@
         value: settings.videoResolution
     }
 
+    Binding {
+        target: camera.imageCapture
+        property: "resolution"
+        value: settings["photoResolution" + camera.advanced.activeCameraIndex]
+    }
+
+    Connections {
+        target: camera.imageCapture
+        onResolutionChanged: {
+            // FIXME: this is a necessary workaround because:
+            // - Neither camera.viewfinder.resolution nor camera.advanced.resolution
+            //   emit a changed signal when the underlying AalViewfinderSettingsControl's
+            //   resolution changes
+            // - we know that qtubuntu-camera changes the resolution of the
+            //   viewfinder automatically when the capture resolution is set
+            // - we need camera.viewfinder.resolution to hold the right
+            //   value
+            camera.viewfinder.resolution = camera.advanced.resolution;
+        }
+    }
+
+    Connections {
+        target: camera.videoRecorder
+        onResolutionChanged: {
+            // FIXME: see workaround setting camera.viewfinder.resolution above
+            camera.viewfinder.resolution = camera.advanced.resolution;
+        }
+    }
+
+    Connections {
+        target: camera
+        onCaptureModeChanged: {
+            // FIXME: see workaround setting camera.viewfinder.resolution above
+            camera.viewfinder.resolution = camera.advanced.resolution;
+        }
+    }
+
     function resolutionToLabel(resolution) {
         // takes in a resolution string (e.g. "1920x1080") and returns a nicer
         // form of it for display in the UI: "1080p"
         return resolution.split("x").pop() + "p";
     }
 
+    function sizeToString(size) {
+        return size.width + "x" + size.height;
+    }
+
+    function stringToSize(resolution) {
+        var r = resolution.split("x");
+        return Qt.size(r[0], r[1]);
+    }
+
+    function sizeToAspectRatio(size) {
+        var ratio = Math.max(size.width, size.height) / Math.min(size.width, size.height);
+        var maxDenominator = 12;
+        var epsilon;
+        var numerator;
+        var denominator;
+        var bestDenominator;
+        var bestEpsilon = 10000;
+        for (denominator = 2; denominator <= maxDenominator; denominator++) {
+            numerator = ratio * denominator;
+            epsilon = Math.abs(Math.round(numerator) - numerator);
+            if (epsilon < bestEpsilon) {
+                bestEpsilon = epsilon;
+                bestDenominator = denominator;
+            }
+        }
+        numerator = Math.round(ratio * bestDenominator);
+        return "%1:%2".arg(numerator).arg(bestDenominator);
+    }
+
+    function sizeToMegapixels(size) {
+        var megapixels = (size.width * size.height) / 1000000;
+        return parseFloat(megapixels.toFixed(1))
+    }
+
     function updateVideoResolutionOptions() {
         // Clear and refill videoResolutionOptionsModel with available resolutions
         // Try to only display well known resolutions: 1080p, 720p and 480p
@@ -111,25 +188,71 @@
         }
 
         // If resolution setting chosen is not supported select the highest available resolution
-        if (supported.indexOf(settings.videoResolution) == -1) {
+        if (supported.length > 0 && supported.indexOf(settings.videoResolution) == -1) {
             settings.videoResolution = supported[supported.length - 1];
         }
     }
 
+    function updatePhotoResolutionOptions() {
+        // Clear and refill photoResolutionOptionsModel with available resolutions
+        photoResolutionOptionsModel.clear();
+
+        var optionMaximum = {"icon": "",
+                             "label": "%1 (%2MP)".arg(sizeToAspectRatio(camera.advanced.maximumResolution))
+                                                 .arg(sizeToMegapixels(camera.advanced.maximumResolution)),
+                             "value": sizeToString(camera.advanced.maximumResolution)};
+
+        var optionFitting = {"icon": "",
+                             "label": "%1 (%2MP)".arg(sizeToAspectRatio(camera.advanced.fittingResolution))
+                                                 .arg(sizeToMegapixels(camera.advanced.fittingResolution)),
+                             "value": sizeToString(camera.advanced.fittingResolution)};
+
+        photoResolutionOptionsModel.insert(0, optionMaximum);
+        if (camera.advanced.fittingResolution != camera.advanced.maximumResolution) {
+            photoResolutionOptionsModel.insert(1, optionFitting);
+        }
+
+        var photoResolution = settings["photoResolution" + camera.advanced.activeCameraIndex];
+        // If resolution setting chosen is not supported select the fitting resolution
+        if (photoResolution != optionFitting.value &&
+            photoResolution != optionMaximum.value) {
+            settings["photoResolution" + camera.advanced.activeCameraIndex] = optionFitting.value;
+        }
+    }
+
     Component.onCompleted: {
+        camera.cameraState = Camera.LoadedState;
         updateVideoResolutionOptions();
+        updatePhotoResolutionOptions();
+        // FIXME: see workaround setting camera.viewfinder.resolution above
+        camera.viewfinder.resolution = camera.advanced.resolution;
+        camera.cameraState = Camera.ActiveState;
     }
 
     Connections {
         target: camera.advanced
         onVideoSupportedResolutionsChanged: updateVideoResolutionOptions();
+        onFittingResolutionChanged: updatePhotoResolutionOptions();
+        onMaximumResolutionChanged: updatePhotoResolutionOptions();
     }
 
     Connections {
         target: camera.advanced
         onActiveCameraIndexChanged: {
+            var hasPhotoResolutionSetting = (settings["photoResolution" + camera.advanced.activeCameraIndex] != "")
+            // FIXME: use camera.advanced.imageCaptureResolution instead of camera.imageCapture.resolution
+            // because the latter is not updated when the backend changes the resolution
+            settings["photoResolution" + camera.advanced.activeCameraIndex] = sizeToString(camera.advanced.imageCaptureResolution);
+            settings.videoResolution = sizeToString(camera.advanced.videoRecorderResolution);
+            updatePhotoResolutionOptions();
             updateVideoResolutionOptions();
-            camera.videoRecorder.resolution = settings.videoResolution;
+            // FIXME: see workaround setting camera.viewfinder.resolution above
+            camera.viewfinder.resolution = camera.advanced.resolution;
+
+            // If no resolution has ever been chosen, select the one that fits the screen
+            if (!hasPhotoResolutionSetting) {
+                settings["photoResolution" + camera.advanced.activeCameraIndex] = sizeToString(camera.advanced.fittingResolution);
+            }
         }
     }
 
@@ -428,6 +551,18 @@
                         label: QT_TR_NOOP("Off")
                         value: false
                     }
+                },
+                ListModel {
+                    id: photoResolutionOptionsModel
+
+                    property string settingsProperty: "photoResolution" + camera.advanced.activeCameraIndex
+                    property string icon: ""
+                    property string label: sizeToAspectRatio(stringToSize(settings[settingsProperty]))
+                    property bool isToggle: false
+                    property int selectedIndex: bottomEdge.indexForValue(photoResolutionOptionsModel, settings[settingsProperty])
+                    property bool available: true
+                    property bool visible: camera.captureMode == Camera.CaptureStillImage
+                    property bool showInIndicators: false
                 }
             ]
 

=== modified file 'ViewFinderView.qml'
--- ViewFinderView.qml	2015-11-16 15:54:25 +0000
+++ ViewFinderView.qml	2015-11-25 17:47:21 +0000
@@ -36,6 +36,7 @@
     Camera {
         id: camera
         captureMode: Camera.CaptureStillImage
+        cameraState: Camera.UnloadedState
         StateSaver.properties: "captureMode"
 
         function manualFocus(x, y) {
@@ -67,10 +68,6 @@
             StateSaver.properties: "activeCameraIndex"
         }
 
-        Component.onCompleted: {
-            camera.start();
-        }
-        
         /* Use only digital zoom for now as it's what phone cameras mostly use.
                TODO: if optical zoom is available, maximumZoom should be the combined
                range of optical and digital zoom and currentZoom should adjust the two
@@ -121,12 +118,14 @@
         target: Qt.application
         onActiveChanged: {
             if (Qt.application.active) {
-                camera.start()
+                if (camera.cameraState == Camera.LoadedState) {
+                    camera.cameraState = Camera.ActiveState;
+                }
             } else if (!application.desktopMode) {
                 if (camera.videoRecorder.recorderState == CameraRecorder.RecordingState) {
                     camera.videoRecorder.stop();
                 }
-                camera.stop()
+                camera.cameraState = Camera.LoadedState;
             }
         }
     }
@@ -250,16 +249,16 @@
                 axis.x: 0; axis.y: 1; axis.z: 0
                 angle: application.desktopMode ? 180 : 0
             }
-
-            ViewFinderGeometry {
-                id: viewFinderGeometry
-                anchors.centerIn: parent
-
-                cameraResolution: camera.advanced.resolution
-                viewFinderHeight: viewFinder.height
-                viewFinderWidth: viewFinder.width
-                viewFinderOrientation: viewFinder.orientation
-            }
+        }
+
+        ViewFinderGeometry {
+            id: viewFinderGeometry
+            anchors.centerIn: parent
+
+            cameraResolution: camera.viewfinder.resolution
+            viewFinderHeight: viewFinder.height
+            viewFinderWidth: viewFinder.width
+            viewFinderOrientation: viewFinder.orientation
         }
 
         Item {