Fixed lingering gpu-path AA clip mask generation bug

http://codereview.appspot.com/6351055/

git-svn-id: http://skia.googlecode.com/svn/trunk@4416 2bbb7eff-a529-9590-31e7-b0007b416f81

Partially rolled forward the SkGrPixelRef changes

http://codereview.appspot.com/6354060/

git-svn-id: http://skia.googlecode.com/svn/trunk@4415 2bbb7eff-a529-9590-31e7-b0007b416f81

Only skip pipe playback for blurs GM in FIXED mode.

Only skip it when it would actually fail. Also add a comment
describing why it fails:

// SkCanvas::drawCircle, used by this test, performs a quick reject.
// The large size given to the device used by SkGPipeCanvas means that
// the device clip will not be set properly and circles will be
// rejected when in FIXED.

Review URL: https://codereview.appspot.com/6354058

git-svn-id: http://skia.googlecode.com/svn/trunk@4414 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix a warning on linux when compiling SkGPipeWrite.cpp

Review URL: https://codereview.appspot.com/6349057

git-svn-id: http://skia.googlecode.com/svn/trunk@4413 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix build by reverting make all to not include the debugger

git-svn-id: http://skia.googlecode.com/svn/trunk@4412 2bbb7eff-a529-9590-31e7-b0007b416f81

Un-trifurcate GrTextContext: get rid of Default and Batched subclasses,
folding their functionality back into the base class.

Requires gyp changes.

http://codereview.appspot.com/6357048/

git-svn-id: http://skia.googlecode.com/svn/trunk@4411 2bbb7eff-a529-9590-31e7-b0007b416f81

Update the makefile and gyp for the debugger.
Review URL: https://codereview.appspot.com/6355052

git-svn-id: http://skia.googlecode.com/svn/trunk@4410 2bbb7eff-a529-9590-31e7-b0007b416f81

Disable pipe playback on android for composeshader GM.

Review URL: https://codereview.appspot.com/6349058

git-svn-id: http://skia.googlecode.com/svn/trunk@4409 2bbb7eff-a529-9590-31e7-b0007b416f81

Allow composeshader GM to run PDF.

Review URL: https://codereview.appspot.com/6373043

git-svn-id: http://skia.googlecode.com/svn/trunk@4408 2bbb7eff-a529-9590-31e7-b0007b416f81

Create a new GM to exercise ComposeShader.

Code borrowed from samplecode/SampleShaders.cpp

Review URL: https://codereview.appspot.com/6371043

git-svn-id: http://skia.googlecode.com/svn/trunk@4407 2bbb7eff-a529-9590-31e7-b0007b416f81

Add missing gyp dependencies for debugger

git-svn-id: http://skia.googlecode.com/svn/trunk@4405 2bbb7eff-a529-9590-31e7-b0007b416f81

clt debugger
Review URL: https://codereview.appspot.com/6267043

git-svn-id: http://skia.googlecode.com/svn/trunk@4404 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix some NV path rendering issues with perspective and inverse paths
Review URL: http://codereview.appspot.com/6347050/

git-svn-id: http://skia.googlecode.com/svn/trunk@4403 2bbb7eff-a529-9590-31e7-b0007b416f81

Pseudo-reverted r4396 (SkGrPixelRef files are still in source pool but are unused)

git-svn-id: http://skia.googlecode.com/svn/trunk@4402 2bbb7eff-a529-9590-31e7-b0007b416f81

Temporarily readding SkGrTexturePixelRef.h/.cpp until WebKit is transitioned to SkGrPixelRef

git-svn-id: http://skia.googlecode.com/svn/trunk@4401 2bbb7eff-a529-9590-31e7-b0007b416f81

[PDF] Fix PDF code to not crash on SkComposeShader.

Review URL: https://codereview.appspot.com/6354053

git-svn-id: http://skia.googlecode.com/svn/trunk@4400 2bbb7eff-a529-9590-31e7-b0007b416f81

Fixed memory deletion error in instance counting mechanism

http://codereview.appspot.com/6344057/

git-svn-id: http://skia.googlecode.com/svn/trunk@4399 2bbb7eff-a529-9590-31e7-b0007b416f81

Moved SkGrTexturePixelRef to SkGrPixelRef

