Issue 642233002: set const ResourceLoaderHost::isControlledByServiceWorker()

Issue 642233002: set const ResourceLoaderHost::isControlledByServiceWorker() (Closed)

Created:
6 years, 2 months ago by horo

Modified:
6 years, 2 months ago

Reviewers:
falken, Mike West

CC:
blink-reviews, Nate Chapin, gavinp+loader_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/blink.git@master

Project:
blink

Visibility:
Public.

set const ResourceLoaderHost::isControlledByServiceWorker() BUG=na Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183551

Created: 6 years, 2 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+3 lines, -3 lines)			Patch
M	Source/core/fetch/ResourceFetcher.h	View	1 chunk	+1 line, -1 line	0 comments	Download
M	Source/core/fetch/ResourceFetcher.cpp	View	1 chunk	+1 line, -1 line	0 comments	Download
M	Source/core/fetch/ResourceLoaderHost.h	View	1 chunk	+1 line, -1 line	0 comments	Download

Total messages: 10 (4 generated)

horo

falken@ Could you please review this? My comment in https://codereview.chromium.org/600393004/#msg41 was wrong. It could be ...

6 years, 2 months ago (2014-10-10 06:00:16 UTC) #2

Mike West

On 2014/10/10 06:00:16, horo wrote: > falken@ > Could you please review this? > > ...

6 years, 2 months ago (2014-10-10 06:48:23 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/642233002/1

6 years, 2 months ago (2014-10-10 16:09:56 UTC) #6

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/642233002/1

6 years, 2 months ago (2014-10-10 16:10:13 UTC) #9

commit-bot: I haz the power

6 years, 2 months ago (2014-10-10 18:10:16 UTC) #10

Message was sent while issue was closed.

Committed patchset #1 (id:1) as 183551