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

Issue 1607004: Add the proxy information to the new net internals page. (Closed)

Created:
10 years, 8 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 proxy information to the new net internals page. BUG=37421 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=43636

Patch Set 1 #

Total comments: 1

Patch Set 2 : Pass the bad proxies as a list, rather than flat HTML #

Patch Set 3 : Max the URL work in presence of '#' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+404 lines, -44 lines) Patch
M chrome/browser/dom_ui/net_internals_ui.cc View 1 2 17 chunks +143 lines, -16 lines 0 comments Download
M chrome/browser/resources/net_internals/index.html View 1 2 chunks +21 lines, -1 line 0 comments Download
M chrome/browser/resources/net_internals/main.css View 1 chunk +11 lines, -1 line 0 comments Download
M chrome/browser/resources/net_internals/main.js View 1 3 chunks +165 lines, -25 lines 0 comments Download
A chrome/browser/resources/net_internals/proxyview.js View 1 1 chunk +62 lines, -0 lines 0 comments Download
M chrome/browser/resources/net_internals/requestsview.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
eroman
Porting more of about:net-internals to the javascript frontend.
10 years, 8 months ago (2010-04-02 21:45:15 UTC) #1
eroman
Updated to address a TODO.
10 years, 8 months ago (2010-04-05 01:43:44 UTC) #2
Mike Belshe
10 years, 8 months ago (2010-04-05 16:45:54 UTC) #3
lgtm

http://codereview.chromium.org/1607004/diff/1/7
File chrome/browser/dom_ui/net_internals_ui.cc (right):

http://codereview.chromium.org/1607004/diff/1/7#newcode336
chrome/browser/dom_ui/net_internals_ui.cc:336:
CallJavascriptFunction(L"g_browser.receivedLogEventTypeConstants", dict);
nit:  Do you think these function names should be made into constants?

Powered by Google App Engine
This is Rietveld 408576698