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

Issue 11092064: Use blocking worker pool for sync file operations in Pepper. (Closed)

Created:
8 years, 2 months ago by Scott Hess - ex-Googler
Modified:
7 years, 7 months ago
Reviewers:
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

[Manually closed, I think something similar was already landed.] Use blocking worker pool for sync file operations in Pepper. Switch from posting file operations on the file thread to posting them on the blocking worker pool. Previous instrumentation indicated that this reduces hung-plugin infobars by around 20%. BUG=153383 Revert: Flapper field trial to use workerpool for sync file operations. Original: https://chromiumcodereview.appspot.com/11016016 The field trial indicated a 20% improvement using blocking workerpool. That seems worth merging to M-23. Revert the change so that a clean change can be landed and drover'ed over. BUG=153383

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -59 lines) Patch
M content/browser/renderer_host/pepper/pepper_file_message_filter.h View 1 chunk +0 lines, -3 lines 0 comments Download
M content/browser/renderer_host/pepper/pepper_file_message_filter.cc View 3 chunks +1 line, -41 lines 0 comments Download
M content/browser/web_contents/web_contents_impl.cc View 2 chunks +0 lines, -15 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698