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

Issue 140044: Reorganize the way the task manager is constructed.... (Closed)

Created:
11 years, 6 months ago by Ben Goodger (Google)
Modified:
9 years, 6 months ago
Reviewers:
jcampan
CC:
chromium-reviews_googlegroups.com, John Grabowski, Ben Goodger (Google), Paweł Hajdan Jr.
Visibility:
Public.

Description

Reorganize the way the task manager is constructed. The BrowserWindow creates the task manager UI directly, which uses the TaskManager to populate itself. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=18975

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+169 lines, -975 lines) Patch
M chrome/browser/browser.cc View 1 1 chunk +1 line, -6 lines 0 comments Download
M chrome/browser/browser_window.h View 1 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/browser_window_cocoa.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/cocoa/browser_window_cocoa.mm View 1 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/gtk/browser_window_gtk.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/gtk/browser_window_gtk.cc View 1 1 chunk +4 lines, -0 lines 0 comments Download
A + chrome/browser/gtk/task_manager_gtk.cc View 1 2 chunks +6 lines, -39 lines 0 comments Download
M chrome/browser/task_manager.h View 1 5 chunks +12 lines, -37 lines 0 comments Download
M chrome/browser/task_manager.cc View 1 2 chunks +14 lines, -67 lines 0 comments Download
M chrome/browser/task_manager_browsertest.cc View 1 1 chunk +3 lines, -7 lines 0 comments Download
D chrome/browser/task_manager_linux.cc View 1 1 chunk +0 lines, -75 lines 0 comments Download
M chrome/browser/task_manager_unittest.cc View 1 1 chunk +0 lines, -1 line 0 comments Download
D chrome/browser/task_manager_win.cc View 1 1 chunk +0 lines, -633 lines 0 comments Download
M chrome/browser/views/browser_dialogs.h View 1 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/views/frame/browser_view.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/views/frame/browser_view.cc View 1 1 chunk +4 lines, -0 lines 0 comments Download
A + chrome/browser/views/task_manager_view.cc View 1 15 chunks +109 lines, -108 lines 0 comments Download
M chrome/chrome.gyp View 1 3 chunks +2 lines, -2 lines 0 comments Download
M chrome/test/test_browser_window.h View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Ben Goodger (Google)
11 years, 6 months ago (2009-06-21 17:26:25 UTC) #1
jcampan
11 years, 6 months ago (2009-06-22 18:52:07 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698