Fix for Bug 2486292, transparent backgrounds not transparent in Froyo.

Recreating the transparent and color of old window when creating a new one, and setting transparent if all the colors are 0.
Updated patch to fix comments from Grace.

Change-Id: I934401630fd7041e7e4ae1cbeeaa198271025c87

2 files changed, 8 insertions, 0 deletions

diff --git a/WebKit/android/jni/WebFrameView.h b/WebKit/android/jni/WebFrameView.h
index d83def1..823f2b4 100644
--- a/WebKit/android/jni/WebFrameView.h
+++ b/WebKit/android/jni/WebFrameView.h
@@ -49,6 +49,10 @@ namespace android {
 
         void setView(WebCore::FrameView* frameView);
 
+        WebCore::FrameView* view() const {
+            return mFrameView;
+        }
+
         virtual bool forFrameView() const { return true; }
 
     private:
diff --git a/WebKit/android/jni/WebViewCore.cpp b/WebKit/android/jni/WebViewCore.cpp
index 6e0ef43..8556dd8 100644
--- a/WebKit/android/jni/WebViewCore.cpp
+++ b/WebKit/android/jni/WebViewCore.cpp
@@ -2365,6 +2365,10 @@ void WebViewCore::setBackgroundColor(SkColor c)
     WebCore::Color bcolor((int)SkColorGetR(c), (int)SkColorGetG(c),
                           (int)SkColorGetB(c), (int)SkColorGetA(c));
     view->setBaseBackgroundColor(bcolor);
+
+    // Background color of 0 indicates we want a transparent background
+    if (c == 0)
+        view->setTransparent(true);
 }
 
 jclass WebViewCore::getPluginClass(const WebCore::String& libName, const char* className)