Issue 15812004: Remove the Activity Log UI

Issue 15812004: Remove the Activity Log UI (Closed)

Created:
7 years, 6 months ago by felt

Modified:
7 years, 6 months ago

Reviewers:
James Hawkins, Matt Perry

CC:
chromium-reviews, Aaron Boodman, arv+watch_chromium.org, chromium-apps-reviews_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Remove the Activity Log UI This CL removes the Activity Log UI. I will be replacing it with an extension once the chrome.activityLogPrivate API is complete. The only code changes in this CL are for code deletion. BUG=241672 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202959

Patch Set 1 #

Created: 7 years, 6 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+2 lines, -380 lines)			Patch
M	build/ios/grit_whitelist.txt	View	1 chunk	+0 lines, -2 lines	0 comments	Download
M	chrome/app/generated_resources.grd	View	3 chunks	+0 lines, -26 lines	0 comments	Download
M	chrome/browser/about_flags.cc	View	1 chunk	+0 lines, -7 lines	0 comments	Download
M	chrome/browser/browser_resources.grd	View	1 chunk	+0 lines, -2 lines	0 comments	Download
M	chrome/browser/extensions/activity_log/activity_log.cc	View	2 chunks	+1 line, -5 lines	0 comments	Download
M	chrome/browser/extensions/activity_log/activity_log_unittest.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	chrome/browser/extensions/api/developer_private/developer_private_api.cc	View	1 chunk	+0 lines, -1 line	0 comments	Download
D	chrome/browser/resources/extensions/extension_activity.css	View	1 chunk	+0 lines, -17 lines	0 comments	Download
D	chrome/browser/resources/extensions/extension_activity.html	View	1 chunk	+0 lines, -61 lines	0 comments	Download
D	chrome/browser/resources/extensions/extension_activity.js	View	1 chunk	+0 lines, -56 lines	0 comments	Download
M	chrome/browser/resources/extensions/extension_list.js	View	1 chunk	+0 lines, -17 lines	0 comments	Download
M	chrome/browser/resources/extensions/extensions.html	View	1 chunk	+0 lines, -2 lines	0 comments	Download
M	chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc	View	2 chunks	+0 lines, -6 lines	0 comments	Download
D	chrome/browser/ui/webui/extensions/extension_activity_ui.h	View	1 chunk	+0 lines, -45 lines	0 comments	Download
D	chrome/browser/ui/webui/extensions/extension_activity_ui.cc	View	1 chunk	+0 lines, -117 lines	0 comments	Download
M	chrome/browser/ui/webui/extensions/extension_settings_handler.cc	View	2 chunks	+0 lines, -5 lines	0 comments	Download
M	chrome/chrome_browser_ui.gypi	View	1 chunk	+0 lines, -2 lines	0 comments	Download
M	chrome/common/chrome_switches.h	View	1 chunk	+0 lines, -1 line	0 comments	Download
M	chrome/common/chrome_switches.cc	View	1 chunk	+0 lines, -3 lines	0 comments	Download
M	chrome/common/url_constants.h	View	2 chunks	+0 lines, -2 lines	0 comments	Download
M	chrome/common/url_constants.cc	View	2 chunks	+0 lines, -2 lines	0 comments	Download

Messages

Total messages: 6 (0 generated)

Expand Messages | Collapse Messages

felt

Matt, Could you please review this? This issue removes the AL UI. I separated it ...

7 years, 6 months ago (2013-05-29 04:45:30 UTC) #1

felt

James, Could you please review this? Just a bunch of deletions, happens to touch some ...

7 years, 6 months ago (2013-05-29 04:46:27 UTC) #2

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/felt@chromium.org/15812004/1

7 years, 6 months ago (2013-05-29 18:59:18 UTC) #5

Message was sent while issue was closed.

Change committed as 202959

Expand Messages | Collapse Messages