android: Fix black screen at startng frame when using ContentViewRenderView

content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java

 // Copyright (c) 2012 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.
 
 package org.chromium.content.browser;
 
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
 import android.graphics.Color;
@@ skipping 42 matching lines @@
      * Constructs a new ContentViewRenderView that should be can to a view hierarchy.
      * Native code should add/remove the layers to be rendered through the ContentViewLayerRenderer.
      * @param context The context used to create this.
      */
     public ContentViewRenderView(Context context, WindowAndroid rootWindow) {
         super(context);
         assert rootWindow != null;
         mNativeContentViewRenderView = nativeInit(rootWindow.getNativePointer());
         assert mNativeContentViewRenderView != 0;
 
         setBackgroundColor(Color.WHITE);

[no sievers, 2013/11/27 21:22:21]

I think this can go.

[powei, 2013/12/02 20:03:38]

Done.

 
         mSurfaceView = createSurfaceView(getContext());
+        mSurfaceView.setBackgroundColor(Color.WHITE);
         mSurfaceCallback = new SurfaceHolder.Callback() {
             @Override
             public void surfaceChanged(SurfaceHolder holder, int format, int width, int height) {
                 assert mNativeContentViewRenderView != 0;
                 nativeSurfaceSetSize(mNativeContentViewRenderView, width, height);
                 if (mCurrentContentView != null) {
                     mCurrentContentView.getContentViewCore().onPhysicalBackingSizeChanged(
                             width, height);
                 }
             }
@@ skipping 201 matching lines @@
         }
 
         boolean didDraw = nativeComposite(mNativeContentViewRenderView);
         if (didDraw) {
             mPendingSwapBuffers++;
             if (getBackground() != null) {
                 post(new Runnable() {
                     @Override
                     public void run() {
                         setBackgroundResource(0);
+                        mSurfaceView.setBackground(null);

[no sievers, 2013/11/27 21:22:21]

This is only available in JB and up.

Does mSurfaceView.setBackgroundResource(0) do the trick.

And then we can remove 287 also.

[powei, 2013/12/02 20:03:38]

Done.

                     }
                 });
             }
         }
     }
 
     private native long nativeInit(long rootWindowNativePointer);
     private native void nativeDestroy(long nativeContentViewRenderView);
     private native void nativeSetCurrentContentView(long nativeContentViewRenderView,
             long nativeContentView);
     private native void nativeSurfaceCreated(long nativeContentViewRenderView, Surface surface);
     private native void nativeSurfaceDestroyed(long nativeContentViewRenderView);
     private native void nativeSurfaceSetSize(long nativeContentViewRenderView,
             int width, int height);
     private native boolean nativeComposite(long nativeContentViewRenderView);
     private native boolean nativeCompositeToBitmap(long nativeContentViewRenderView, Bitmap bitmap);
 }