Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(49)

Issue 12321055: Revert 183382 (Closed)

Created:
7 years, 10 months ago by karen
Modified:
7 years, 10 months ago
Reviewers:
James Simonsen
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Revert 183382 > Add a ResourceScheduler to ResourceDispatcherHost. > > For this CL, ResourceScheduler mimicks WebKit's ResourceLoadScheduler. That means only JS and CSS are loaded before first paint. > > Eventually, we will improve it to do things like: > - Lower priority of background tabs. > - Preconnect for low priority resources. > - Dynamically adapt scheduling to the user's connection. > - Experiment with other scheduling ideas. > > BUG=157763 > > > Review URL: https://chromiumcodereview.appspot.com/11270027 TBR=simonjam@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=183843

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -779 lines) Patch
M content/browser/loader/resource_dispatcher_host_impl.h View 4 chunks +0 lines, -8 lines 0 comments Download
M content/browser/loader/resource_dispatcher_host_impl.cc View 6 chunks +35 lines, -18 lines 0 comments Download
D content/browser/loader/resource_scheduler.h View 1 chunk +0 lines, -113 lines 0 comments Download
D content/browser/loader/resource_scheduler.cc View 1 chunk +0 lines, -242 lines 0 comments Download
D content/browser/loader/resource_scheduler_filter.h View 1 chunk +0 lines, -35 lines 0 comments Download
D content/browser/loader/resource_scheduler_filter.cc View 1 chunk +0 lines, -49 lines 0 comments Download
D content/browser/loader/resource_scheduler_unittest.cc View 1 chunk +0 lines, -211 lines 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M content/common/resource_dispatcher.h View 1 chunk +0 lines, -4 lines 0 comments Download
M content/common/resource_dispatcher.cc View 4 chunks +0 lines, -19 lines 0 comments Download
M content/common/resource_messages.h View 3 chunks +1 line, -7 lines 0 comments Download
M content/common/view_messages.h View 1 chunk +0 lines, -5 lines 0 comments Download
M content/content_browser.gypi View 1 chunk +0 lines, -4 lines 0 comments Download
M content/content_tests.gypi View 1 chunk +0 lines, -1 line 0 comments Download
M content/public/common/common_param_traits_macros.h View 2 chunks +1 line, -2 lines 0 comments Download
M content/renderer/render_view_impl.h View 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/render_view_impl.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M webkit/glue/resource_loader_bridge.h View 3 chunks +1 line, -7 lines 0 comments Download
M webkit/glue/resource_loader_bridge.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/glue/weburlloader_impl.h View 1 chunk +0 lines, -1 line 0 comments Download
M webkit/glue/weburlloader_impl.cc View 5 chunks +1 line, -39 lines 0 comments Download
M webkit/tools/test_shell/simple_resource_loader_bridge.cc View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
karen
7 years, 10 months ago (2013-02-21 18:57:15 UTC) #1
karen
Committed patchset #1 manually as r183843.
7 years, 10 months ago (2013-02-21 18:57:38 UTC) #2
James Simonsen
7 years, 10 months ago (2013-02-21 19:01:24 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698