New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 661220 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Android MediaRouter only (left Chro...
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 658313



Sign in to add a comment

Only prevent media preload for network files

Project Member Reported by mlamouri@chromium.org, Nov 1 2016

Issue description

Don't block preload on files that are not http:// or https:// such as blob://, data:// or file://.
 
Labels: M-56
Cc: mlamouri@chromium.org
Owner: zqzh...@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 5 2016

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

commit f9d8e609f1298c6a563274fbdbe6ee5c45df3034
Author: zqzhang <zqzhang@chromium.org>
Date: Mon Dec 05 18:25:42 2016

[MediaElement] Allow preloading for non-network scheme src

This CL weakens the restriction of media element preloading. The
reason for this change is that the preload attribute for media
elements is overriden as "none" when data saver is
enabled. However there is no need to apply this override for
non-network media sources, since they do not cost network
traffic.

BUG= 661220 

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

[rename] https://crrev.com/f9d8e609f1298c6a563274fbdbe6ee5c45df3034/third_party/WebKit/LayoutTests/http/tests/media/preload-conditions.html
[modify] https://crrev.com/f9d8e609f1298c6a563274fbdbe6ee5c45df3034/third_party/WebKit/Source/core/html/HTMLMediaElement.cpp
[modify] https://crrev.com/f9d8e609f1298c6a563274fbdbe6ee5c45df3034/third_party/WebKit/Source/core/html/HTMLMediaElement.h
[modify] https://crrev.com/f9d8e609f1298c6a563274fbdbe6ee5c45df3034/third_party/WebKit/Source/core/html/HTMLMediaElementTest.cpp

Thanks for fixing!
Status: Fixed (was: Assigned)

Sign in to add a comment