| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
* commit '465864a6d72a2505b7a725ddaadfff05cdece8fc':
DO NOT MERGE Null check
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Cherry pick
Bug: 6975836
Fixes a crash in find on page
Change-Id: I20e1597f2f3a381ee693c1bcef944243ca654110
|
|\ \
| |/
| |
| |
| | |
* commit '28c43cfac56492d4d307386c44ab9a1121f7d3bf':
DO NOT MERGE Fix memory leak
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Cherry pick
Bug: 6952980
GraphicsContext::createOffscreenContext creates an instance of both
PlatformGraphicsSkia and GraphicsContext for ImageBuffer. However,
ImageBuffer will only call delete on the GraphicsContext. In normal
GC usage, the PlatformGraphicsContext's lifecycle is longer than the
GCs, and is cleaned up by itself. This will result in leaking the
PlatformGraphicsSkia context, though. We need to make sure to call
delete on the PlatformGraphicsSkia context if we were initialized
with the deleteUs() flag, which is used to indicate just this scenario.
Change-Id: If908dcb44e99568b92ee63d6337ce9f84adc9adf
|
|\ \
| |/
|/|
| |
| |
| |
| | |
rendered black"
* commit '47387cc7716048dcea1cc5f9bf37da65d38e10be':
Fix browser rendering issue: pure color tiles are rendered black
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The “pure color tile” optimization finds the pure-color tiles and
then draws the tiles simply with the color (otherwise the texture).
But the RGBA color pointer type is char, which may overflow when
the value is larger than 127. In such case, the tiles in question
are rendered black, because of the wrong color. This bug is fixed
by defining the pointer as unsigned char.
Change-Id: I5c7214423fa6961e707bde72f0e18c610374745d
Author: Yuyang Du <yuyang.du@intel.com>
Signed-off-by: Yuyang Du <yuyang.du@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
Signed-off-by: Bruce Beare <bruce.j.beare@intel.com>
|
|/
|
|
|
|
|
| |
Need to make sure that img is not zero.
bug: 6864545
Change-Id: I2a2c5440347681b462abaa5ac9101ddb6137c8c4
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| | |
If a layer marks the viewport dirty, it should mark all of its children as well,
since they likely have also moved.
bug:6547356
Change-Id: I0db2704f35e7c8a78108d6ed8372201a7f025994
|
|/
|
|
|
|
|
|
|
|
|
| |
bug:6694807
They don't paint onto surfacebackings, and draw afterward, so ordering issues
occur. Ideally, they should still be allowed to merge onto surfaces and not
allow other painting layers to join the surface afterward, but this shouldn't be
wasteful in practice.
Change-Id: I7a8764bc2b8e531e718602c6114b027a9dbce42d
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Bug: 6634077
Software was accidentally calling the inner updatePositions instead
of the outer updatePositions which the hardware path calls. Move
the inner updatePositions to be protected instead of public to
guard against this in the future as well.
Change-Id: Ib509f151ce64bf390383eaa0c32a2bae89428e89
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
An issue occurs on Stingray where calling glTexSubImage2d in order to blit an
update to a texture fails silently if the texture has already been updated this
frame.
With this change, we now draw the tile off-screen to work around the issue just
before the second blit is required. This way, the second blit succeeds.
bug:6630040
Change-Id: I07213954669b0f3d3f5b87a748a591e532b097b4
|
|/
|
|
|
|
|
| |
bug:6579681
Regression from https://android-git.corp.google.com/g/#/c/197452/
Change-Id: If185faec6bb2d014c304afbb8ff752ebfc6e4b51
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The available tile textures pool is not updated while we try to prepare image.
This can lead to getting obsolete tile textures.
bug:6613009
Change-Id: I0170f0cfba197c7a7ed48866028b30075417af30
|
|\ \
| |/ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In the slow draw path, we are drawing the small background image one by one.
Now in the fast path, we can use the GL shader to repeat the iamges.
bug:6579681
Change-Id: I8ab413bdd073cb867002b77b36d4a1361290ecb2
|
| |
| |
| |
| |
| |
| |
| | |
Bug: 6628376
m_scrollLimits has the correct max scroll range, even for iframes
Change-Id: Id6f23e23652e38fe6df8d6540977dc0ab7e8d332
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
With no knowledge of the actual SkTypeface used to render text before
render-time, the best we can do is turn fake bold on for all text and
simply ignore it at render-time if the SkTypeface's style is bold.
Also the complex FontPlatformData hashing mechanism needed to account
for fake styles, as the versions of FontPlatformData with fake styles
enabled are otherwise indistinguishable from the normal style version
Bug: 6522642
Change-Id: Ic0e9f1bbd8cae9fdd3a6d1d015bb9224c8be545c
|
| |
| |
| |
| |
| | |
bug:6605853
Change-Id: I65583e61197452ac70c5adf5f967182f0fd7578a
|
| |
| |
| |
| |
| |
| |
| |
| | |
Fixes an issue where tiles outside the new bounds weren't invalidated, as the
layer shrank in one dimension.
bug:6609350
Change-Id: I1f6715670654c9e6211dbec71875aa2ea11cd8f8
|
| |
| |
| |
| |
| |
| |
| |
| | |
Base surface can expand dynamically (see BaseLayerAndroid::updatePosition) to
fill the viewport, so disable clipping on it.
bug:6593707
Change-Id: I3afafb28fbe95d3162c76f8e835932577ded65ce
|
| |
| |
| |
| |
| |
| |
| | |
Also isolates layer translation fudging so as to not affect child transforms.
bug:6585341
Change-Id: If416c6fad90ee37fbe3438e01706b34e683165b7
|
|/
|
|
|
|
| |
bug:5683630
Change-Id: I4d617962a8412de884903f150799f733ea996e1c
|
|\ |
|
| |
| |
| |
| |
| |
| | |
bug:5683630
Change-Id: I43f738f2649a79b4ad7865ed27375c07195fa9b9
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| | |
Also fixes matrix initialization in BaseLayerAndroid::getLocalTransform()
bug:6389297
Change-Id: I6a3df3de129aa263986c5082614d8d76bc7408f7
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Bug: 6505013
When drawing a bitmap, don't use setupPaintFill(). The reason is that
setupPaintFill will use the fillColor to influence the alpha set on the
SkPaint. However, that fillColor does not apply to the bitmap, and can
have the result of applying the wrong alpha (as the alpha set is fillColor
alpha * graphicsContext's alpha). Instead, set the alpha on the
paint directly and use setupPaintCommon
Change-Id: Ib8d7ef494593a2896e6e3ed03c00d8d4d4b7232a
|
| |
| |
| |
| |
| | |
bug:6560078
Change-Id: I1ef7b3e6398f1cb9a80027a6917d9a557deb8c6f
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If the current clip covers the entire view, we don't have to calculate framework
invals from swapping layers - we already draw them immediately.
bug:6530473
Change-Id: I4b639e06781371e37c246f3220b79f28e7d95649
|
| |
| |
| |
| |
| |
| | |
Bug: 6563848
Change-Id: I167487fab64c97b6c4ec7f1b14639415ce15e26f
|
|\ \
| | |
| | |
| | | |
usage" into jb-dev
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Using m_scale would mean that while zooming out, the clipped area would get
larger, but the scale would stay the same.
bug:6561401
Change-Id: I0806ecb8f565c4d227a2be2b7a27a0af6aff7b46
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
bug:5461143
frameworks change:
https://android-git.corp.google.com/g/#/c/193751/
Change-Id: I691a8aa7eb2a7fc196358a3ab2929075bd4a8a8b
|
|\ \
| | |
| | |
| | | |
into jb-dev
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously, hundreds of layers could take 4+ ms to update transforms. Now, only
those that are fixed, have animations, or are scrollable calculate transforms.
bug:6530473
Change-Id: Ic9b14b76190c5bd0e5a405914a624d92b7ca7980
|
| | |
| | |
| | |
| | |
| | |
| | | |
Bug: 6317798
Change-Id: I8c4cd3bf4c538aff202ed0e3f84982fb112936a1
|
|/ /
| |
| |
| |
| | |
bug:6535726
Change-Id: Id944448a9dbafa951fd3c7eab0f248380d0a5314
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Layer can now use the offset to paint, resulting in correct translates for
overflow:scroll layers in the Layer::draw() path.
bug:6548000
Change-Id: I1dfb88b15491e72e6c3841ce2953e114124e40fd
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We do this check in other places. We still need a better
solution for the general case.
bug:6540356
Change-Id: I3f75c62141d7b6993928927117d972d0ec224b55
|
|\ \ \ \ |
|