Issue 858743002: Make ThrottleController an ExecutionContext supplement.

Issue 858743002: Make ThrottleController an ExecutionContext supplement. (Closed)

Created:
5 years, 11 months ago by sof

Modified:
5 years, 11 months ago

Reviewers:
oilpan-reviews, kinuko, haraken, nhiroki

CC:
blink-reviews, kinuko+fileapi, nhiroki, tzik

Base URL:
svn://svn.chromium.org/blink/trunk

Target Ref:
refs/heads/master

Project:
blink

Visibility:
Public.

More Reviews

Description

Make ThrottleController an ExecutionContext supplement. Simplify the FileReader's throttle controller's supplement assocation, turning it into a supplement of ExecutionContext only. R=haraken,nhiroki,kinuko BUG=340515 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188795

Patch Set 1 #

Patch Set 2 : check for document-detached state + rebase #

Created: 5 years, 11 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+19 lines, -34 lines)			Patch
	M	Source/core/fileapi/FileReader.h	View	1	1 chunk	+0 lines, -1 line	0 comments	Download
	M	Source/core/fileapi/FileReader.cpp	View	1	7 chunks	+19 lines, -33 lines	0 comments	Download

Messages

Total messages: 8 (2 generated)

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages

sof

Please take a look. Not much in it, but slightly simpler code by changing its ...

5 years, 11 months ago (2015-01-21 21:36:14 UTC) #2

commit-bot: I haz the power

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

5 years, 11 months ago (2015-01-22 07:52:59 UTC) #7

commit-bot: I haz the power

5 years, 11 months ago (2015-01-22 07:56:28 UTC) #8

Message was sent while issue was closed.

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188795

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages