| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Create a single implementation of MotionEvent's |Clone()| and
|Cancel()| methods using MotionEventGeneric. Expand
MotionEventGeneric to support historical events, also removing
CompoundMotionEvent from the MotionEventBuffer implementation.
By avoiding JNI and garbage creation, this reduces the cost of both
methods by ~10x on Android, from ~25us call to ~2us.
BUG=501503003
Review URL: https://codereview.chromium.org/502993004
Cr-Commit-Position: refs/heads/master@{#301136}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This skia API is marked as deprecated, so we need to remove all usages
of it, to be able to remove it in Skia.
BUG=None
TEST=None
TBR=skyostil@chromium.org
Review URL: https://codereview.chromium.org/669303005
Cr-Commit-Position: refs/heads/master@{#301045}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The purpose is to avoid the white flash before the web content is loaded.
This CL only handles the view implementation, mac implementation will be handled in the next cl.
BUG=368225
Committed: https://crrev.com/fe517ffc1a9550ff4557b5718b01b103bd134616
Cr-Commit-Position: refs/heads/master@{#299594}
Review URL: https://codereview.chromium.org/637083002
Cr-Commit-Position: refs/heads/master@{#300961}
|
|
|
|
|
|
|
|
|
|
|
| |
Move PostMessageToFrame method from ContentViewCore to WebContentsAndroid
which is a more suitable location.
BUG=393291
Review URL: https://codereview.chromium.org/649933006
Cr-Commit-Position: refs/heads/master@{#300615}
|
|
|
|
|
|
|
|
| |
BUG=423621
Review URL: https://codereview.chromium.org/663563002
Cr-Commit-Position: refs/heads/master@{#300128}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Java class org.chromium.content.browser.WebContentsObserverAndroid
is incorrectly named and should be WebContentsObserver to follow
the same pattern as for other classes.
The current WebContentsObserverAndroid is deprecated and it extends
new class WebContentsObserver inorder to delegate all method calls.
This is done for external repositories to adapt new class name to
WebContentsObserver after this change.
Once this is done WebContentsObserverAndroid will be removed in
next patch.
BUG=396118
Review URL: https://codereview.chromium.org/589113002
Cr-Commit-Position: refs/heads/master@{#299923}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Finally, we can revert the temporary hacks for using Blink UTF conversion
routines, as we implemented a similar fast path in the routines from base::.
I have double-checked that this doesn't regress performance on the benchmark
that we were using.
This reverts commit ed921a04a86cc068cd1e7452da7ddb1861da246a.
BUG=391492
Review URL: https://codereview.chromium.org/660843002
Cr-Commit-Position: refs/heads/master@{#299916}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When the browser receives the toggleFullscreenModeForTab
IPC we ask the app to show the custom ViewGroup (containing
a FullscreenView with the web contents in fullscreen). If
later (for the video element case) a ContentVideoView is
created then we add it to the custom ViewGroup.
The next steps (in subsequent changes) are:
1. Add tests for fullscreen for non-media elements in the
WebView.
2. Changes to ensure that the power saver blocker works
for video elements contained inside other elements.
3. Changes to ensure that scrolling works in fullscreen.
This change also removes the now unused
disallow_fullscreen_for_non_media_elements setting and
simplifies the JNI interface in ContentVideoView.
BUG=398485
Review URL: https://codereview.chromium.org/618013003
Cr-Commit-Position: refs/heads/master@{#299889}
|
|
|
|
|
|
|
|
|
| |
R=avi@chromium.org
BUG=423134
Review URL: https://codereview.chromium.org/654403002
Cr-Commit-Position: refs/heads/master@{#299850}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
manager to grey (patchset #8 id:450001 of https://codereview.chromium.org/637083002/)
Reason for revert:
Broke content_unittests on Blink waterfall. See https://code.google.com/p/chromium/issues/detail?id=368225#c5 .
Original issue's description:
> Sets the default background color of inline signin and user manager to grey
>
> The purpose is to avoid the white flash before the web content is loaded.
>
> This CL only handles the view implementation, mac implementation will be handled in the next cl.
>
> BUG=368225
>
> Committed: https://crrev.com/fe517ffc1a9550ff4557b5718b01b103bd134616
> Cr-Commit-Position: refs/heads/master@{#299594}
TBR=rogerta@chromium.org,noms@chromium.org,sky@chromium.org,asvitkine@chromium.org,ccameron@google.com,sievers@chromium.org,ccameron@chromium.org,benwells@chromium.org,fsamuel@chromium.org,nasko@chromium.org,guohui@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=368225
Review URL: https://codereview.chromium.org/654123004
Cr-Commit-Position: refs/heads/master@{#299624}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The purpose is to avoid the white flash before the web content is loaded.
This CL only handles the view implementation, mac implementation will be handled in the next cl.
BUG=368225
Review URL: https://codereview.chromium.org/637083002
Cr-Commit-Position: refs/heads/master@{#299594}
|
|
|
|
|
|
|
|
|
|
|
| |
remove the ununsed & invalid ProcessHandle parameter
ProcessHandle(pid_t) will be 0 when RenderProcessExited observer API called. There is no use for this parameter to be present as the value is 0 and none of the callers is using this parameter.
BUG=
Review URL: https://codereview.chromium.org/644473004
Cr-Commit-Position: refs/heads/master@{#299439}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GpuMemoryBuffers.
This implements proper out-of-process support for SurfaceTexture backed
GpuMemoryBuffers using our generic GPU service side GpuMemoryBuffer
allocation mechanism. Both renderer and browser compositor usage is
supported.
BUG=368716,418553
TBR=jln@chromium.org,sky@chromium.org
Review URL: https://codereview.chromium.org/634643002
Cr-Commit-Position: refs/heads/master@{#299052}
|
|
|
|
|
|
|
|
| |
BUG=422077
Review URL: https://codereview.chromium.org/644813003
Cr-Commit-Position: refs/heads/master@{#299025}
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a Java wrapper over ServiceRegistries held by RenderProcessHosts
and RenderFrameHosts, allowing registration of the Java browser services.
BUG=411294
Review URL: https://codereview.chromium.org/555233002
Cr-Commit-Position: refs/heads/master@{#298753}
|
|
|
|
|
|
|
|
|
|
| |
Manual fixups for cases the automated tooling elected to skip.
BUG=110610
Review URL: https://codereview.chromium.org/632283004
Cr-Commit-Position: refs/heads/master@{#298745}
|
|
|
|
|
|
|
|
|
|
|
| |
It looks like these places are using incorrect casting (reinterpret_cast instead of
static_cast) for GetUserData.
BUG=
Review URL: https://codereview.chromium.org/629503004
Cr-Commit-Position: refs/heads/master@{#298743}
|
|
|
|
|
|
|
|
|
|
|
| |
This will allow WebContentsObserver implementations to be aware if a
frame hase been navigated without having to worry about provisional load.
BUG=304341,421085
Review URL: https://codereview.chromium.org/600043004
Cr-Commit-Position: refs/heads/master@{#298722}
|
|
|
|
|
|
|
|
|
|
|
|
| |
locking/unlocking and notifying the dispatcher of success.
A ScreenOrientationDelegate has been added to handle platform specific implementations. Such as actual locking/unlocking, as well as if the actions are supported.
BUG=396760
Review URL: https://codereview.chromium.org/546453004
Cr-Commit-Position: refs/heads/master@{#298719}
|
|
|
|
|
|
|
|
|
|
|
| |
This patch was generated by running the rewrite_scoped_refptr clang tool
on an Android build.
BUG=110610
Review URL: https://codereview.chromium.org/628333003
Cr-Commit-Position: refs/heads/master@{#298364}
|
|
|
|
|
|
|
|
|
|
| |
Remove the new owners from subdirectory OWNERS files.
R=davidben@chromium.org, nasko@chromium.org, sievers@chromium.org
Review URL: https://codereview.chromium.org/635573003
Cr-Commit-Position: refs/heads/master@{#298314}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
content/browser/android
This step is a giant search and replace for OVERRIDE and FINAL to
replace them with their lowercase versions.
BUG=417463
Review URL: https://codereview.chromium.org/629183002
Cr-Commit-Position: refs/heads/master@{#298249}
|
|
|
|
|
|
|
|
|
|
| |
BUG=359244
TEST=None
R=avi@chromium.org
Review URL: https://codereview.chromium.org/628793003
Cr-Commit-Position: refs/heads/master@{#298176}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These places use *foo.Pass() to give a reference to the thing inside foo
and also delete it afterward. Pass() will become std::move() which does
not have side effects, so this will not work in that case, and the
delete would happen at a later time.
In order to make Pass() act like std::move() now, we need to fix these.
BUG=418297
Review URL: https://codereview.chromium.org/624943002
Cr-Commit-Position: refs/heads/master@{#298046}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:1 of https://codereview.chromium.org/625543002/)
Reason for revert:
Oops, wasn't supposed to revert this after all.
Original issue's description:
> Revert of Use the new java_cpp_enum rule in content. (patchset #8 id:140001 of https://codereview.chromium.org/615893003/)
>
> Reason for revert:
> Fails to compile on several bots:
>
> https://chromegw.corp.google.com/i/clank.tot/builders/clang-clankium-tot-builder/builds/56800/steps/compile/logs/stdio
>
> First error:
> FAILED: cd ../../clank/native/framework; python <snip ...>
> ../../../clank/java/apps/chrome/src/com/google/android/apps/chrome/webapps/FullScreenActivityTab.java:28: error: cannot find symbol
> import org.chromium.content.common.TopControlsState;
> ^
> symbol: class TopControlsState
> location: package org.chromium.content.common
>
> Original issue's description:
> > Use the new java_cpp_enum rule in content.
> >
> > This moves most of the generated Java enums under content/ to use the
> > java_cpp_enum rule removing the need for keeping the enums in
> > separate list files and the need for the .template files.
> >
> > BUG=405532, 351558
> >
> > Committed: https://crrev.com/7d5f0a581ba2f0e79b09ecbf6127ad453a861a96
> > Cr-Commit-Position: refs/heads/master@{#297789}
>
> TBR=jam@chromium.org,yfriedman@chromium.org,scottmg@chromium.org,jdduke@chromium.org,brettw@chromium.org,mkosiba@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=405532, 351558
>
> Committed: https://crrev.com/949b6e5050c7a2ee2a894799db19ec325ba911ef
> Cr-Commit-Position: refs/heads/master@{#297793}
TBR=jam@chromium.org,yfriedman@chromium.org,scottmg@chromium.org,jdduke@chromium.org,brettw@chromium.org,mkosiba@chromium.org,perezju@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=405532, 351558
Review URL: https://codereview.chromium.org/617103007
Cr-Commit-Position: refs/heads/master@{#297794}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
of https://codereview.chromium.org/615893003/)
Reason for revert:
Fails to compile on several bots:
https://chromegw.corp.google.com/i/clank.tot/builders/clang-clankium-tot-builder/builds/56800/steps/compile/logs/stdio
First error:
FAILED: cd ../../clank/native/framework; python <snip ...>
../../../clank/java/apps/chrome/src/com/google/android/apps/chrome/webapps/FullScreenActivityTab.java:28: error: cannot find symbol
import org.chromium.content.common.TopControlsState;
^
symbol: class TopControlsState
location: package org.chromium.content.common
Original issue's description:
> Use the new java_cpp_enum rule in content.
>
> This moves most of the generated Java enums under content/ to use the
> java_cpp_enum rule removing the need for keeping the enums in
> separate list files and the need for the .template files.
>
> BUG=405532, 351558
>
> Committed: https://crrev.com/7d5f0a581ba2f0e79b09ecbf6127ad453a861a96
> Cr-Commit-Position: refs/heads/master@{#297789}
TBR=jam@chromium.org,yfriedman@chromium.org,scottmg@chromium.org,jdduke@chromium.org,brettw@chromium.org,mkosiba@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=405532, 351558
Review URL: https://codereview.chromium.org/625543002
Cr-Commit-Position: refs/heads/master@{#297793}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This moves most of the generated Java enums under content/ to use the
java_cpp_enum rule removing the need for keeping the enums in
separate list files and the need for the .template files.
BUG=405532, 351558
Review URL: https://codereview.chromium.org/615893003
Cr-Commit-Position: refs/heads/master@{#297789}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is for preparing converting blink side scroll offset to be double
type. Assuming blink side starts to use double scroll offset, it can
be passed back from main/blink to compositor without converting to int
type.
BUG=414283
Review URL: https://codereview.chromium.org/584503005
Cr-Commit-Position: refs/heads/master@{#297692}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a part of "kiling FileDescriptor" effort.
This CL turns vector<FileDescriptorInfo> into FileDescriptorInfo,
and makes the class a container of file descriptor and
associated ID.
The goal is to make ownership of file descriptor easy to
reason. Now the lifetime is tracked by ScopedFD and it's clear
who is responsible for that.
R=jam@chromium.org, jln@chromium.org
BUG=415294
Review URL: https://codereview.chromium.org/585203002
Cr-Commit-Position: refs/heads/master@{#297584}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
WebGraphicsContext3DCommandBufferImpl and WebGraphicsContext3DInProcessCommandBufferImpl
in preparation for dis-inheriting these classes
BUG=338338
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/609973002
Cr-Commit-Position: refs/heads/master@{#297362}
|
|
|
|
|
|
|
|
|
|
| |
Remove the legacy fullscreen video path that was used before
html5 video controls were introduced. The last user was the
WebView, but this has been migrated to html5 controls now.
Review URL: https://codereview.chromium.org/597523003
Cr-Commit-Position: refs/heads/master@{#296935}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
is in background in WebView.
Previously, we couldn't trigger renderer's WasHidden/WasShown event in WebView because the compositer can't stop there, so the dedicated pause video IPC message was added as a work around.
Now we override the blink's page visibility, so renderer's onHide/onShow methods could be called, and compositer is still running. This patch
- Call ContentViewCore.onHide()/onShow() in AwContents
- Video playback is changed to hook into WasHidden() in RenderMediaPlayerManager to pause video, and remove a bunch of video pause code in browser side.
So, The pause of video capture and playback are both handled in renderer side now. There is no dedicated pause video IPC message.
BUG=414506
Review URL: https://codereview.chromium.org/570183002
Cr-Commit-Position: refs/heads/master@{#296792}
|
|
|
|
|
|
|
|
| |
BUG=417623
Review URL: https://codereview.chromium.org/606633002
Cr-Commit-Position: refs/heads/master@{#296738}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Android link disambiguation popup is rendered by PopupZoomer on the Java side. This CL tries to recreate that UI in Aura using a BubbleDelegate.
This part works just like the Android code:
When Blink detects an ambiguous gesture it calls RenderViewImpl::didTapMultipleTargets(), which uses paintCompositedDeprecated() in the WebWidget to get a zoomed bitmap region containing the ambiguous links the gesture could possibly indicate. RenderViewImpl then sends a ShowDisambiguationPopup message via IPC which ultimately ends up in RenderWidgetHostImpl::OnShowDisambiguationPopup(), which marshals the bitmap and then then calls ShowDisambiguationPopup on the view_.
This part is specific to aura:
After a series of delegation steps the callback ends up in WebContentsViewAura::OnLinkDisambiguationPopupRequested, which converts the target rectangle from pixels to DIPs and then requests the popup be rendered by asking its WebContentsViewDelegate, ChromeWebContentsViewDelegateViews, which owns a LinkDisambiguationPopup and asks it to render.
The LinkDisambiguationPopup uses BubbleDelegate to show the zoomed image. Any GestureEvents it receives converted into the coordinate space of the content view and then are relayed via callback back to WebContentsViewAura, which then converts them in to Blink events and sends them on to the RenderWidgetHostImpl for relaying back to Blink.
jam: content/ parts
sky: everything else?
BUG=385249
Review URL: https://codereview.chromium.org/351683002
Cr-Commit-Position: refs/heads/master@{#296355}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Move PageTransition enumeration from //content/public/common to //ui/base
so that it can be used by iOS and layered components that are used on iOS.
BUG=372373
R=avi@chromium.org, ben@chromium.org, wfh@chromium.org
TBR=ben, nasko
Review URL: https://codereview.chromium.org/562603002
Cr-Commit-Position: refs/heads/master@{#295686}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add plumbing for a ResetStreamTextureProxy which should be
called on the event of a context loss. Only hooked up the
call for the in-process implementation in this CL when the
contex is restored.
ResetStreamTextureProxy currently handles "onContextLoss" as
well as "onContextRestored" concepts. It deletes the old
stream texture id, and recreates and binds a new
StreamTextureProxy.
BUG=412578
Review URL: https://codereview.chromium.org/532993002
Cr-Commit-Position: refs/heads/master@{#295219}
|
|
|
|
|
|
|
|
|
| |
Move ScreenOrientationProvider and ScreenOrientationDispatcherHost from content/browser/screen_orientation/ to content/public/browser/ to expose them to other libraries. This will allow Ash to use them to implement the ScreenOrientationProvider on Chrome OS. Created an implementation of ScreenOrientationDispatcherHost.
BUG=396760
Review URL: https://codereview.chromium.org/508603003
Cr-Commit-Position: refs/heads/master@{#294819}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The only way to get the tracing data used to be by reading the file
provided in TracingController::DisableTracing() interface. This does
not fit well for most usages, so this introduces a TraceDataSink
interface that is backed either by a file, string, or, in case of
DevTools, by the protocol client.
This resolves the OOM in browser and imporves performance while
recording very large traces through DevTools, streamlines
TracingController core and reduces coplexity of code for most
TracingController clients.
BUG=409733,361045
Review URL: https://codereview.chromium.org/541763002
Cr-Commit-Position: refs/heads/master@{#294801}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Allow event modifier flags to propagate through the touch and/or
gestures derived from a MotionEvent. This allows, for example,
ctrl + tap to mimic ctrl + click. Note that, previously, Aura routed
these modifier flags separately, and Android failed to route them at
all.
BUG=398439,413335
Review URL: https://codereview.chromium.org/567783002
Cr-Commit-Position: refs/heads/master@{#294695}
|
|
|
|
|
|
|
|
|
|
| |
Add back auto recede, and hasten the recession upon explicit release.
BUG=389744
Review URL: https://codereview.chromium.org/543123002
Cr-Commit-Position: refs/heads/master@{#294240}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The current logic ends up creating a MailboxManager per ContextGroup,
so change this to share the mailbox manager across
InProcessCommandBuffer::Service instances.
This also allows for removing code in Android WebView that deals with
passing around the share group. None of these contexts (child, parent
compositor and video) need client-side share groups since mailbox names
are used.
TBR=tedchoc@chromium.org for content/public
Review URL: https://codereview.chromium.org/540143002
Cr-Commit-Position: refs/heads/master@{#293902}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NavigationController functionalities inside ContentViewCore.
I have started to migrate the ownership to respective class for handling specific functionalities.
In this patch I have taken ownership of NavigationController functionalities to drop in NavaigationController classes.
BUG=398263
R=avi@chromium.org, boliu@chromium.org, nasko@chromium.org, yfriedman@chromium.org
Review URL: https://codereview.chromium.org/406023002
Patch from ajith.v <ajith.v@samsung.com>
.
Cr-Commit-Position: refs/heads/master@{#293550}
|
|
|
|
|
|
|
|
|
|
| |
depends on blink CL https://codereview.chromium.org/540033002/
TBR=jochen
Review URL: https://codereview.chromium.org/545513002
Cr-Commit-Position: refs/heads/master@{#293531}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enable the InputEventStreamValidator by default on Android. Also add
some basic unit tests, and clean up several corner cases that previously
went unnoticed.
BUG=345372
Review URL: https://codereview.chromium.org/537733003
Cr-Commit-Position: refs/heads/master@{#293437}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added UMAs for:
1) The % of fullscreen video that is opened in portrait mode
2) The % of the video is suited for portrait mode
3) How long does a video play before user rotate the screen to landscape mode
4) How long video plays after user rotate the screen to landscape mode
5) How long does a video play before user rotate to portrait mode
6) The % of a video that the user switched from portrait to landscape mode
BUG=326572
Review URL: https://codereview.chromium.org/297773004
Cr-Commit-Position: refs/heads/master@{#293045}
|
|
|
|
|
|
|
|
|
|
| |
Completed the path for touch orientation attributes (i.e. radiusX, radiusY and rotationAngle of a touch ellipse) from android to Blink.
BUG=381394
Review URL: https://codereview.chromium.org/494833003
Cr-Commit-Position: refs/heads/master@{#292923}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
WebView passes the correct viewport and transform for tile
priority in both hardware and resourceless software draws. As
a result, PictureLayerImpl does not need to treat
resourceless software draws differently.
BUG=398587
Review URL: https://codereview.chromium.org/517893002
Cr-Commit-Position: refs/heads/master@{#292723}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
deletion.
When the WebView app makes a call to java over the JavaScriptBridge,
we leave the renderer hanging on a synchronous IPC. Once control
is passed into Java, it's possible that the WebView may get destroyed
(and thus the IPC channel back to the renderer closed) which means we
can't unblock the renderer waiting on the IPC response.
Instead we cache the IPC reply message and while waiting on Java to come
back to us, if we detect that the RenderFrame has been deleted, send a
reponse back before the IPC channel is closed.
BUG=408188
Review URL: https://codereview.chromium.org/498633003
Cr-Commit-Position: refs/heads/master@{#292631}
|
|
|
|
|
|
|
|
|
|
| |
We don't need to call it before using GL context anymore.
BUG=404121
Review URL: https://codereview.chromium.org/470973002
Cr-Commit-Position: refs/heads/master@{#292592}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Skia makes no guarantees about the contents of a newly allocated
SkBitmap. Consequently, it's possible that the bitmap allocated for the
overscroll effect isn't zeroed before the effect itself is painted.
Prevent related issues by always clearing the allocated SkBitmap's
contents (to fully transparent) before painting the effect.
BUG=408225
Review URL: https://codereview.chromium.org/484373004
Cr-Commit-Position: refs/heads/master@{#292280}
|