The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e43859889b1cac0f6ba7d1855cb8e7753c8f89ba

commit e43859889b1cac0f6ba7d1855cb8e7753c8f89ba
Author: qinmin <qinmin@chromium.org>
Date: Mon Oct 10 23:47:01 2016

Don't show download button for HLS streams

Downloading the m3u8 file isn't useful to user.

BUG= 654507 

Review-Url: https://codereview.chromium.org/2400293003
Cr-Commit-Position: refs/heads/master@{#424291}

[modify] https://crrev.com/e43859889b1cac0f6ba7d1855cb8e7753c8f89ba/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/e43859889b1cac0f6ba7d1855cb8e7753c8f89ba/third_party/WebKit/Source/core/html/HTMLMediaElement.h
[modify] https://crrev.com/e43859889b1cac0f6ba7d1855cb8e7753c8f89ba/third_party/WebKit/Source/core/html/shadow/MediaControlElements.cpp

Guessing this is OS-Android, but please be sure to mark OSes in the future for merge requests, that's how we process them.

Approved for M55 branch 2883.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/07b38a3f37f9578807bf73661f084f23f0812f32

commit 07b38a3f37f9578807bf73661f084f23f0812f32
Author: Min Qin <qinmin@chromium.org>
Date: Tue Oct 11 19:53:55 2016

Don't show download button for HLS streams

Downloading the m3u8 file isn't useful to user.

TBR=mlamouri@chromium.org
BUG= 654507 

Review-Url: https://codereview.chromium.org/2400293003
Cr-Commit-Position: refs/heads/master@{#424291}
(cherry picked from commit e43859889b1cac0f6ba7d1855cb8e7753c8f89ba)

Review URL: https://codereview.chromium.org/2411043002 .

Cr-Commit-Position: refs/branch-heads/2883@{#42}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/07b38a3f37f9578807bf73661f084f23f0812f32/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/07b38a3f37f9578807bf73661f084f23f0812f32/third_party/WebKit/Source/core/html/HTMLMediaElement.h
[modify] https://crrev.com/07b38a3f37f9578807bf73661f084f23f0812f32/third_party/WebKit/Source/core/html/shadow/MediaControlElements.cpp

Thanks! :)

verified in M56-56.0.2889.0

Verified in M55-55.0.2883.18 build

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/07b38a3f37f9578807bf73661f084f23f0812f32

commit 07b38a3f37f9578807bf73661f084f23f0812f32
Author: Min Qin <qinmin@chromium.org>
Date: Tue Oct 11 19:53:55 2016

Don't show download button for HLS streams

Downloading the m3u8 file isn't useful to user.

TBR=mlamouri@chromium.org
BUG= 654507 

Review-Url: https://codereview.chromium.org/2400293003
Cr-Commit-Position: refs/heads/master@{#424291}
(cherry picked from commit e43859889b1cac0f6ba7d1855cb8e7753c8f89ba)

Review URL: https://codereview.chromium.org/2411043002 .

Cr-Commit-Position: refs/branch-heads/2883@{#42}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/07b38a3f37f9578807bf73661f084f23f0812f32/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/07b38a3f37f9578807bf73661f084f23f0812f32/third_party/WebKit/Source/core/html/HTMLMediaElement.h
[modify] https://crrev.com/07b38a3f37f9578807bf73661f084f23f0812f32/third_party/WebKit/Source/core/html/shadow/MediaControlElements.cpp

[Automated comment] removing mislabelled merge-merged-2840

[Automated comment] removing mislabelled merge-merged-2840

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b6920f2e6be77a44ee16b052b45b2c4228854ae0

commit b6920f2e6be77a44ee16b052b45b2c4228854ae0
Author: johnme <johnme@chromium.org>
Date: Tue Feb 21 22:17:06 2017

Media Controls: Remove download button for infinite streams

Specifically, removes the download button for media with infinite
duration, since there is no clear end at which to finish the download.

Also adds tests for the download button being hidden for empty URLs and
HLS streams, and fixes the test for the download button being hidden
when hideDownloadUI setting is disabled by making its video non-local.

BUG= 691561 , 654507 ,650738

Review-Url: https://codereview.chromium.org/2710713003
Cr-Commit-Position: refs/heads/master@{#451852}

[rename] https://crrev.com/b6920f2e6be77a44ee16b052b45b2c4228854ae0/third_party/WebKit/LayoutTests/http/tests/media/video-controls-download-button-not-displayed-hide-download-ui.html
[modify] https://crrev.com/b6920f2e6be77a44ee16b052b45b2c4228854ae0/third_party/WebKit/Source/core/html/shadow/MediaControlElements.cpp
[modify] https://crrev.com/b6920f2e6be77a44ee16b052b45b2c4228854ae0/third_party/WebKit/Source/core/html/shadow/MediaControlsTest.cpp