http://codereview.appspot.com/6344054/

git-svn-id: http://skia.googlecode.com/svn/trunk@4397 2bbb7eff-a529-9590-31e7-b0007b416f81

Collapsed SkGrTexturePixelRef and SkGrRenderTargetPixelRef into SkGrPixelRef using new GrSurface class

http://codereview.appspot.com/6359045/

git-svn-id: http://skia.googlecode.com/svn/trunk@4396 2bbb7eff-a529-9590-31e7-b0007b416f81

Disable pipe for bitmapfilters gm on Android.

Until I can investigate why the test is broken.

Review URL: https://codereview.appspot.com/6357045

git-svn-id: http://skia.googlecode.com/svn/trunk@4395 2bbb7eff-a529-9590-31e7-b0007b416f81

Rebaseline MSAA16 AA clip images. We now enable MSAA when stencil clipping if the clip element is AA.

git-svn-id: http://skia.googlecode.com/svn/trunk@4393 2bbb7eff-a529-9590-31e7-b0007b416f81

Change CT assert to RT assert

git-svn-id: http://skia.googlecode.com/svn/trunk@4391 2bbb7eff-a529-9590-31e7-b0007b416f81

Initial support for GL_NV_path_renering. Experimental, there are still some issues to resolve, set gyp variable skia_nv_path_rendering=1 or build flag GR_GL_USE_NV_PATH_RENDERING to enable.

http://codereview.appspot.com/6349049/

git-svn-id: http://skia.googlecode.com/svn/trunk@4390 2bbb7eff-a529-9590-31e7-b0007b416f81

Improve sorting of skdiff output, and make it consistent across platforms
BUG=https://code.google.com/p/skia/issues/detail?id=677
Review URL: https://codereview.appspot.com/6351045

git-svn-id: http://skia.googlecode.com/svn/trunk@4388 2bbb7eff-a529-9590-31e7-b0007b416f81

fix the build

git-svn-id: http://skia.googlecode.com/svn/trunk@4387 2bbb7eff-a529-9590-31e7-b0007b416f81

Turn off the blurs GM for Pipe playback.

It was failing on FIXED. Turn it off while we fix the real problem.

Review URL: https://codereview.appspot.com/6349051

git-svn-id: http://skia.googlecode.com/svn/trunk@4386 2bbb7eff-a529-9590-31e7-b0007b416f81

Separate target architecture type and width into separate variables.

Also add support for building mac in 64 bit mode.
Review URL: https://codereview.appspot.com/6343050

git-svn-id: http://skia.googlecode.com/svn/trunk@4385 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix SkGPipe drawing, and turn it on by default.

A recent change broke SkGPipe. Fix it, and turn on pipe drawing
in GM by default so we will catch these in the future.

We already had a bug where SkGPipeWriter had to use its Cross Process
flag to work, so for a quick fix, force the reader to use the Cross
Process flag as well. The bug to allow both cross and non cross process
is http://code.google.com/p/skia/issues/detail?id=663

Review URL: https://codereview.appspot.com/6333071

git-svn-id: http://skia.googlecode.com/svn/trunk@4384 2bbb7eff-a529-9590-31e7-b0007b416f81

add SkData::NewFromCString()

git-svn-id: http://skia.googlecode.com/svn/trunk@4383 2bbb7eff-a529-9590-31e7-b0007b416f81

fixed another clang++ compiler error

git-svn-id: http://skia.googlecode.com/svn/trunk@4382 2bbb7eff-a529-9590-31e7-b0007b416f81

Added EOF marker to address clang++ error

git-svn-id: http://skia.googlecode.com/svn/trunk@4381 2bbb7eff-a529-9590-31e7-b0007b416f81

Split GrSWMaskHelper into its own files

http://codereview.appspot.com/6350046/

git-svn-id: http://skia.googlecode.com/svn/trunk@4380 2bbb7eff-a529-9590-31e7-b0007b416f81

fixed missing "return" error

git-svn-id: http://skia.googlecode.com/svn/trunk@4379 2bbb7eff-a529-9590-31e7-b0007b416f81

Adding checksum to SkFlatData to accelerate SkPicture recording.
The checksum triggers an early exit in the mem compare use to search for duplicate flattened objects. Also, call to memcmp was replaced with 64-bit at a time comparison loop.

