Differential D10811 Diff 27967 plugins/scenes/opengl/scene_opengl.cpp

Changeset View

Standalone View

View Options

plugins/scenes/opengl/scene_opengl.cpp

Show First 20 Lines • Show All 2131 Lines • ▼ Show 20 Line(s)2131 if (effects) {
2132 effects->makeOpenGLContextCurrent(); 2132​ effects->makeOpenGLContextCurrent();
2133 DecorationShadowTextureCache::instance().unregister(this); 2133​ DecorationShadowTextureCache::instance().unregister(this);
2134 m_texture.reset(); 2134​ m_texture.reset();
2135 } 2135​ }
2136} 2136​}
2137 2137​
2138void SceneOpenGLShadow::buildQuads() 2138​void SceneOpenGLShadow::buildQuads()
2139{ 2139​{
2140 // prepare window quads ​
2141 m_shadowQuads.clear(); ​
2142 const QSizeF top(elementSize(ShadowElementTop)); 2140​ const QSizeF top(elementSize(ShadowElementTop));
2143 const QSizeF topRight(elementSize(ShadowElementTopRight)); 2141​ const QSizeF topRight(elementSize(ShadowElementTopRight));
2144 const QSizeF right(elementSize(ShadowElementRight)); 2142​ const QSizeF right(elementSize(ShadowElementRight));
2145 const QSizeF bottomRight(elementSize(ShadowElementBottomRight)); 2143​ const QSizeF bottomRight(elementSize(ShadowElementBottomRight));
2146 const QSizeF bottom(elementSize(ShadowElementBottom)); 2144​ const QSizeF bottom(elementSize(ShadowElementBottom));
2147 const QSizeF bottomLeft(elementSize(ShadowElementBottomLeft)); 2145​ const QSizeF bottomLeft(elementSize(ShadowElementBottomLeft));
2148 const QSizeF left(elementSize(ShadowElementLeft)); 2146​ const QSizeF left(elementSize(ShadowElementLeft));
2149 const QSizeF topLeft(elementSize(ShadowElementTopLeft)); 2147​ const QSizeF topLeft(elementSize(ShadowElementTopLeft));
2148​
2149​ // TODO: Do we need this? Turning shadows off when a window is too small
2150​ // is not the best KWin could do.
2151​ // KDecoration or KWin could mask out inner shadow rect.
graesslinUnsubmitted
Not Done

To explain why this was added: in KPart based applications it could happen that a menu is empty. In that case all we got was the shadow. IIRC kdevelop was able to trigger this.

graesslin: To explain why this was added: in KPart based applications it could happen that a menu is empty.
zzagAuthorUnsubmitted
Not Done

Is there a test plan to see what it looks like without this if statement?

zzag: Is there a test plan to see what it looks like without this if statement?
graesslinUnsubmitted
Not Done

I hoped to find a bug report with the issue, but the commit which introduced that if statement doesn't have one linked: b837a3fca184fce0c7da317b2866d409892e71eb

graesslin: I hoped to find a bug report with the issue, but the commit which introduced that if statement…
2150 if ((left.width() - leftOffset() > topLevel()->width()) || 2152​ if ((left.width() - leftOffset() > topLevel()->width()) ||
2151 (right.width() - rightOffset() > topLevel()->width()) || 2153​ (right.width() - rightOffset() > topLevel()->width()) ||
2152 (top.height() - topOffset() > topLevel()->height()) || 2154​ (top.height() - topOffset() > topLevel()->height()) ||
2153 (bottom.height() - bottomOffset() > topLevel()->height())) { 2155​ (bottom.height() - bottomOffset() > topLevel()->height())) {
2154 // if our shadow is bigger than the window, we don't render the shadow 2156​ // if our shadow is bigger than the window, we don't render the shadow
2155 setShadowRegion(QRegion()); 2157​ setShadowRegion(QRegion());
2156 return; 2158​ return;
2157 } 2159​ }
2158 2160​
2159 const QRectF outerRect(QPointF(-leftOffset(), -topOffset()), 2161​ const QRectF outerRect(QPointF(-leftOffset(), -topOffset()),
2160 QPointF(topLevel()->width() + rightOffset(), topLevel()->height() + bottomOffset())); 2162​ QPointF(topLevel()->width() + rightOffset(),
2163​ topLevel()->height() + bottomOffset()));
2161 2164​
2162 const int width = std::max({topLeft.width(), left.width(), bottomLeft.width()}) + 2165​ const int width = std::max({topLeft.width(), left.width(), bottomLeft.width()})
2163 std::max(top.width(), bottom.width()) + 2166​ + std::max(top.width(), bottom.width())
2164 std::max({topRight.width(), right.width(), bottomRight.width()}); 2167​ + std::max({topRight.width(), right.width(), bottomRight.width()});
2165 const int height = std::max({topLeft.height(), top.height(), topRight.height()}) + 2168​ const int height = std::max({topLeft.height(), top.height(), topRight.height()})
2166 std::max(left.height(), right.height()) + 2169​ + std::max(left.height(), right.height())
2167 std::max({bottomLeft.height(), bottom.height(), bottomRight.height()}); 2170​ + std::max({bottomLeft.height(), bottom.height(), bottomRight.height()});
2171​
2172​ QRectF topLeftRect(outerRect.topLeft(), topLeft);
2173​ QRectF topRightRect(outerRect.topRight() - QPointF(topRight.width(), 0), topRight);
2174​ QRectF bottomRightRect(
2175​ outerRect.bottomRight() - QPointF(bottomRight.width(), bottomRight.height()),
2176​ bottomRight);
2177​ QRectF bottomLeftRect(outerRect.bottomLeft() - QPointF(0, bottomLeft.height()), bottomLeft);
2178​
2179​ // Re-distribute the corner tiles so no one of them is overlapping with others.
2180​ // By doing this, we assume that shadow's corner tiles are symmetric
2181​ // and it is OK to not draw top/right/bottom/left tile between corners.
2182​ // For example, let's say top-left and top-right tiles are overlapping.
2183​ // In that case, the right side of the top-left tile will be shifted to left,
2184​ // the left side of the top-right tile will shifted to right, and the top
2185​ // tile won't be rendered.
2186​ bool drawTop = true;
2187​ if (topLeftRect.right() > topRightRect.left()) {
2188​ const float halfOverlap = qAbs(topLeftRect.right() - topRightRect.left()) / 2;
2189​ topLeftRect.setRight(topLeftRect.right() - halfOverlap);
2190​ topRightRect.setLeft(topRightRect.left() + halfOverlap);
2191​ drawTop = false;
2192​ }
2193​
2194​ bool drawRight = true;
2195​ if (topRightRect.bottom() > bottomRightRect.top()) {
2196​ const float halfOverlap = qAbs(topRightRect.bottom() - bottomRightRect.top()) / 2;
2197​ topRightRect.setBottom(topRightRect.bottom() - halfOverlap);
2198​ bottomRightRect.setTop(bottomRightRect.top() + halfOverlap);
2199​ drawRight = false;
2200​ }
2201​
2202​ bool drawBottom = true;
2203​ if (bottomLeftRect.right() > bottomRightRect.left()) {
2204​ const float halfOverlap = qAbs(bottomLeftRect.right() - bottomRightRect.left()) / 2;
2205​ bottomLeftRect.setRight(bottomLeftRect.right() - halfOverlap);
2206​ bottomRightRect.setLeft(bottomRightRect.left() + halfOverlap);
2207​ drawBottom = false;
2208​ }
2209​
2210​ bool drawLeft = true;
2211​ if (topLeftRect.bottom() > bottomLeftRect.top()) {
2212​ const float halfOverlap = qAbs(topLeftRect.bottom() - bottomLeftRect.top()) / 2;
2213​ topLeftRect.setBottom(topLeftRect.bottom() - halfOverlap);
2214​ bottomLeftRect.setTop(bottomLeftRect.top() + halfOverlap);
2215​ drawLeft = false;
2216​ }
2217​
2218​ qreal tx1 = 0.0,
2219​ tx2 = 0.0,
2220​ ty1 = 0.0,
2221​ ty2 = 0.0;
2168 2222​
2169 qreal tx1(0.0), tx2(0.0), ty1(0.0), ty2(0.0); 2223​ m_shadowQuads.clear();
2170 2224​
2171 tx2 = topLeft.width()/width; 2225​ tx1 = 0.0;
2172 ty2 = topLeft.height()/height; 2226​ ty1 = 0.0;
2227​ tx2 = topLeftRect.width() / width;
2228​ ty2 = topLeftRect.height() / height;
2173 WindowQuad topLeftQuad(WindowQuadShadow); 2229​ WindowQuad topLeftQuad(WindowQuadShadow);
2174 topLeftQuad[ 0 ] = WindowVertex(outerRect.x(), outerRect.y(), tx1, ty1); 2230​ topLeftQuad[0] = WindowVertex(topLeftRect.left(), topLeftRect.top(), tx1, ty1);
2175 topLeftQuad[ 1 ] = WindowVertex(outerRect.x() + topLeft.width(), outerRect.y(), tx2, ty1); 2231​ topLeftQuad[1] = WindowVertex(topLeftRect.right(), topLeftRect.top(), tx2, ty1);
2176 topLeftQuad[ 2 ] = WindowVertex(outerRect.x() + topLeft.width(), outerRect.y() + topLeft.height(), tx2, ty2); 2232​ topLeftQuad[2] = WindowVertex(topLeftRect.right(), topLeftRect.bottom(), tx2, ty2);
2177 topLeftQuad[ 3 ] = WindowVertex(outerRect.x(), outerRect.y() + topLeft.height(), tx1, ty2); 2233​ topLeftQuad[3] = WindowVertex(topLeftRect.left(), topLeftRect.bottom(), tx1, ty2);
2178 m_shadowQuads.append(topLeftQuad); 2234​ m_shadowQuads.append(topLeftQuad);
2179 2235​
2180 tx1 = tx2; 2236​ tx1 = 1.0 - topRightRect.width() / width;
2181 tx2 = (topLeft.width() + top.width())/width; 2237​ ty1 = 0.0;
2182 ty2 = top.height()/height; ​
2183 WindowQuad topQuad(WindowQuadShadow); ​
2184 topQuad[ 0 ] = WindowVertex(outerRect.x() + topLeft.width(), outerRect.y(), tx1, ty1); ​
2185 topQuad[ 1 ] = WindowVertex(outerRect.right() - topRight.width(), outerRect.y(), tx2, ty1); ​
2186 topQuad[ 2 ] = WindowVertex(outerRect.right() - topRight.width(), outerRect.y() + top.height(),tx2, ty2); ​
2187 topQuad[ 3 ] = WindowVertex(outerRect.x() + topLeft.width(), outerRect.y() + top.height(), tx1, ty2); ​
2188 m_shadowQuads.append(topQuad); ​
2189 ​
2190 tx1 = tx2; ​
2191 tx2 = 1.0; 2238​ tx2 = 1.0;
2192 ty2 = topRight.height()/height; 2239​ ty2 = topRightRect.height() / height;
2193 WindowQuad topRightQuad(WindowQuadShadow); 2240​ WindowQuad topRightQuad(WindowQuadShadow);
2194 topRightQuad[ 0 ] = WindowVertex(outerRect.right() - topRight.width(), outerRect.y(), tx1, ty1); 2241​ topRightQuad[0] = WindowVertex(topRightRect.left(), topRightRect.top(), tx1, ty1);
2195 topRightQuad[ 1 ] = WindowVertex(outerRect.right(), outerRect.y(), tx2, ty1); 2242​ topRightQuad[1] = WindowVertex(topRightRect.right(), topRightRect.top(), tx2, ty1);
2196 topRightQuad[ 2 ] = WindowVertex(outerRect.right(), outerRect.y() + topRight.height(), tx2, ty2); 2243​ topRightQuad[2] = WindowVertex(topRightRect.right(), topRightRect.bottom(), tx2, ty2);
2197 topRightQuad[ 3 ] = WindowVertex(outerRect.right() - topRight.width(), outerRect.y() + topRight.height(), tx1, ty2); 2244​ topRightQuad[3] = WindowVertex(topRightRect.left(), topRightRect.bottom(), tx1, ty2);
2198 m_shadowQuads.append(topRightQuad); 2245​ m_shadowQuads.append(topRightQuad);
2199 2246​
2200 tx1 = (width - right.width())/width; 2247​ tx1 = 1.0 - bottomRightRect.width() / width;
2201 ty1 = topRight.height()/height; 2248​ tx2 = 1.0;
2202 ty2 = (topRight.height() + right.height())/height; 2249​ ty1 = 1.0 - bottomRightRect.height() / height;
2203 WindowQuad rightQuad(WindowQuadShadow); ​
2204 rightQuad[ 0 ] = WindowVertex(outerRect.right() - right.width(), outerRect.y() + topRight.height(), tx1, ty1); ​
2205 rightQuad[ 1 ] = WindowVertex(outerRect.right(), outerRect.y() + topRight.height(), tx2, ty1); ​
2206 rightQuad[ 2 ] = WindowVertex(outerRect.right(), outerRect.bottom() - bottomRight.height(), tx2, ty2); ​
2207 rightQuad[ 3 ] = WindowVertex(outerRect.right() - right.width(), outerRect.bottom() - bottomRight.height(), tx1, ty2); ​
2208 m_shadowQuads.append(rightQuad); ​
2209 ​
2210 tx1 = (width - bottomRight.width())/width; ​
2211 ty1 = ty2; ​
2212 ty2 = 1.0; 2250​ ty2 = 1.0;
2213 WindowQuad bottomRightQuad(WindowQuadShadow); 2251​ WindowQuad bottomRightQuad(WindowQuadShadow);
2214 bottomRightQuad[ 0 ] = WindowVertex(outerRect.right() - bottomRight.width(), outerRect.bottom() - bottomRight.height(), tx1, ty1); 2252​ bottomRightQuad[0] = WindowVertex(bottomRightRect.left(), bottomRightRect.top(), tx1, ty1);
2215 bottomRightQuad[ 1 ] = WindowVertex(outerRect.right(), outerRect.bottom() - bottomRight.height(), tx2, ty1); 2253​ bottomRightQuad[1] = WindowVertex(bottomRightRect.right(), bottomRightRect.top(), tx2, ty1);
2216 bottomRightQuad[ 2 ] = WindowVertex(outerRect.right(), outerRect.bottom(), tx2, ty2); 2254​ bottomRightQuad[2] = WindowVertex(bottomRightRect.right(), bottomRightRect.bottom(), tx2, ty2);
2217 bottomRightQuad[ 3 ] = WindowVertex(outerRect.right() - bottomRight.width(), outerRect.bottom(), tx1, ty2); 2255​ bottomRightQuad[3] = WindowVertex(bottomRightRect.left(), bottomRightRect.bottom(), tx1, ty2);
2218 m_shadowQuads.append(bottomRightQuad); 2256​ m_shadowQuads.append(bottomRightQuad);
2219 2257​
2220 tx2 = tx1; 2258​ tx1 = 0.0;
2259​ tx2 = bottomLeftRect.width() / width;
2260​ ty1 = 1.0 - bottomLeftRect.height() / height;
2261​ ty2 = 1.0;
2262​ WindowQuad bottomLeftQuad(WindowQuadShadow);
2263​ bottomLeftQuad[0] = WindowVertex(bottomLeftRect.left(), bottomLeftRect.top(), tx1, ty1);
2264​ bottomLeftQuad[1] = WindowVertex(bottomLeftRect.right(), bottomLeftRect.top(), tx2, ty1);
2265​ bottomLeftQuad[2] = WindowVertex(bottomLeftRect.right(), bottomLeftRect.bottom(), tx2, ty2);
2266​ bottomLeftQuad[3] = WindowVertex(bottomLeftRect.left(), bottomLeftRect.bottom(), tx1, ty2);
2267​ m_shadowQuads.append(bottomLeftQuad);
2268​
2269​ if (drawTop) {
2270​ QRectF topRect(
2271​ topLeftRect.topRight(),
2272​ topRightRect.topLeft() + QPointF(0, top.height()));
2273​ tx1 = topLeft.width() / width;
2274​ ty1 = 0.0;
2275​ tx2 = 1.0 - topRight.width() / width;
2276​ ty2 = top.height() / height;
2277​ WindowQuad topQuad(WindowQuadShadow);
2278​ topQuad[0] = WindowVertex(topRect.left(), topRect.top(), tx1, ty1);
2279​ topQuad[1] = WindowVertex(topRect.right(), topRect.top(), tx2, ty1);
2280​ topQuad[2] = WindowVertex(topRect.right(), topRect.bottom(), tx2, ty2);
2281​ topQuad[3] = WindowVertex(topRect.left(), topRect.bottom(), tx1, ty2);
2282​ m_shadowQuads.append(topQuad);
2283​ }
2284​
2285​ if (drawRight) {
2286​ QRectF rightRect(
2287​ topRightRect.bottomRight() - QPointF(right.width(), 0),
2288​ bottomRightRect.topRight());
2289​ tx1 = 1.0 - right.width() / width;
2290​ ty1 = topRight.height() / height;
2291​ tx2 = 1.0;
2292​ ty2 = 1.0 - bottomRight.height() / height;
2293​ WindowQuad rightQuad(WindowQuadShadow);
2294​ rightQuad[0] = WindowVertex(rightRect.left(), rightRect.top(), tx1, ty1);
2295​ rightQuad[1] = WindowVertex(rightRect.right(), rightRect.top(), tx2, ty1);
2296​ rightQuad[2] = WindowVertex(rightRect.right(), rightRect.bottom(), tx2, ty2);
2297​ rightQuad[3] = WindowVertex(rightRect.left(), rightRect.bottom(), tx1, ty2);
2298​ m_shadowQuads.append(rightQuad);
2299​ }
2300​
2301​ if (drawBottom) {
2302​ QRectF bottomRect(
2303​ bottomLeftRect.bottomRight() - QPointF(0, bottom.height()),
2304​ bottomRightRect.bottomLeft());
2221 tx1 = bottomLeft.width()/width; 2305​ tx1 = bottomLeft.width() / width;
2222 ty1 = (height - bottom.height())/height; 2306​ ty1 = 1.0 - bottom.height() / height;
2307​ tx2 = 1.0 - bottomRight.width() / width;
2308​ ty2 = 1.0;
2223 WindowQuad bottomQuad(WindowQuadShadow); 2309​ WindowQuad bottomQuad(WindowQuadShadow);
2224 bottomQuad[ 0 ] = WindowVertex(outerRect.x() + bottomLeft.width(), outerRect.bottom() - bottom.height(), tx1, ty1); 2310​ bottomQuad[0] = WindowVertex(bottomRect.left(), bottomRect.top(), tx1, ty1);
2225 bottomQuad[ 1 ] = WindowVertex(outerRect.right() - bottomRight.width(), outerRect.bottom() - bottom.height(), tx2, ty1); 2311​ bottomQuad[1] = WindowVertex(bottomRect.right(), bottomRect.top(), tx2, ty1);
2226 bottomQuad[ 2 ] = WindowVertex(outerRect.right() - bottomRight.width(), outerRect.bottom(), tx2, ty2); 2312​ bottomQuad[2] = WindowVertex(bottomRect.right(), bottomRect.bottom(), tx2, ty2);
2227 bottomQuad[ 3 ] = WindowVertex(outerRect.x() + bottomLeft.width(), outerRect.bottom(), tx1, ty2); 2313​ bottomQuad[3] = WindowVertex(bottomRect.left(), bottomRect.bottom(), tx1, ty2);
2228 m_shadowQuads.append(bottomQuad); 2314​ m_shadowQuads.append(bottomQuad);
2315​ }
2229 2316​
2317​ if (drawLeft) {
2318​ QRectF leftRect(
2319​ topLeftRect.bottomLeft(),
2320​ bottomLeftRect.topLeft() + QPointF(left.width(), 0));
2230 tx1 = 0.0; 2321​ tx1 = 0.0;
2231 tx2 = bottomLeft.width()/width; ​
2232 ty1 = (height - bottomLeft.height())/height; ​
2233 WindowQuad bottomLeftQuad(WindowQuadShadow); ​
2234 bottomLeftQuad[ 0 ] = WindowVertex(outerRect.x(), outerRect.bottom() - bottomLeft.height(), tx1, ty1); ​
2235 bottomLeftQuad[ 1 ] = WindowVertex(outerRect.x() + bottomLeft.width(), outerRect.bottom() - bottomLeft.height(), tx2, ty1); ​
2236 bottomLeftQuad[ 2 ] = WindowVertex(outerRect.x() + bottomLeft.width(), outerRect.bottom(), tx2, ty2); ​
2237 bottomLeftQuad[ 3 ] = WindowVertex(outerRect.x(), outerRect.bottom(), tx1, ty2); ​
2238 m_shadowQuads.append(bottomLeftQuad); ​
2239 ​
2240 tx2 = left.width()/width; ​
2241 ty2 = ty1; ​
2242 ty1 = topLeft.height()/height; 2322​ ty1 = topLeft.height() / height;
2323​ tx2 = left.width() / width;
2324​ ty2 = 1.0 - bottomRight.height() / height;
2243 WindowQuad leftQuad(WindowQuadShadow); 2325​ WindowQuad leftQuad(WindowQuadShadow);
2244 leftQuad[ 0 ] = WindowVertex(outerRect.x(), outerRect.y() + topLeft.height(), tx1, ty1); 2326​ leftQuad[0] = WindowVertex(leftRect.left(), leftRect.top(), tx1, ty1);
2245 leftQuad[ 1 ] = WindowVertex(outerRect.x() + left.width(), outerRect.y() + topLeft.height(), tx2, ty1); 2327​ leftQuad[1] = WindowVertex(leftRect.right(), leftRect.top(), tx2, ty1);
2246 leftQuad[ 2 ] = WindowVertex(outerRect.x() + left.width(), outerRect.bottom() - bottomLeft.height(), tx2, ty2); 2328​ leftQuad[2] = WindowVertex(leftRect.right(), leftRect.bottom(), tx2, ty2);
2247 leftQuad[ 3 ] = WindowVertex(outerRect.x(), outerRect.bottom() - bottomLeft.height(), tx1, ty2); 2329​ leftQuad[3] = WindowVertex(leftRect.left(), leftRect.bottom(), tx1, ty2);
2248 m_shadowQuads.append(leftQuad); 2330​ m_shadowQuads.append(leftQuad);
2249} 2331​ }
2332​}
2250 2333​
2251bool SceneOpenGLShadow::prepareBackend() 2334​bool SceneOpenGLShadow::prepareBackend()
2252{ 2335​{
2253 if (hasDecorationShadow()) { 2336​ if (hasDecorationShadow()) {
2254 // simplifies a lot by going directly to 2337​ // simplifies a lot by going directly to
2255 effects->makeOpenGLContextCurrent(); 2338​ effects->makeOpenGLContextCurrent();
2256 m_texture = DecorationShadowTextureCache::instance().getTexture(this); 2339​ m_texture = DecorationShadowTextureCache::instance().getTexture(this);
2257 2340​
▲ Show 20 Lines • Show All 209 Lines • Show Last 20 Lines