From c5a5ee158569102adcdbcca961d39505a78199e8 Mon Sep 17 00:00:00 2001 From: oysteine Date: Fri, 29 May 2015 15:22:33 -0700 Subject: [PATCH] Revert of Revert of Flip extension background page throttling on by default (patchset #1 id:1 of https://codereview.chromium.org/1106953002/) Reason for revert: Re-landing, tests fixed in https://codereview.chromium.org/1108133002/ Original issue's description: > Revert of Flip extension background page throttling on by default (patchset #2 id:20001 of https://codereview.chromium.org/1106653002/) > > Reason for revert: > Speculative revert - multiple browser_tests fail on Mac10.9 Tests (dbg) > http://goo.gl/4AqhVG > > Original issue's description: > > Flip extension background page throttling on by default > > > > R=kalman > > BUG=424661 > > > > Committed: https://crrev.com/34ea66b8ac1d56dadd670a9f5f8431063857ffdd > > Cr-Commit-Position: refs/heads/master@{#326953} > > TBR=kalman@chromium.org,oysteine@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=424661 > > Committed: https://crrev.com/76a7e3446188256ca240dc31cd5287ef78de2951 > Cr-Commit-Position: refs/heads/master@{#327021} TBR=kalman@chromium.org,nkostylev@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=424661 Review URL: https://codereview.chromium.org/1161773008 Cr-Commit-Position: refs/heads/master@{#332062} --- extensions/browser/extension_host.cc | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/extensions/browser/extension_host.cc b/extensions/browser/extension_host.cc index 0b501a6ceef7..124a8fd2ddf3 100644 --- a/extensions/browser/extension_host.cc +++ b/extensions/browser/extension_host.cc @@ -153,12 +153,9 @@ void ExtensionHost::CreateRenderViewNow() { "464206 ExtensionHost::CreateRenderViewNow2")); DCHECK(IsRenderViewLive()); if (extension_) { - std::string group_name = base::FieldTrialList::FindFullName( - "ThrottleExtensionBackgroundPages"); - if ((group_name == "ThrottlePersistent" && - extensions::BackgroundInfo::HasPersistentBackgroundPage( - extension_)) || - group_name == "ThrottleAll") { + if (extensions::BackgroundInfo::HasPersistentBackgroundPage(extension_) && + base::FieldTrialList::FindFullName( + "ThrottleExtensionBackgroundPages") != "Disabled") { host_contents_->WasHidden(); } } -- 2.11.4.GIT