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

Issue 10867087: Revert 153136 - Revert 139366 - [OSX/Pepper] Don't have browser cache shmem ref behind ImageData. (Closed)

Created:
8 years, 3 months ago by sail
Modified:
8 years, 3 months ago
Reviewers:
sail
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Revert 153136 - Revert 139366 - [OSX/Pepper] Don't have browser cache shmem ref behind ImageData. On OSX the browser has to create shmem blocks for sandboxed processes. The browser process can keep a handle for synchronizing between processes by id, but pepper's implementation just passes the handle between processes so that is not necessary. Before, the browser ran out of handles which breaks all sorts of stuff, but everything kept running, with various visual glitches (and no fds in browser is really bad). With this change, the renderer crashes. That sounds bad, but is actually better in terms of resolving the underlying problems. Also, histogram sizes for all platforms. Using a TransportDIB for small images is probably a bad idea. The histograms should show how things are being used in the wild. BUG=129430, 141541 TEST=flapper works, histograms appear. Review URL: https://chromiumcodereview.appspot.com/10424007 TBR=shess@chromium.org Review URL: https://chromiumcodereview.appspot.com/10879050 TBR=sail@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=153492

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -11 lines) Patch
M content/renderer/pepper/pepper_platform_image_2d_impl.cc View 3 chunks +5 lines, -11 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
sail
8 years, 3 months ago (2012-08-27 16:47:51 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698