Review URL: http://codereview.appspot.com/6339046/
BUG=http://code.google.com/p/chromium/issues/detail?id=54079
TEST=Checksum and PictureRecord tests in bench.exe

git-svn-id: http://skia.googlecode.com/svn/trunk@4378 2bbb7eff-a529-9590-31e7-b0007b416f81

Altered SkInstCnt to print out atexit

http://codereview.appspot.com/6348043/

git-svn-id: http://skia.googlecode.com/svn/trunk@4377 2bbb7eff-a529-9590-31e7-b0007b416f81

Separated Inst counting from the SK_DEBUG #define (now SK_ENABLE_INST_COUNT)

http://codereview.appspot.com/6353047/

git-svn-id: http://skia.googlecode.com/svn/trunk@4376 2bbb7eff-a529-9590-31e7-b0007b416f81

land https://codereview.appspot.com/6349043/

git-svn-id: http://skia.googlecode.com/svn/trunk@4375 2bbb7eff-a529-9590-31e7-b0007b416f81

Added ReadPixels format-supported check to GL Caps

http://codereview.appspot.com/6336055/

git-svn-id: http://skia.googlecode.com/svn/trunk@4374 2bbb7eff-a529-9590-31e7-b0007b416f81

whitespace change to trigger Skia buildbots after updating Android build

git-svn-id: http://skia.googlecode.com/svn/trunk@4373 2bbb7eff-a529-9590-31e7-b0007b416f81

whitespace change to trigger Skia buildbots after updating Android build

git-svn-id: http://skia.googlecode.com/svn/trunk@4372 2bbb7eff-a529-9590-31e7-b0007b416f81

When building for Android, package executables in APKs (trunk)
Review URL: https://codereview.appspot.com/6307101

git-svn-id: http://skia.googlecode.com/svn/trunk@4368 2bbb7eff-a529-9590-31e7-b0007b416f81

add explicit purgeAll() so we don't get foiled by the min cache-limit
call unref instead of delete on the GrFontScaler
Review URL: https://codereview.appspot.com/6353045

git-svn-id: http://skia.googlecode.com/svn/trunk@4366 2bbb7eff-a529-9590-31e7-b0007b416f81

Enable Scratch texture detach in SW path renderer's onDrawPath (again)

http://codereview.appspot.com/6331057/

git-svn-id: http://skia.googlecode.com/svn/trunk@4365 2bbb7eff-a529-9590-31e7-b0007b416f81

Bump max flattenable from 64 to 1024.
Ran into limit on Mac.

git-svn-id: http://skia.googlecode.com/svn/trunk@4364 2bbb7eff-a529-9590-31e7-b0007b416f81

remove duplicate definition of SkAlign8()

git-svn-id: http://skia.googlecode.com/svn/trunk@4363 2bbb7eff-a529-9590-31e7-b0007b416f81

add SkAlign8 and SkIsAlign8 macros

git-svn-id: http://skia.googlecode.com/svn/trunk@4362 2bbb7eff-a529-9590-31e7-b0007b416f81

Fixed Resource counting bug in AutoScratchTexture detach mechanism

http://codereview.appspot.com/6356043/

git-svn-id: http://skia.googlecode.com/svn/trunk@4361 2bbb7eff-a529-9590-31e7-b0007b416f81

fix leak of Light object (in this private instance, the caller is transferring
ownership of the object, so the receive does not need to call ref).
Review URL: https://codereview.appspot.com/6348045

git-svn-id: http://skia.googlecode.com/svn/trunk@4360 2bbb7eff-a529-9590-31e7-b0007b416f81

Added Inst counting to SkFlattenable-derived classes

http://codereview.appspot.com/6355043/

git-svn-id: http://skia.googlecode.com/svn/trunk@4359 2bbb7eff-a529-9590-31e7-b0007b416f81

don't leak filters

git-svn-id: http://skia.googlecode.com/svn/trunk@4358 2bbb7eff-a529-9590-31e7-b0007b416f81

write my own os.path.abspath() to work around Win path-length limits
BUG=https://code.google.com/p/skia/issues/detail?id=674
Review URL: https://codereview.appspot.com/6342052

