diff options
author | John Reck <jreck@google.com> | 2011-10-03 09:26:49 -0700 |
---|---|---|
committer | John Reck <jreck@google.com> | 2011-10-03 14:22:37 -0700 |
commit | b31c03a968a578cc43e606ede80cefcd07e8ee07 (patch) | |
tree | 9dbcf1b515b55b1e41d476632320e440f10824d6 | |
parent | 89829a24e4efac83486ad878de3438e272aeeddf (diff) | |
download | external_webkit-b31c03a968a578cc43e606ede80cefcd07e8ee07.zip external_webkit-b31c03a968a578cc43e606ede80cefcd07e8ee07.tar.gz external_webkit-b31c03a968a578cc43e606ede80cefcd07e8ee07.tar.bz2 |
HW accelate button focus rings
Bug: 5353510
Change-Id: I1bb306381864357dab4c38fd5c182fda2dd08d8c
-rw-r--r-- | Source/WebCore/platform/graphics/android/GLWebViewState.cpp | 5 | ||||
-rw-r--r-- | Source/WebCore/platform/graphics/android/GLWebViewState.h | 3 | ||||
-rw-r--r-- | 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 85fa23f..2f62223 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 6c7d31e..27e8148 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 7aae758..13e1f5f 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(); } |