summaryrefslogtreecommitdiffstats
path: root/WebCore
Commit message (Collapse)AuthorAgeFilesLines
* am 8527f14d: am 92c3acf6: Fix error where mediaLayer is derefed on UI thread ↵Derek Sollenberger2011-03-255-9/+30
|\ | | | | | | | | | | | | prior to being derefed in Webkit. * commit '8527f14d378e1fb4ba315d791f3c8c7b776bff4c': Fix error where mediaLayer is derefed on UI thread prior to being derefed in Webkit.
| * Fix error where mediaLayer is derefed on UI thread prior to being derefed in ↵Derek Sollenberger2011-03-255-9/+30
| | | | | | | | | | | | | | Webkit. bug: 4171470 Change-Id: I6afe957b2eccae249afa46baede822c34f8f1a7f
* | am 70e1ff0b: am 557855a0: Fix regression blocking the page cycler The bug ↵Nicolas Roard2011-03-241-1/+1
|\ \ | |/ | | | | | | | | | | caused a continuous redraw if you had a webview without content; the Activity containing the webview would thus not be marked as idle. The pagecycler is launching such an activity and wait until * commit '70e1ff0b3913af6a122390bbe4768c2c0fea045d': Fix regression blocking the page cycler The bug caused a continuous redraw if you had a webview without content; the Activity containing the webview would thus not be marked as idle. The pagecycler is launching such an activity and wait until it's idle to determinate that it's usable to loade urls...
| * Fix regression blocking the page cyclerNicolas Roard2011-03-241-1/+1
| | | | | | | | | | | | | | | | | | | | The bug caused a continuous redraw if you had a webview without content; the Activity containing the webview would thus not be marked as idle. The pagecycler is launching such an activity and wait until it's idle to determinate that it's usable to loade urls... bug:4172381 Change-Id: I6c838c5f6757186d01e8e476a8fc5101ea00ed15
* | Merge "Add defensive code for crash in FontPlatformData"Russell Brenner2011-03-241-3/+13
|\ \
| * | Add defensive code for crash in FontPlatformDataRussell Brenner2011-03-231-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 4163318 In FontPlatformData, both 0 and -1 are used as special values for mTypeface. Skia has checks to handle null typefaces, but -1 is a special value (hashTableDeletedFontValue) unique to FontPlatformData. The constructors and operators appear to have the necessary guards, but setupPaint(), uniqueID(), and isFixedPitch() needed some touchup to better handle 0 or -1. There's still the question of whether or not it's appropriate to be calling any of those functions with the typeface set to 0 or -1, so these changes may simply be deflecting the underlying problem, but these changes should guard against the immediate failure. Change-Id: Ib68a64ba6d4eeffbd502f29b68074e38c511a746
* | | am 284cf17e: am 909e9c7e: Improve performances for the base surface renderingNicolas Roard2011-03-233-26/+30
|\ \ \ | | |/ | |/| | | | | | | * commit '284cf17e2bcea995294e7ecdca145593523c3edc': Improve performances for the base surface rendering
| * | Improve performances for the base surface renderingNicolas Roard2011-03-233-26/+30
| | | | | | | | | | | | | | | | | | | | | | | | - don't queue large number of tiles when scrolling/zooming - while scrolling/zooming, schedule the tiles generation at the front of the queue bug:4136077 Change-Id: I652464d01f3000e88e42fad1e98402436a85dce4
* | | am 3c1bcdb4: am 0cdc1296: Fix the repainting problems with overflow-scroll ↵Nicolas Roard2011-03-231-0/+1
|\ \ \ | |/ / | | | | | | | | | | | | | | | div elements * commit '3c1bcdb44e7bd2272e7ba02da9e1b7422da7fb17': Fix the repainting problems with overflow-scroll div elements
| * | Fix the repainting problems with overflow-scroll div elementsNicolas Roard2011-03-231-0/+1
| | | | | | | | | | | | | | | bug:4163243 bug:4163233 bug:4159272 bug:4159270 bug:3442104 bug:4163237 Change-Id: Iab71d1da3f0ca6c52b23ddfb8b86c58b118b60e0
* | | am 6fd1c352: am 5bdfd148: Merge "Regression fix from bug:4136077 Plus ↵Nicolas Roard2011-03-238-16/+40
|\ \ \ | |/ / | | | | | | | | | | | | | | | animations improvement bug:3389597" into honeycomb-mr1 * commit '6fd1c35298bb73147bb585e01b86d5b7c1c8f640': Regression fix from bug:4136077 Plus animations improvement bug:3389597
| * | Merge "Regression fix from bug:4136077 Plus animations improvement ↵Nicolas Roard2011-03-238-16/+40
| |\ \ | | | | | | | | | | | | bug:3389597" into honeycomb-mr1
| | * | Regression fix from bug:4136077Nicolas Roard2011-03-238-16/+40
| | | | | | | | | | | | | | | | | | | | | | | | Plus animations improvement bug:3389597 Change-Id: I07319910ae366d3e237bf3a1042cf28217434437
* | | | am 16c80237: am f32b2ce6: Fix GPU leak where plugin textures in the WebKit ↵Derek Sollenberger2011-03-238-8/+143
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | thread were not being deleted. * commit '16c80237214715ba15e45748164ecd02a558a138': Fix GPU leak where plugin textures in the WebKit thread were not being deleted.
| * | | Fix GPU leak where plugin textures in the WebKit thread were not being deleted.Derek Sollenberger2011-03-238-8/+143
| |/ / | | | | | | | | | | | | bug: 4165067 Change-Id: I2f50f600133300cec712b0177ead1a7afbbee2c3
* | | am 7fea06dd: am 77274212: Fix performance problems with layersNicolas Roard2011-03-235-22/+62
|\ \ \ | |/ / | | | | | | | | | * commit '7fea06dd1f7251b6f078821c5859bdf435fee9ca': Fix performance problems with layers
| * | Fix performance problems with layersNicolas Roard2011-03-225-22/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | We were preventing the layers' positions to be updated while waiting for the base surface to render. This also fix some repainting issues. bug:4136077 Change-Id: I6870faf8296e205c6ee58e80b1cf95833a67814b
* | | am efe35076: am 55c8e9b1: Merge "Workaround for poor dashed line ↵John Reck2011-03-221-3/+4
|\ \ \ | |/ / | | | | | | | | | | | | | | | performance" into honeycomb-mr1 * commit 'efe35076f0066a6ed0e326ea15c6d5b42a7bbf60': Workaround for poor dashed line performance
| * | Merge "Workaround for poor dashed line performance" into honeycomb-mr1John Reck2011-03-221-3/+4
| |\ \
| | * | Workaround for poor dashed line performanceJohn Reck2011-03-221-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | Bug: 4163023 Change-Id: Ie76a2f5855130ff174a80d1c7e9187bea89f3aa5
* | | | am 176fb0a1: am 50e0a1d1: Fix a browser ANRNicolas Roard2011-03-221-1/+6
|\ \ \ \ | |/ / / | | | | | | | | | | | | * commit '176fb0a1767db036303ab2df17d9e889b5e88db0': Fix a browser ANR
| * | | Fix a browser ANRNicolas Roard2011-03-221-1/+6
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Layer release textures (setting their owner to zero). Texture with no owners are routinely destroyed by the TilesManager. When a texture is busy, the release is delayed until the end of the painting; if the owner of the texture is still the same, we then remove it. The problem was that a layer could release a busy texture, thereby delaying the release; and in a next cycle re-acquire that same texture. The delayed release would then kicks in, and proceed, as the owner was still the same one as the one that originally asked for the release. We would then have a situation where the layer thought it acquired a texture, while that texture's owner was ultimately set to zero. The TilesManager would then proceed to destroy that texture. The next time the layer tried to use the texture, it would either crash or more often block on acquiring a lock on it, causing an ANR. bug:3472320 Change-Id: I84a064b1aa6636a18e0518e34c8572cbea3eb558
* | | am 71b73b3d: am 983c9f05: Merge "Fix invals for CSS3D" into honeycomb-mr1Nicolas Roard2011-03-223-3/+36
|\ \ \ | |/ / | | / | |/ |/| * commit '71b73b3d1c71449e21f04d52f2a708816c03bfca': Fix invals for CSS3D
| * Merge "Fix invals for CSS3D" into honeycomb-mr1Nicolas Roard2011-03-213-3/+36
| |\
| | * Fix invals for CSS3DNicolas Roard2011-03-213-3/+36
| | | | | | | | | | | | | | | bug:4129388 Change-Id: I52963d38412d0fd69c4695a281ad6558c27999ed
* | | Merge "Fix graphics shadow w/o blur"Russell Brenner2011-03-221-4/+9
|\ \ \
| * | | Fix graphics shadow w/o blurRussell Brenner2011-03-221-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 4129238 Changed implementation logic so alpha==0 indicates no shadowing, rather than blur==0. Added ShadowRec constructor to cleanly init entire struct. Change-Id: Ide6ee5da0f6cb174d0ef77e7d9a99b981998d7f1
* | | | resolved conflicts for merge of 4691a532 to masterTeng-Hui Zhu2011-03-212-23/+131
|\ \ \ \ | |/ / / |/| / / | |/ / Change-Id: I7a525d5179545e4faec46e58e9606d9598c241ca
| * | Support drawing paused and loading image as GL textureTeng-Hui Zhu2011-03-212-23/+131
| | | | | | | | | | | | | | | | | | bug:4142131 Change-Id: I871924cb7587077f50fdf4f9a00056f795a8daea
* | | am 9f6d6cba: am e1c3e599: Fix native crash Don\'t save the extras in the ↵Nicolas Roard2011-03-212-5/+10
|\ \ \ | |/ / | | | | | | | | | | | | | | | picture when drawing. * commit '9f6d6cba4bd687bc65baa3db29ac4f6593df65c1': Fix native crash Don't save the extras in the picture when drawing.
| * | Fix native crashNicolas Roard2011-03-192-5/+10
| |/ | | | | | | | | | | | | Don't save the extras in the picture when drawing. bug:4126884 bug:4132721 Change-Id: I52c46a33f847e64c1f8245a0bb84445a948d72a4
* | am 5616e1fe: am a5ec3c20: Fix ANR introduced by recent CL... before locking ↵Nicolas Roard2011-03-191-0/+5
|\ \ | |/ | | | | | | | | | | a texture, it needs to be ready. * commit '5616e1fe6fd63f6b8b43c0aba485d3eb7dad992e': Fix ANR introduced by recent CL... before locking a texture, it needs to be ready.
| * Fix ANR introduced by recent CL...Nicolas Roard2011-03-181-0/+5
| | | | | | | | | | | | | | before locking a texture, it needs to be ready. bug:4136077 Change-Id: Ib8235694e1a43524c52a02a5f288235d56088f46
* | am cf9235ad: am 93a2b8c5: Fix for layers textures selectionNicolas Roard2011-03-181-1/+2
|\ \ | |/ | | | | | | * commit 'cf9235adf6475a0f214598689b1fd4499594766e': Fix for layers textures selection
| * Fix for layers textures selectionNicolas Roard2011-03-181-1/+2
| | | | | | | | | | bug:4136077 Change-Id: I2dabdf612e29c4e4c9a0b744ddecaeda7a537ba5
* | am 17fa4288: am ce96fa32: Merge "Do not recreate textures if not needed for ↵Nicolas Roard2011-03-181-0/+3
|\ \ | |/ | | | | | | | | | | layers..." into honeycomb-mr1 * commit '17fa42883eed9d7de2e840b08ddbce448d34bef9': Do not recreate textures if not needed for layers...
| * Do not recreate textures if not needed for layers...Nicolas Roard2011-03-181-0/+3
| | | | | | | | | | bug:4136077 Change-Id: Ia975ac78f724450be6f36a61316c872c4d88d7a0
* | am 27679ae9: am 776c5d44: Limit the SkPicture size we use to paint. This ↵Nicolas Roard2011-03-173-11/+20
|\ \ | |/ | | | | | | | | | | improves performances by a visible factor on some websites. * commit '27679ae9d9002cfee94fa77bfb05a58863fc25f2': Limit the SkPicture size we use to paint. This improves performances by a visible factor on some websites.
| * Limit the SkPicture size we use to paint.Nicolas Roard2011-03-173-11/+20
| | | | | | | | | | | | | | This improves performances by a visible factor on some websites. bug:4124418 Change-Id: I9b24408b686e24cf323eb5bae977d74a0f2e2a34
* | am 9b533bf8: am 1ae2d07b: Merge "Enabled shadow drawing" into honeycomb-mr1Russell Brenner2011-03-172-25/+33
|\ \ | |/ | | | | | | * commit '9b533bf8fcfe66f4adbf692c1a4e1534983c4651': Enabled shadow drawing
| * Merge "Enabled shadow drawing" into honeycomb-mr1Russell Brenner2011-03-172-25/+33
| |\
| | * Enabled shadow drawingRussell Brenner2011-03-172-25/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | Reworked the logic of setupForText() and setupShadowPaint() to render correctly, as exhibited by http://acid3.acidtests.org. Bug: 4075135 Change-Id: I595456b658d0872183dedcbc726be44783440f89
* | | am b45c98a4: am 439cfed4: Fix Browser ANRNicolas Roard2011-03-173-11/+16
|\ \ \ | |/ / | | | | | | | | | * commit 'b45c98a4115c710d2d840fd3c81d1567c9f6fd57': Fix Browser ANR
| * | Fix Browser ANRNicolas Roard2011-03-173-11/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The problem was that when attempting to forcefully destroy a texture in TilesManager::cleanupLayersTextures(), the call to LayerAndroid::removeTexture() may fail if the texture was busy being painted -- the call to BackedDoubleBufferedTexture::release() would return false, and the layer may thus still keep a pointer to the texture. But the release() call, while indicating it failed, was only delaying the release -- as soon as the texture was marked as not busy, it could set its owner to nil. We could thus have a situation where the layer did not reset its texture pointers because the owner of the texture was not yet changed, but the texture would then reset its owner to nil as soon as it was not busy painting. In TilesManager::cleanupLayersTexture() the next step before deleting a texture is to check that the texture does not have an owner -- but by then the texture could have been marked as not busy, and removed its owner, letting TilesManager destroying it. bug:3472320 Change-Id: I3bcf169b30dfacba1773d3b79a3c0d205bf3cbdb
* | | am 788e19b5: am 9f5143f9: Implement partial screen invalidationsNicolas Roard2011-03-169-28/+120
|\ \ \ | |/ / | | | | | | | | | * commit '788e19b58382869e807c5da22f5052fdb0ce5ab5': Implement partial screen invalidations
| * | Implement partial screen invalidationsNicolas Roard2011-03-169-28/+120
| | | | | | | | | | | | | | | | | | bug:3461349 Change-Id: Id654d176c58027c67be7cb604b87c0ec68984525
* | | am 07b3e39f: am d6f0aa4a: Merge "Fix crash in ↵Russell Brenner2011-03-161-46/+95
|\ \ \ | |/ / | | | | | | | | | | | | | | | WebCore::GlyphPageTreeNode::getChild" into honeycomb-mr1 * commit '07b3e39f693d793798b442025da951df628105ef': Fix crash in WebCore::GlyphPageTreeNode::getChild
| * | Merge "Fix crash in WebCore::GlyphPageTreeNode::getChild" into honeycomb-mr1Russell Brenner2011-03-161-46/+95
| |\ \
| | * | Fix crash in WebCore::GlyphPageTreeNode::getChildRussell Brenner2011-03-161-46/+95
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using existing Skia APIs, FontCache::createFontPlatformData can detect when the retrieved font is the Skia default and will reject that choice, allowing WebCore to try the next font on the CSS fallback list. If the requested family is a generic family, e.g. "serif" or "monotype", the Skia default will be accepted. A prior attempt at this fix was not properly handling bold and italic stylings, which could eventually lead to dangling references to freed font data used for layout while custom fonts were loading, causing segfaults in getChild and elsewhere during page rendering. Bug: 3469204 Bug: 2720133 Change-Id: I3a9e746e18fd27848d0c4b9e1232392df33254a6
* | | am 2322a2c9: am 6a3bd3f2: For first layout, reset current textures and ↵Shimeng (Simon) Wang2011-03-162-3/+8
|\ \ \ | |/ / | | | | | | | | | | | | | | | update base layer. * commit '2322a2c93ef52b0b74f95103d0fc590a3b01d3ef': For first layout, reset current textures and update base layer.