git-svn-id: http://skia.googlecode.com/svn/trunk@4357 2bbb7eff-a529-9590-31e7-b0007b416f81

Make GrInOrderDrawBuffer playback command driven

Review URL: http://codereview.appspot.com/6341052/

git-svn-id: http://skia.googlecode.com/svn/trunk@4356 2bbb7eff-a529-9590-31e7-b0007b416f81

Fixed alignment issue in Linux build

git-svn-id: http://skia.googlecode.com/svn/trunk@4355 2bbb7eff-a529-9590-31e7-b0007b416f81

Removed cached copy of the GrDebugGLInterface

http://codereview.appspot.com/6329068/

git-svn-id: http://skia.googlecode.com/svn/trunk@4354 2bbb7eff-a529-9590-31e7-b0007b416f81

plumb SkInstCnt to all subclasses of GrRefCnt

git-svn-id: http://skia.googlecode.com/svn/trunk@4353 2bbb7eff-a529-9590-31e7-b0007b416f81

remove SkInstCnt tracking of SkViewRegistry

git-svn-id: http://skia.googlecode.com/svn/trunk@4352 2bbb7eff-a529-9590-31e7-b0007b416f81

don't add SkViewRegistry to SkInstCnt

git-svn-id: http://skia.googlecode.com/svn/trunk@4351 2bbb7eff-a529-9590-31e7-b0007b416f81

remove bogus SkViewFactory reference

git-svn-id: http://skia.googlecode.com/svn/trunk@4350 2bbb7eff-a529-9590-31e7-b0007b416f81

add dump for SkWStream inst-leaks

git-svn-id: http://skia.googlecode.com/svn/trunk@4349 2bbb7eff-a529-9590-31e7-b0007b416f81

more SkInstCnt plumbing
fix some typeface leaks

git-svn-id: http://skia.googlecode.com/svn/trunk@4348 2bbb7eff-a529-9590-31e7-b0007b416f81

show # of unknown subclasses in dump

git-svn-id: http://skia.googlecode.com/svn/trunk@4347 2bbb7eff-a529-9590-31e7-b0007b416f81

In GrBatchedTextContext, use GrSafeSetNull(), for more safeness!

http://codereview.appspot.com/6326058/

git-svn-id: http://skia.googlecode.com/svn/trunk@4345 2bbb7eff-a529-9590-31e7-b0007b416f81

more SkInstCnt plumbing

git-svn-id: http://skia.googlecode.com/svn/trunk@4344 2bbb7eff-a529-9590-31e7-b0007b416f81

add more SkInstCnt plumbing

git-svn-id: http://skia.googlecode.com/svn/trunk@4343 2bbb7eff-a529-9590-31e7-b0007b416f81

Re-enable gm fontscaler for Android

This was fixed in 4.1, and we're upgrading the bots since no non-beta client
uses 4.0.
Review URL: https://codereview.appspot.com/6334058

git-svn-id: http://skia.googlecode.com/svn/trunk@4342 2bbb7eff-a529-9590-31e7-b0007b416f81

stick SkInstCnt in more classes, to help track down leaks

git-svn-id: http://skia.googlecode.com/svn/trunk@4341 2bbb7eff-a529-9590-31e7-b0007b416f81

Cleanup flattening code to properly serialize lighting image filters.
Review URL: https://codereview.appspot.com/6333070

git-svn-id: http://skia.googlecode.com/svn/trunk@4340 2bbb7eff-a529-9590-31e7-b0007b416f81

Reduce iteration count on picture_playback benchmarks to bring their cost
more in line with the rest of the benches.

http://codereview.appspot.com/6325063/

git-svn-id: http://skia.googlecode.com/svn/trunk@4339 2bbb7eff-a529-9590-31e7-b0007b416f81

Reverting r4324

git-svn-id: http://skia.googlecode.com/svn/trunk@4338 2bbb7eff-a529-9590-31e7-b0007b416f81

Baseline images for msaa complexclip tests

git-svn-id: http://skia.googlecode.com/svn/trunk@4337 2bbb7eff-a529-9590-31e7-b0007b416f81

Temporarily disable failing GM tests on Android

- fontscaler - this test fails due to a GL driver bug fixed in Jelly Bean
- drawbitmaprect - suspect this is also a driver bug
- shadertext - causes the device to reboot
Review URL: https://codereview.appspot.com/6330057

