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

Issue 9166009: Move about:net-internals webui files into a subdirectory + OWNERS (Closed)

Created:
8 years, 11 months ago by mmenke
Modified:
8 years, 11 months ago
Reviewers:
eroman, Evan Stade
CC:
chromium-reviews, cbentzel+watch_chromium.org, eroman, darin-cc_chromium.org, mmenke, Nirnimesh
Visibility:
Public.

Description

Move about:net-internals webui files into a subdirectory and add an OWNERs file for it. R=estade@chromium.org,eroman@chromium.org BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=117221

Patch Set 1 #

Patch Set 2 : Update year #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -2328 lines) Patch
M WATCHLISTS View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/net/net_log_logger.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/chrome_web_ui_factory.cc View 1 1 chunk +1 line, -1 line 0 comments Download
A chrome/browser/ui/webui/net_internals/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download
A + chrome/browser/ui/webui/net_internals/net_internals_ui.h View 1 2 chunks +4 lines, -4 lines 0 comments Download
A + chrome/browser/ui/webui/net_internals/net_internals_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
A + chrome/browser/ui/webui/net_internals/net_internals_ui_browsertest.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
D chrome/browser/ui/webui/net_internals_ui.h View 1 1 chunk +0 lines, -27 lines 0 comments Download
D chrome/browser/ui/webui/net_internals_ui.cc View 1 1 chunk +0 lines, -1647 lines 0 comments Download
D chrome/browser/ui/webui/net_internals_ui_browsertest.cc View 1 1 chunk +0 lines, -643 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
mmenke
Change in plans. I'll land this one first, then update the other CL. Figure this ...
8 years, 11 months ago (2012-01-10 16:19:08 UTC) #1
eroman
lgtm
8 years, 11 months ago (2012-01-10 20:51:03 UTC) #2
Evan Stade
8 years, 11 months ago (2012-01-10 21:21:12 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698