From b31c03a968a578cc43e606ede80cefcd07e8ee07 Mon Sep 17 00:00:00 2001
From: John Reck <jreck@google.com>
Date: Mon, 3 Oct 2011 09:26:49 -0700
Subject: [PATCH 1/1] HW accelate button focus rings

 Bug: 5353510

Change-Id: I1bb306381864357dab4c38fd5c182fda2dd08d8c
---
 .../platform/graphics/android/GLWebViewState.cpp   |  5 ++++-
 .../platform/graphics/android/GLWebViewState.h     |  3 ++-
 Source/WebKit/android/nav/WebView.cpp              | 22 +++++++++-------------
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/Source/WebCore/platform/graphics/android/GLWebViewState.cpp b/Source/WebCore/platform/graphics/android/GLWebViewState.cpp
index 85fa23f4b..2f62223f9 100644
--- a/Source/WebCore/platform/graphics/android/GLWebViewState.cpp
+++ b/Source/WebCore/platform/graphics/android/GLWebViewState.cpp
@@ -183,7 +183,7 @@ void GLWebViewState::setBaseLayer(BaseLayerAndroid* layer, const SkRegion& inval
     TilesManager::instance()->setShowVisualIndicator(showVisualIndicator);
 }
 
-void GLWebViewState::setRings(Vector<IntRect>& rings, bool isPressed)
+void GLWebViewState::setRings(Vector<IntRect>& rings, bool isPressed, bool isButton)
 {
     android::Mutex::Autolock lock(m_baseLayerLock);
     m_displayRings = true;
@@ -195,6 +195,7 @@ void GLWebViewState::setRings(Vector<IntRect>& rings, bool isPressed)
             m_rings.op(rings.at(i), SkRegion::kUnion_Op);
     }
     m_ringsIsPressed = isPressed;
+    m_ringsIsButton = isButton;
 }
 
 void GLWebViewState::invalRegion(const SkRegion& region)
@@ -294,6 +295,8 @@ void GLWebViewState::paintExtras()
                 rgnIter.next();
             }
         }
+        if (m_ringsIsButton && m_ringsIsPressed)
+            return;
         SkPath path;
         if (!m_rings.getBoundaryPath(&path))
             return;
diff --git a/Source/WebCore/platform/graphics/android/GLWebViewState.h b/Source/WebCore/platform/graphics/android/GLWebViewState.h
index 6c7d31ebc..27e81487b 100644
--- a/Source/WebCore/platform/graphics/android/GLWebViewState.h
+++ b/Source/WebCore/platform/graphics/android/GLWebViewState.h
@@ -177,7 +177,7 @@ public:
     void setExtra(BaseLayerAndroid*, SkPicture&, const IntRect&, bool allowSame);
     void paintExtras();
 
-    void setRings(Vector<IntRect>& rings, bool isPressed);
+    void setRings(Vector<IntRect>& rings, bool isPressed, bool isButton);
     void resetRings();
     void drawFocusRing(SkRect& rect);
 
@@ -271,6 +271,7 @@ private:
     bool m_displayRings;
     SkRegion m_rings;
     bool m_ringsIsPressed;
+    bool m_ringsIsButton;
     int m_focusRingTexture;
 
     bool m_isScrolling;
diff --git a/Source/WebKit/android/nav/WebView.cpp b/Source/WebKit/android/nav/WebView.cpp
index 7aae7583a..13e1f5f1e 100644
--- a/Source/WebKit/android/nav/WebView.cpp
+++ b/Source/WebKit/android/nav/WebView.cpp
@@ -83,6 +83,8 @@
 #define TRIM_MEMORY_BACKGROUND 40
 // Moderate free (clear cached tiles, keep visible ones)
 #define TRIM_MEMORY_UI_HIDDEN 20
+// Duration to show the pressed cursor ring
+#define PRESSED_STATE_DURATION 400
 
 namespace android {
 
@@ -282,6 +284,7 @@ void hideCursor()
         return;
     DBG_NAV_LOG("");
     hideCursor(root);
+    viewInvalidate();
 }
 
 void hideCursor(CachedRoot* root)
@@ -289,7 +292,6 @@ void hideCursor(CachedRoot* root)
     DBG_NAV_LOG("inner");
     m_viewImpl->m_hasCursorBounds = false;
     root->hideCursor();
-    viewInvalidate();
 }
 
 #if DUMP_NAV_CACHE
@@ -556,7 +558,7 @@ bool drawGL(WebCore::IntRect& viewRect, WebCore::IntRect* invalRect, WebCore::In
             break;
         case DrawExtrasCursorRing:
             if (drawCursorPreamble(root) && m_ring.setup()) {
-                if (!m_ring.m_isButton)
+                if (m_ring.m_isPressed || m_ringAnimationEnd == UINT_MAX)
                     extra = &m_ring;
                 drawCursorPostamble();
             }
@@ -574,7 +576,7 @@ bool drawGL(WebCore::IntRect& viewRect, WebCore::IntRect* invalRect, WebCore::In
     if (extra) {
         if (extra == &m_ring) {
             WTF::Vector<IntRect> rings;
-            if (root == m_ring.m_frame)
+            if (!m_ring.m_isButton && root == m_ring.m_frame)
                 rings = m_ring.rings();
             else {
                 // TODO: Fix the navcache to work with layers correctly
@@ -584,11 +586,12 @@ bool drawGL(WebCore::IntRect& viewRect, WebCore::IntRect* invalRect, WebCore::In
                 for (size_t i = 0; i < m_ring.m_node->rings().size(); i++) {
                     IntRect rect = m_ring.m_node->rings().at(i);
                     rect = m_ring.m_frame->adjustBounds(m_ring.m_node, rect);
-                    rect.inflate(4);
+                    if (!m_ring.m_isButton)
+                        rect.inflate(4);
                     rings.append(rect);
                 }
             }
-            m_glWebViewState->setRings(rings, m_ring.m_isPressed);
+            m_glWebViewState->setRings(rings, m_ring.m_isPressed, m_ring.m_isButton);
             extra = 0;
         } else {
             LayerAndroid mainPicture(m_navPictureUI);
@@ -598,13 +601,6 @@ bool drawGL(WebCore::IntRect& viewRect, WebCore::IntRect* invalRect, WebCore::In
             extra->draw(canvas, &mainPicture, &rect);
             picture.endRecording();
         }
-    } else if (root && extras == DrawExtrasCursorRing && m_ring.m_isButton) {
-        const CachedFrame* cachedFrame;
-        const CachedNode* cachedCursor = root->currentCursor(&cachedFrame);
-        if (cachedCursor) {
-            rect = cachedCursor->bounds(cachedFrame);
-            allowSame = true;
-        }
     }
     m_glWebViewState->setExtra(m_baseLayer, picture, rect, allowSame);
 
@@ -1246,7 +1242,7 @@ void setFindIsEmpty()
 void showCursorTimed()
 {
     DBG_NAV_LOG("");
-    m_ringAnimationEnd = SkTime::GetMSecs() + 500;
+    m_ringAnimationEnd = SkTime::GetMSecs() + PRESSED_STATE_DURATION;
     viewInvalidate();
 }
 
-- 
2.11.0