diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-03 05:03:22 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-03 05:03:22 +0000 |
| commit | 396c3a461549b02dcb830a287ac4656b2ca181e7 (patch) | |
| tree | dfd70c68c3fc4aa501a8f1bf13bf4853d2f7faef /chrome/common | |
| parent | 29a694916746e366724f7ac6f9aa0d103894f4a9 (diff) | |
| download | chromium_src-396c3a461549b02dcb830a287ac4656b2ca181e7.zip chromium_src-396c3a461549b02dcb830a287ac4656b2ca181e7.tar.gz chromium_src-396c3a461549b02dcb830a287ac4656b2ca181e7.tar.bz2 | |
Make the pepper 2D flush callback actually function as advertised. It will now
get called asynchronously when the bits are actually copied to the screen,
rather than synchronously from inside the paint function. This makes it useful
for plugins to use the callback for rate limiting.
This also adds a lot of infrastructure for running tests on pepper devices, and
includes a unit test for the new flush behavior.
I made the existing RenderProcess object an abstract interface and made the
existing MockProcess (renamed to be more clear) implement that. This avoids
a static cast that would actually crash during a unit test because some code
was hardcoded to expect a RenderProcess object.
This fixes base's IDMap iterator which has apparently never been used for an
IDMap with ownership semantics.
TEST=Unit test included
BUG=none
Review URL: http://codereview.chromium.org/661124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40490 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common')
| -rw-r--r-- | chrome/common/transport_dib.h | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/chrome/common/transport_dib.h b/chrome/common/transport_dib.h index e36e8bf..4f8051e 100644 --- a/chrome/common/transport_dib.h +++ b/chrome/common/transport_dib.h @@ -1,4 +1,4 @@ -// Copyright (c) 2006-2009 The Chromium Authors. All rights reserved. +// Copyright (c) 2010 The Chromium Authors. All rights reserved. // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. @@ -90,10 +90,15 @@ class TransportDIB { static Handle DefaultHandleValue() { return -1; } #endif - // Create a new TransportDIB - // size: the minimum size, in bytes - // epoch: Windows only: a global counter. See comment above. - // returns: NULL on failure + // Create a new TransportDIB, returning NULL on failure. + // + // The size is the minimum size in bytes of the memory backing the transport + // DIB (we may actually allocate more than that to give us better reuse when + // cached). + // + // The sequence number is used to uniquely identify the transport DIB. It + // should be unique for all transport DIBs ever created in the same + // renderer. static TransportDIB* Create(size_t size, uint32 sequence_num); // Map the referenced transport DIB. Returns NULL on failure. |