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

Issue 16703003: Remove trampoline between threads in SupportedImageTypeValidator. (Closed)

Created:
7 years, 6 months ago by dcheng
Modified:
7 years, 6 months ago
CC:
chromium-reviews, tzik+watch_chromium.org, kinuko+watch
Visibility:
Public.

Description

Remove trampoline between threads in SupportedImageTypeValidator. It's not necessary to trampoline between the IO thread and the blocking worker pool. The actual image decoding takes place in another process anyway; the task runner is merely used to invoke the callbacks. BUG=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=205721

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -11 lines) Patch
M chrome/browser/image_decoder.h View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/media_galleries/fileapi/supported_image_type_validator.cc View 3 chunks +4 lines, -10 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
dcheng
7 years, 6 months ago (2013-06-09 15:18:52 UTC) #1
vandebo (ex-Chrome)
LGTM
7 years, 6 months ago (2013-06-10 19:04:11 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dcheng@chromium.org/16703003/1
7 years, 6 months ago (2013-06-10 19:04:24 UTC) #3
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=8036
7 years, 6 months ago (2013-06-10 20:38:27 UTC) #4
dcheng
+jhawkins for OWNERS approval.
7 years, 6 months ago (2013-06-10 21:26:47 UTC) #5
James Hawkins
lgtm
7 years, 6 months ago (2013-06-11 00:00:18 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dcheng@chromium.org/16703003/1
7 years, 6 months ago (2013-06-11 14:20:02 UTC) #7
commit-bot: I haz the power
Retried try job too often on win_rel for step(s) sync_integration_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&number=161232
7 years, 6 months ago (2013-06-11 18:29:46 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dcheng@chromium.org/16703003/1
7 years, 6 months ago (2013-06-11 23:20:58 UTC) #9
commit-bot: I haz the power
7 years, 6 months ago (2013-06-12 04:18:19 UTC) #10
Message was sent while issue was closed.
Change committed as 205721

Powered by Google App Engine
This is Rietveld 408576698