From 84934149e1e479c03f835e7a1319ccdf558eda7e Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Fri, 10 Sep 2021 14:33:50 +0200
Subject: [PATCH] MOBILE-3844 media: Hide download option in audio and video

---
 .../url/components/index/addon-mod-url-index.html    |  2 +-
 src/core/directives/format-text.ts                   | 12 ++++++++++++
 .../components/capture-media/capture-media.html      |  6 ++++--
 src/core/services/utils/mimetype.ts                  |  2 +-
 4 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/src/addons/mod/url/components/index/addon-mod-url-index.html b/src/addons/mod/url/components/index/addon-mod-url-index.html
index 48f9b1f91..7c13f1b46 100644
--- a/src/addons/mod/url/components/index/addon-mod-url-index.html
+++ b/src/addons/mod/url/components/index/addon-mod-url-index.html
@@ -32,7 +32,7 @@
         <ion-item *ngIf="shouldEmbed">
             <ion-label>
                 <img *ngIf="isImage" [alt]="name" [src]="url">
-                <video *ngIf="isVideo" [title]="name" controls>
+                <video *ngIf="isVideo" [title]="name" controls controlsList="nodownload">
                     <source [src]="url" [type]="mimetype">
                 </video>
                 <audio *ngIf="isAudio" [title]="name" controls>
diff --git a/src/core/directives/format-text.ts b/src/core/directives/format-text.ts
index 719c1845c..22b5d3e4a 100644
--- a/src/core/directives/format-text.ts
+++ b/src/core/directives/format-text.ts
@@ -678,6 +678,18 @@ export class CoreFormatTextDirective implements OnChanges {
         this.addMediaAdaptClass(element);
         this.addExternalContent(element);
 
+        // Hide download button if not hidden already.
+        let controlsList = element.getAttribute('controlsList') || '';
+        if (!controlsList.includes('nodownload')) {
+            if (!controlsList.trim()) {
+                controlsList = 'nodownload';
+            } else {
+                controlsList = controlsList.split(' ').concat('nodownload').join(' ');
+            }
+
+            element.setAttribute('controlsList', controlsList);
+        }
+
         const sources = Array.from(element.querySelectorAll('source'));
         const tracks = Array.from(element.querySelectorAll('track'));
 
diff --git a/src/core/features/emulator/components/capture-media/capture-media.html b/src/core/features/emulator/components/capture-media/capture-media.html
index 544e4dc95..19c040eba 100644
--- a/src/core/features/emulator/components/capture-media/capture-media.html
+++ b/src/core/features/emulator/components/capture-media/capture-media.html
@@ -15,7 +15,9 @@
             <video *ngIf="!isAudio" [hidden]="hasCaptured" class="core-webcam-stream" autoplay #streamVideo></video>
 
             <!-- For video recording, use 2 videos and show/hide them because a CSS rule caused problems with the controls. -->
-            <video *ngIf="isVideo" [hidden]="!hasCaptured" class="core-webcam-video-captured" controls #previewVideo></video>
+            <video *ngIf="isVideo" [hidden]="!hasCaptured" class="core-webcam-video-captured" controls #previewVideo
+                controlsList="nodownload">
+            </video>
 
             <!-- Canvas to treat the image and an img to show the result. -->
             <canvas *ngIf="isImage" class="core-webcam-image-canvas" #imgCanvas></canvas>
@@ -35,7 +37,7 @@
                 </ion-button>
 
                 <!-- Audio player to listen to the result. -->
-                <audio [hidden]="!hasCaptured" class="core-audio-captured" controls #previewAudio></audio>
+                <audio [hidden]="!hasCaptured" class="core-audio-captured" controls #previewAudio controlsList="nodownload"></audio>
             </div>
         </div>
     </core-loading>
diff --git a/src/core/services/utils/mimetype.ts b/src/core/services/utils/mimetype.ts
index 9f5526ef3..78ea7322f 100644
--- a/src/core/services/utils/mimetype.ts
+++ b/src/core/services/utils/mimetype.ts
@@ -191,7 +191,7 @@ export class CoreMimetypeUtilsProvider {
                 case 'audio':
                 case 'video':
                     return [
-                        `<${embedType} controls title="${filename}" src="${path}">`,
+                        `<${embedType} controls title="${filename}" src="${path}" controlsList="nodownload">`,
                         `<source src="${path}" type="${mimeType}">`,
                         `</${embedType}>`,
                     ].join('');