git-svn-id: http://skia.googlecode.com/svn/trunk@4336 2bbb7eff-a529-9590-31e7-b0007b416f81

Register some flattenable factories.

Now, SkComposeShaders, SkDiscreteMapper, SkCosineMapper,
Line2DPathEffect, and SkTransparentShader can be flattened
and unflattened. This makes more of our samples work when
using a pipe.

Review URL: https://codereview.appspot.com/6338064

git-svn-id: http://skia.googlecode.com/svn/trunk@4334 2bbb7eff-a529-9590-31e7-b0007b416f81

Add -o option to bench_graph_svg.py: write the output to a file rather than stdout
While I was at it, put the command-line options in alphabetical order.
BUG=https://code.google.com/p/skia/issues/detail?id=674
Review URL: https://codereview.appspot.com/6329061

git-svn-id: http://skia.googlecode.com/svn/trunk@4333 2bbb7eff-a529-9590-31e7-b0007b416f81

Disable assert blocking DEPS roll. This is a sanity-checking assert not
necessary to correct operation (yet). Will pursue a fix & reenable.

http://codereview.appspot.com/6330056/

git-svn-id: http://skia.googlecode.com/svn/trunk@4332 2bbb7eff-a529-9590-31e7-b0007b416f81

if bounds don't intersect in DIFF mode, just return src (don't return empty)
Review URL: https://codereview.appspot.com/6337052

git-svn-id: http://skia.googlecode.com/svn/trunk@4331 2bbb7eff-a529-9590-31e7-b0007b416f81

skdiff: no longer skip .pdf files by default
Review URL: https://codereview.appspot.com/6257080

git-svn-id: http://skia.googlecode.com/svn/trunk@4330 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix crash when buidling with no deferred drawing in grcontext

git-svn-id: http://skia.googlecode.com/svn/trunk@4329 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix the stencil rules to perfom an rdiff with an inverse path

Review URL: http://codereview.appspot.com/6336056/

git-svn-id: http://skia.googlecode.com/svn/trunk@4328 2bbb7eff-a529-9590-31e7-b0007b416f81

Add header needed to use the SWAP_PER_DRAW debug facility on windows.

git-svn-id: http://skia.googlecode.com/svn/trunk@4327 2bbb7eff-a529-9590-31e7-b0007b416f81

Add GrPaint::*StageEnabled() and GrDrawState::stageEnabled() functions.
These wrap the question of "is this stage of the shader enabled?" so that
we can change the semantics - previously iff there was a texture, now
if there is a texture OR a GrCustomStage, soon (post-cl 6306097) iff there
is a GrCustomStage, which at that point will hold whatever texture is
necessary.

http://codereview.appspot.com/6306104/

git-svn-id: http://skia.googlecode.com/svn/trunk@4325 2bbb7eff-a529-9590-31e7-b0007b416f81

Enable Scratch texture detach in SW path renderer's onDrawPath (take 2)

http://codereview.appspot.com/6331057/

git-svn-id: http://skia.googlecode.com/svn/trunk@4324 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix metrics test to no longer include the cpp.

The inclusion of the cpp caused linker errors when building skia
components as shared libaries for inclusion in an Android app.
Review URL: https://codereview.appspot.com/6338063

git-svn-id: http://skia.googlecode.com/svn/trunk@4323 2bbb7eff-a529-9590-31e7-b0007b416f81

Simplify vertex layout bitfields by making Position-as-texcoords the
default.

http://codereview.appspot.com/6337050/

git-svn-id: http://skia.googlecode.com/svn/trunk@4321 2bbb7eff-a529-9590-31e7-b0007b416f81

Reverting r4319

git-svn-id: http://skia.googlecode.com/svn/trunk@4320 2bbb7eff-a529-9590-31e7-b0007b416f81

Enable Scratch texture detach in SW path renderer's onDrawPath

http://codereview.appspot.com/6331057/

git-svn-id: http://skia.googlecode.com/svn/trunk@4319 2bbb7eff-a529-9590-31e7-b0007b416f81

export flags on the state of the world when a picture was serialized.
e.g. don't read/write functionptrs in that case (sizeof may be different for one)
Review URL: https://codereview.appspot.com/6331050

