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

Issue 829935 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2018
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Priority not reported correctly for frames

Project Member Reported by jcivelli@chromium.org, Apr 6 2018

Issue description

The priority is not reported correctly in the network::ResourceResponse for frame navigations.
This causes the failure in the WebContentsImplBrowserTest.                       ResourceLoadCompleteWithScriptSubresource test.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 12 2018

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

commit 183f54d2a7d77a41cf953baf8c52ca3abf878269
Author: Jay Civelli <jcivelli@google.com>
Date: Thu Apr 12 07:39:13 2018

Remove priority from ResourceLoadInfo

The predictor code has been simplified and does not need to know about
resource requests priority anymore. As that code was supposed to be the
only consumer of that field, removing it from resource_load_info.mojom.

Bug:  829935 
Tbr: tsepez@chromium.org
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I36e5da88fb4e55af2ad3e94c27be59dd30e1b9c3
Reviewed-on: https://chromium-review.googlesource.com/999829
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Conley Owens <cco3@chromium.org>
Commit-Queue: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550081}
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/content/browser/loader/resource_loader.cc
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/content/browser/web_contents/web_contents_impl_browsertest.cc
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/content/public/common/resource_load_info.mojom
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/content/renderer/loader/resource_dispatcher.cc
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/content/renderer/loader/resource_dispatcher.h
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/services/network/public/cpp/network_param_ipc_traits.h
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/services/network/public/cpp/resource_response_info.cc
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/services/network/public/cpp/resource_response_info.h
[modify] https://crrev.com/183f54d2a7d77a41cf953baf8c52ca3abf878269/services/network/url_loader.cc

Status: Fixed (was: Untriaged)

Sign in to add a comment