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

Issue 856150 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Blink Scheduler: Experiment with different priorities for ad frames.

Project Member Reported by farahcharab@chromium.org, Jun 25 2018

Issue description

Experiment with setting priority of ads to low priority or best effort priority during the loading phase or at all times.

 
Labels: -OS-Linux -OS-Android -OS-Windows -OS-iOS -OS-Chrome -OS-Mac
Description: Show this description
By the way, should we also add some metrics to track the amount of ad-related vs other work?
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 28 2018

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

commit 47b6d8317f1d48b7e6a9b91d468175791623631f
Author: Farah Charab <farahcharab@google.com>
Date: Thu Jun 28 13:47:08 2018

Scheduler: Experiment with different priorities for ad frames.

Set priority of ad frames to low or best effort depending on the
feature list enabled.

Bug: 856150
Change-Id: Ia76d55e1fc4b8617d5db9bca26b79ac25b5856c9
Reviewed-on: https://chromium-review.googlesource.com/1112251
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Alex Clarke <alexclarke@chromium.org>
Commit-Queue: Farah Charab <farahcharab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571103}
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/testing/variations/fieldtrial_testing_config.json
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/core/frame/local_frame.cc
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/core/frame/local_frame.h
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/child/features.h
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/main_thread/frame_scheduler_impl.cc
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/main_thread/frame_scheduler_impl.h
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/main_thread/frame_scheduler_impl_unittest.cc
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/main_thread/main_thread_scheduler_impl.cc
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/main_thread/main_thread_scheduler_impl.h
[modify] https://crrev.com/47b6d8317f1d48b7e6a9b91d468175791623631f/third_party/blink/renderer/platform/scheduler/public/frame_scheduler.h

Status: Started (was: Assigned)

Sign in to add a comment