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

Issue 749174 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Enabled PPS Nano by Default (i.e. remove the Finch Flag)

Project Member Reported by lafo...@chromium.org, Jul 26 2017

Issue description

PPS Nano is currently shipping to Stable users in Chrome 60.  Now would be a good time to remove the feature flag on trunk (and potentially merge back to 61).
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 1 2017

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

commit c1081e9467418ca4289f767a625ffbfa0f32f692
Author: Tommy C. Li <tommycli@chromium.org>
Date: Tue Aug 01 23:45:04 2017

Plugin Power Saver: Remove the same-origin tiny plugin exception flag.

It's been enabled by default for a long time. We're just removing the
feature flag now.

Bug:  749174 
Change-Id: I6cad01d596855c4320e219597789347c63a5b9a6
Reviewed-on: https://chromium-review.googlesource.com/587428
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>
Commit-Queue: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491156}
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/chrome/browser/plugins/flash_permission_browsertest.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/chrome/browser/plugins/plugin_power_saver_browsertest.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/chrome/browser/prerender/prerender_browsertest.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/chrome/renderer/chrome_content_renderer_client.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/chrome/renderer/plugins/power_saver_info.cc
[delete] https://crrev.com/a0679cfb4e4b417ed0fdedb88d0133e5fddd8118/chrome/test/data/plugin_power_saver/same_origin_tiny_plugin.html
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/chrome/test/data/prerender/prerender_plugin_delay_load.html
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/content/public/common/content_features.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/content/public/common/content_features.h
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/content/public/common/content_switches.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/content/renderer/peripheral_content_heuristic.cc
[modify] https://crrev.com/c1081e9467418ca4289f767a625ffbfa0f32f692/content/renderer/peripheral_content_heuristic_unittest.cc

Status: Fixed (was: Assigned)
Since we have successfully updated the field trial to include 62, I see no reason to merge this back to 61. 

Sign in to add a comment