New issue
Advanced search Search tips

Issue 763322 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Feature



Sign in to add a comment

YouTube Flash Embed Rewrite: enable it for enablejsapi on desktop too

Project Member Reported by mlamouri@chromium.org, Sep 8 2017

Issue description

YouTube has deprecated Flash embeds so it will provide a better experience for our users to have them inlined with possible breakage instead of having to use Flash and navigate to YouTube.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 13 2017

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

commit a92266e9d9d2d68f33ffecbfe156cd8c83897831
Author: Mounir Lamouri <mlamouri@chromium.org>
Date: Wed Sep 13 10:35:16 2017

YouTube Flash Embeds: rewrite all embeds, even enablejsapi=1.

YouTube has deprecated Flash embeds so it will provide a better
experience for our users to have them inlined with possible breakage
instead of having to use Flash and navigate to YouTube.

Bug:  763322 
Change-Id: I3243e3048f499ec51e3220ebd18201a94e3eca46
Reviewed-on: https://chromium-review.googlesource.com/657277
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501587}
[modify] https://crrev.com/a92266e9d9d2d68f33ffecbfe156cd8c83897831/chrome/renderer/chrome_content_renderer_client.cc
[modify] https://crrev.com/a92266e9d9d2d68f33ffecbfe156cd8c83897831/chrome/renderer/chrome_content_renderer_client_browsertest.cc
[modify] https://crrev.com/a92266e9d9d2d68f33ffecbfe156cd8c83897831/chrome/renderer/chrome_content_renderer_client_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment