Issue 358993007: Fix nits in content/browser/appcache/

Issue 358993007: Fix nits in content/browser/appcache/ (Closed)

Created:
6 years, 5 months ago by pilgrim_google

Modified:
6 years, 5 months ago

Reviewers:
michaeln, darin (slow to review), jam

CC:
chromium-reviews, michaeln, darin-cc_chromium.org, jam

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

Project:
chromium

Visibility:
Public.

More Reviews

Description

Fix nits in content/browser/appcache/ Followup to https://codereview.chromium.org/344493002/ BUG=338338 TBR=darin@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280512

Patch Set 1 #

Created: 6 years, 5 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+52 lines, -118 lines)			Patch
M	content/browser/appcache/appcache.h	View	1 chunk	+3 lines, -7 lines	0 comments	Download
M	content/browser/appcache/appcache_database.h	View	1 chunk	+0 lines, -3 lines	0 comments	Download
M	content/browser/appcache/appcache_group.h	View	1 chunk	+2 lines, -6 lines	0 comments	Download
M	content/browser/appcache/appcache_host.h	View	1 chunk	+6 lines, -10 lines	0 comments	Download
M	content/browser/appcache/appcache_interceptor.h	View	1 chunk	+0 lines, -3 lines	0 comments	Download
M	content/browser/appcache/appcache_quota_client.h	View	1 chunk	+0 lines, -5 lines	0 comments	Download
M	content/browser/appcache/appcache_request_handler.h	View	1 chunk	+0 lines, -4 lines	0 comments	Download
M	content/browser/appcache/appcache_response.h	View	1 chunk	+1 line, -5 lines	0 comments	Download
M	content/browser/appcache/appcache_service_impl.h	View	1 chunk	+3 lines, -7 lines	0 comments	Download
M	content/browser/appcache/appcache_storage.h	View	1 chunk	+3 lines, -7 lines	0 comments	Download
M	content/browser/appcache/appcache_storage_impl.h	View	1 chunk	+0 lines, -3 lines	0 comments	Download
M	content/browser/appcache/appcache_update_job.h	View	1 chunk	+0 lines, -4 lines	0 comments	Download
M	content/browser/appcache/appcache_url_request_job.h	View	1 chunk	+2 lines, -6 lines	0 comments	Download
M	content/browser/appcache/view_appcache_internals_job.h	View	1 chunk	+0 lines, -3 lines	0 comments	Download
M	content/browser/quota/mock_quota_manager_unittest.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	content/browser/transition_browsertest.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	content/browser/webui/url_data_manager_backend.h	View	1 chunk	+2 lines, -4 lines	0 comments	Download
M	content/public/browser/resource_context.h	View	2 chunks	+2 lines, -4 lines	0 comments	Download
M	content/public/browser/resource_dispatcher_host_delegate.h	View	2 chunks	+6 lines, -12 lines	0 comments	Download
M	content/public/browser/storage_partition.h	View	2 chunks	+1 line, -4 lines	0 comments	Download
M	content/public/common/appcache_info.h	View	1 chunk	+19 lines, -19 lines	0 comments	Download

Messages

Total messages: 4 (0 generated)

Expand Messages | Collapse Messages

commit-bot: I haz the power

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

6 years, 5 months ago (2014-06-28 02:04:25 UTC) #3

Message was sent while issue was closed.

Change committed as 280512

Expand Messages | Collapse Messages