Chromium Code Reviews
Help | Chromium Project | Sign in
(44)

Issue 2756373003: Record the data use of service worker initiated requests (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 3 days ago by Raj
Modified:
1 week, 2 days ago
Reviewers:
rajendrant, Ryan Sturm
CC:
chromium-reviews, jam, darin-cc_chromium.org, asvitkine+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Record the data use of service worker initiated requests BUG=673820

Patch Set 1 #

Patch Set 2 : Fix unittest #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -0 lines) Patch
M components/data_use_measurement/content/content_url_request_classifier.h View 1 chunk +2 lines, -0 lines 0 comments Download
M components/data_use_measurement/content/content_url_request_classifier.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M components/data_use_measurement/core/data_use_measurement.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M components/data_use_measurement/core/data_use_measurement_unittest.cc View 1 1 chunk +4 lines, -0 lines 0 comments Download
M components/data_use_measurement/core/data_use_network_delegate_unittest.cc View 1 1 chunk +4 lines, -0 lines 0 comments Download
M components/data_use_measurement/core/data_use_user_data.h View 1 chunk +1 line, -0 lines 0 comments Download
M components/data_use_measurement/core/data_use_user_data.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M components/data_use_measurement/core/url_request_classifier.h View 1 chunk +3 lines, -0 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 1 chunk +1 line, -0 lines 0 comments Download
Commit queue not available (can’t edit this change).

Messages

Total messages: 11 (9 generated)
rajendrant
1 week, 2 days ago (2017-03-20 20:44:27 UTC) #10
rajendrant
1 week, 2 days ago (2017-03-21 15:56:35 UTC) #11
Will close this CL.
Will create a separate CL for recording different content::ResourceType that
will cover serviceworkers as well.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld cc6ac46