git-svn-id: http://skia.googlecode.com/svn/trunk@4318 2bbb7eff-a529-9590-31e7-b0007b416f81

Remove multi-byte character constants.
http://codereview.appspot.com/6305117/

git-svn-id: http://skia.googlecode.com/svn/trunk@4317 2bbb7eff-a529-9590-31e7-b0007b416f81

[android] Add an API to load customized font config for testing.

When running WebKit layout tests, we need to load some fonts from Linux to
match baselines of chromium linux. However, font config file is
hard-coded in Skia now. This change add an API to customize font config
in skia.

https://codereview.appspot.com/6296067/

git-svn-id: http://skia.googlecode.com/svn/trunk@4316 2bbb7eff-a529-9590-31e7-b0007b416f81

Fix misplaced return in GrDrawState::stageDisabled().
Original patch by guanqun.lu@gmail.com

http://codereview.appspot.com/6338058/

git-svn-id: http://skia.googlecode.com/svn/trunk@4315 2bbb7eff-a529-9590-31e7-b0007b416f81

Raster implementation of diffuse and specular lighting filters.  Externally,
the caller instantiates a light (distant, point or spot), and an
SkDiffuseLightingFilter or SkSpecularLightingImageFilter with that light.  A
Sobel edge detection filter is applied to the alpha of the incoming bitmap, and
the result is used as a height map for lighting calculations.

Review URL:  http://codereview.appspot.com/6302101/

git-svn-id: http://skia.googlecode.com/svn/trunk@4314 2bbb7eff-a529-9590-31e7-b0007b416f81

Extend texture release on GrDrawState to also handle custom stages.
Add asserts to GrContext::setPaint() to make sure we're keeping things
cleaned up.
Remove double-call of setPaint() during text context initialization.

http://codereview.appspot.com/6324046/

git-svn-id: http://skia.googlecode.com/svn/trunk@4313 2bbb7eff-a529-9590-31e7-b0007b416f81

Adding files skipped from previous patch: code.google.com/p/skia/source/detail?r=4310

Review URL: https://codereview.appspot.com/6339043/

git-svn-id: http://skia.googlecode.com/svn/trunk@4312 2bbb7eff-a529-9590-31e7-b0007b416f81

rebaseline after recent aa-clipping fix on gpu

git-svn-id: http://skia.googlecode.com/svn/trunk@4311 2bbb7eff-a529-9590-31e7-b0007b416f81

Refactoring in preparation for adding the picture benchmarking program.

Review URL: https://codereview.appspot.com/6339043/

git-svn-id: http://skia.googlecode.com/svn/trunk@4310 2bbb7eff-a529-9590-31e7-b0007b416f81

Revert #if 0ed out code accidentally committed as part of r4299.

git-svn-id: http://skia.googlecode.com/svn/trunk@4309 2bbb7eff-a529-9590-31e7-b0007b416f81

Expand header in serialized pictures
Make chunk/tag reading allow for optional and (somewhat) out-of-order chunks.
Return empty picture instead of throwing on error
Review URL: https://codereview.appspot.com/6329049

git-svn-id: http://skia.googlecode.com/svn/trunk@4308 2bbb7eff-a529-9590-31e7-b0007b416f81

change gm option from -config to --config, to match the double-dash convention of
all of its other options.
Review URL: https://codereview.appspot.com/6337044

git-svn-id: http://skia.googlecode.com/svn/trunk@4307 2bbb7eff-a529-9590-31e7-b0007b416f81

Revert r4287. Large numbers of textures created by random gradients can cause poor driver performance (at least on Mac)

BUG=670

git-svn-id: http://skia.googlecode.com/svn/trunk@4306 2bbb7eff-a529-9590-31e7-b0007b416f81

Removing redundant calls to GEN_ID_INC_EVAL in SkPaint.cpp

Review URL: http://codereview.appspot.com/6302099/

git-svn-id: http://skia.googlecode.com/svn/trunk@4305 2bbb7eff-a529-9590-31e7-b0007b416f81

Image rebaseline for r4303

git-svn-id: http://skia.googlecode.com/svn/trunk@4304 2bbb7eff-a529-9590-31e7-b0007b416f81