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

Issue 1994006: Add the requests dumps to the text summary on chrome://net2#data (Closed)

Created:
10 years, 7 months ago by eroman
Modified:
9 years, 6 months ago
Reviewers:
Mike Belshe
CC:
chromium-reviews, arv (Not doing code reviews), ben+cc_chromium.org
Visibility:
Public.

Description

Add the requests dumps to the text summary on chrome://net2#data BUG=37421 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=46748

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+233 lines, -41 lines) Patch
M chrome/browser/resources/net_internals/dataview.js View 1 chunk +50 lines, -3 lines 0 comments Download
M chrome/browser/resources/net_internals/index.html View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/net_internals/logviewpainter.js View 4 chunks +50 lines, -35 lines 0 comments Download
M chrome/browser/resources/net_internals/main.css View 1 chunk +5 lines, -1 line 0 comments Download
M chrome/browser/resources/net_internals/sourceentry.js View 1 chunk +10 lines, -1 line 0 comments Download
M chrome/browser/resources/net_internals/util.js View 1 chunk +117 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
eroman
10 years, 7 months ago (2010-05-07 18:38:50 UTC) #1
willchan no longer on Chromium
I'm punting this review to mbelshe since I am totally the wrong guy to do ...
10 years, 7 months ago (2010-05-07 18:49:46 UTC) #2
Mike Belshe
10 years, 7 months ago (2010-05-07 21:17:36 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698