Merge "keep rects separate across canvas push"

keep rects separate across canvas push

The nav cache identifies visible elements by accumulating
the drawing commands used in an interesting part of the picture,
then eliminating the portions that are too large or overdrawn.

Consecutive draws of the same type are combined, so that, for
instance, glyphs assimilate to form a string, or bitmaps form
a nine-patch. Draws of the same type shouldn't be combined when
they represent different objects.

Break apart draws of the same object if there is a push/pop
pair surrounding the previous draw of that object type.

bug:3241830
bug:3241992
Change-Id: I4eb65a0ae0d923d4950cda88e16cb4bbacfde663

Rewrite scrollable layers using multiple layers.

Remove PlatformBridge::updateLayers and instead call layersDraw from
ChromeClientAndroid during a requested sync.

Instead of LayerAndroid having knowledge of scrolling pictures, use multiple
layers (background, mask, foreground).

Update navigation to use the new structure.  Always record nodes in absolute,
unscrolled coordinates.  Do not track layers unless the node has a composited
layer and a last child.  Check for composited parents when adding to tracked
layers as well.

TODO:
* developer.android.com doesn't completely work with navigation due to some
  nodes being clipped out.
* BaseLayerAndroid needs to be able to operate with a matrix other than
  identity.
* nested fixed position elements do not draw correctly.
* nested overflow layers haven't been tested.

Change-Id: I0e2bd37612341e8884d68153ab36194cb2dc1eeb

Remove some temporary logging in the JNI bridge

This was added to try to track down a crashing bug but is no longer
required.

Bug: 2435360

Change-Id: I5e88a842e42cd89e96c7f2eea54e96a440f890ca

Merge WebKit at r72274: Update WebKit revision number.

Change-Id: I38e2e15fe1061f51a5fc299f45763b0e49300b4c

Merge WebKit at r72274: Implement PlatformBridge memory management methods for WebKit change 71639

Get device class specific memory constraints for V8.

V8 can be more efficient in terms of memory management but to do
so needs to know about what resources are available to it.
This varies device to device on Android so add implement the functions
that the the PlatformBridge needs to query the relevant values from
Java.

MemoryUsage.cpp will compile for JSC, but needs tuning to report
correct JSC stats. It's currently a non-issue, as this routine is only
used with V8, at present.

Requires a frameworks/base change:
https://android-git.corp.google.com/g/#change,78690

Bug: 3075565

Requires a previously upstreamed webkit change:
http://trac.webkit.org/changeset/71639

Change-Id: I25ff54b5c8e25f5de5380e0d43e56551587adc5d

Merge WebKit at r72274: Moved definition of EXPORT

The definition of EXPORT was removed from WebCorePrefix.h, citing that it's only needed by WebKit/android. Added definition to WebKit/android.

http://trac.webkit.org/changeset/71578

Change-Id: I3fe6909d5e5c668299ca25dc39ef0e444620fd33

Merge WebKit at r72274: Fix CodeGeneratorV8.pm

Some of the script-generation code puts include statements in source files when
they need to be in headers.

Cherry-picked from webkit:
http://trac.webkit.org/changeset/72948

Change-Id: I07d425bcb97abcdda25b5ff2ea7af887a925f489

Merge WebKit at r72274: Add new flag to ImageSource constructor with asssociated storage member

Patch for webkit change:
http://trac.webkit.org/changeset/72130

Change-Id: I5718b8c1ee5b2a9c8afa31de24e882f30d62972f

Merge WebKit at r72274: Add orientation stub

Orientation is hardwired to horizontal. Needs proper implementation.

http://trac.webkit.org/changeset/71970

Change-Id: If838dd9689e0e54b282baaa82515b2460e50578f

Merge WebKit at r72274: Fix Makefiles

Add ResourceLoadScheduler.cpp to build

http://trac.webkit.org/changeset/71562

Change-Id: Ia2fabc508bbcd65023824424951484f644066b1d

Merge WebKit at r72272: Fix conflicts.

WebCore/Android.v8bindings.mk
http://trac.webkit.org/changeset/71802

WebCore/bindings/v8/V8Proxy.cpp
http://trac.webkit.org/changeset/71735

WebCore/css/CSSParser.cpp
http://trac.webkit.org/changeset/71767
http://trac.webkit.org/changeset/72116 (several APIs purged)

WebCore/loader/ResourceLoader.cpp
http://trac.webkit.org/changeset/71884

WebCore/platform/graphics/MediaPlayer.cpp
http://trac.webkit.org/changeset/72117

WebCore/platform/image-decoders/ImageDecoder.cpp
http://trac.webkit.org/changeset/72130

WebCore/rendering/RenderBlockLineLayout.cpp
http://trac.webkit.org/changeset/72235

WebCore/rendering/RenderView.cpp
http://trac.webkit.org/changeset/71851

Change-Id: If0a3466754b9755c1f4be97bf589f88c48eece0a

Merge WebKit at r72274: Initial merge by git.

Change-Id: Ie51f0b4a16da82942bd516dce59cfb79ebbe25fb

Merge "Fix crash when syncing autofill settings."

am 6a2e8695: am 5d1bf1f6: Merge from open-source master

* commit '6a2e86958d2c1827daa0639e5e9ed9515252f3be':
  Fix compilation error:

am 5d1bf1f6: Merge from open-source master

* commit '5d1bf1f6b21848ec1086d59111306d2d4a4b5fc3':
  Fix compilation error:

Merge from open-source master

Change-Id: I22fd2b911ff05cd0d01fe151b18d64b4ac98e547

Fix crash when syncing autofill settings.

Since we moved the autofill initiation code out of the constructor
and into it's own method, we may crash if there is an autofill profile
set but the feature is turned off (i.e. the user had it on at one point
and had a profile set up but turned it off without deleting the
profile). That's because in this case we execute setAutoFillProfile()
in WebSettings without having first called the new init method. It makes
sense to not sync any profiles though when the feature is not enabled,
so add that check.

Change-Id: I9d99e96347770e5b08c982513f2370cb770cbe94

Merge "Fix style in WebCore/platform/network/android"

Enable SPDY, and renames initChromiumLogging

initChromiumLogging is now renamed to the more generic initChromium,
the files are renamed to ChromiumInit.*

Change-Id: Ic9639d637f25e664b51d076b9b0ee58c2867c1b5

Merge "Fix compilation error:"

am 1fc015f0: (-s ours) am 4741f411: Do not merge.

* commit '1fc015f0ce86a2bed612d2eec53b01cac6b8a793':
  Do not merge.

am 4741f411: Do not merge.

* commit '4741f41129460f4c8cdca5df12a38305f38945d0':
  Do not merge.

Fix style in WebCore/platform/network/android

This matches what was upstreamed in
http://trac.webkit.org/changeset/73062

Change-Id: I2cf2064ab51105fb16039f8af285a3797abe49a9

Use one WebRequestContext per WebView

This change switches from using a pair of WebRequestContexts (one for
private browsing, one for regular browsing) to using one
WebRequestContext per WebView. This is required to allow us to set the
userAgent on each WebView.

The WebRequestContext is now owned by the WebView.

Bug: 3113804
Change-Id: Iba2b1490e7ce4ff65c08a04a310963fa2c7e4f83

Merge "b/2864818 setting the appropriate error code for certificate error."

Do not merge.

Re-implement a security fix against our older webkit code base.

Change-Id: Iaefa1d060bb71b717558a427e326742f8fa5c4fd

modified:   WebCore/rendering/RenderBlock.cpp

modified:   WebCore/rendering/RenderBlock.cpp

b/2864818 setting the appropriate error code for certificate error.

Change-Id: I628b6fdd82089207c5a25ad816d3e6475d85911a

Merge "Do more AutoFill steup lazily"

Do more AutoFill steup lazily

Currently, much of the AutoFill setup is done in the WebAutoFill
constructor, which is called when the Page is created. This change moves
this setup to be done lazily when searchDocument() is called, which is
the entry point into the feature.

Change-Id: I0d437ed53391b5a7d8d16ec1000dc783c710e564

Merge "Fix a bug when cleaning up private browsing"

Merge "Youtube urls can now start with /v/ or /e/"

Youtube urls can now start with /v/ or /e/

Fix for bug 3242961

Note that the youtube player crashes when you launch it,
but that also happens with old style urls.

Tested with the testcase in the bug.

Change-Id: I0cb59f8404fe47aabec34c0a21e614f553ea0c32

Fix a bug when cleaning up private browsing

Delete the singleton WebCache and WebCookieJar objects when we
clear up private browsing, so that they are correctly recreated when
next required.

Note that this patch causes the WebCache to leak the HostResolver
object, as was the case before the refactoring in b/3113804. See
b/3243797.

Bug: 3113804
Change-Id: Ic0e2a45b5fd2b30097880e57eac2f6cadbd5dc85

WtfStringToJString should be able to return a valid string on empty input

It is sometimes desirable to return a valid but 0-length string created
from a 0-length WTF string but currently the WtfStringToJString function
will always return a null jstring on an empty input. Add a parameter to
specify that it should return an empty jstring on empty WTF string and
default it to false.

Change-Id: Ia449f1f9ae9b37c3da7d0272424e4fd4878ceb31

Merge "Add FlushCookieStore() method to CookieManager (C++ side)."

Merge "attach the last extra to the next picture"

pass the x and y to retrieve anchor data

During a long press, the original pointer to the
node and frame may change. Pass the location instead
to attempt to find the anchor.

companion change in frameworks/base

bug:3240869
Change-Id: Idf5eb67e2cb544dfa312e41f9224813c6ca80563

attach the last extra to the next picture

When a new picture is generated for the content,
use the last cursor ring / text selection etc.
for the interval between the picture message and
the view inval. This reduces flickering.

bug:3230094
Change-Id: Ie099a8db395bfa707790ffe3c1f7f5398cc9afa4

Add FlushCookieStore() method to CookieManager (C++ side).

The browser app will use this to sync cookies to flash when it is
sent to the background. Corresponding Java code is in I04e96aff.

Depends on external/chromium changes in Ie600c588.

Bug: 3231371

Change-Id: Ia9b56f3cdb5acd1fca23102b68b2f83a94ba6e69

am e5097951: (-s ours) am 5793dcc1: Merge "Do not merge: fix dns prefetch, multiply range" into gingerbread

* commit 'e5097951a2019ae742b9c348b21e613631f59cdc':
  Do not merge: fix dns prefetch, multiply range

am 5793dcc1: Merge "Do not merge: fix dns prefetch, multiply range" into gingerbread

* commit '5793dcc19c1dd5f2f3daacd05646ae237f050e58':
  Do not merge: fix dns prefetch, multiply range

Merge "hit test complete node even if partially visible"

Remove a superfluous TODO in ResourceLoaderAndroid.cpp

Change-Id: I290228dce1ce3e0be56666775f591c6c2b7dab32

Merge "Do not merge: fix dns prefetch, multiply range" into gingerbread

Merge "Provide a URLRequestContext directly to AutoFill, rather than a factory function"

Provide a URLRequestContext directly to AutoFill, rather than a factory function

In b/3044989, AutoFill was modified to take a factory function for the
URLRequestContext (which was then created when needed), rather than
taking the context directly. This was to avoid a race condition where
the context is created before the required storage paths have been
synced to the WebCore thread through the BrowserFrame.

Since https://android-git.corp.google.com/g/76579 and
https://android-git.corp.google.com/g/80056, we read the required
paths directly from JniUtils, so the race condition no longer exists. We
can therefore provide the context directly to AutoFill.

Note that while the context may be created before the Settings have been
synced to the WebCore thread, the necessary properties will be updated
as part of this sync before the context is used.

This change will also simplify things when we use one context per
WebView.

Also requires a change in external/chromium ...
https://android-git.corp.google.com/g/82146

Change-Id: Idfc34d435bae21654a02e5898aef0e84da98218a

am a5b8875c: (-s ours) am 1baa4609: Cherry-pick WebKit security fix (webkit.org r64706) Do not merge

* commit 'a5b8875c0b61e9dc9eb76e16a4cc1f741709ee74':
  Cherry-pick WebKit security fix (webkit.org r64706) Do not merge

am 1baa4609: Cherry-pick WebKit security fix (webkit.org r64706) Do not merge

* commit '1baa4609252ea42065c4399b9081b2087c8d70a2':
  Cherry-pick WebKit security fix (webkit.org r64706) Do not merge

hit test complete node even if partially visible

An older test incorrectly discards nodes if they are partially
off the screen. The test was attempting to ignore nodes that
were fully invisible, so change the test to do that.

bug:3198707
Change-Id: I3cb13de3c86b43d3388849161bdfc44f84d93b3d

Merge "Refactoring the zoom"

Merge "AutoFill no longer needs a unique profile id native side."

Merge changes I006cddf2,Ib304a845,I4622749d

* changes:
  Moves implementation of ResourceLoaderAndroid to ResourceLoaderAndroid.cpp
  Add a static WebRequestContext::acceptLanguage() method
  Make WebCookieJar::get() threadsafe

Moves implementation of ResourceLoaderAndroid to ResourceLoaderAndroid.cpp

This means that the implmentation of this class for both HTTP stacks is
in the same place.

Change-Id: I006cddf27c245a88327643314bb7564a2486ff38

Add a static WebRequestContext::acceptLanguage() method

This removes the need for PlatformBridge::computeDefaultLanguage() to
get an instance of WebRequestContext. This is part of the refactoring to
use a WebRequestContext per WebView.

Bug: 3113804
Change-Id: Ib304a8457a82d09d80fe4d6298f00f688ec53b7c

scroll the selection when dragging near the edge

Return the bounds of a layer.

support for a companion change in frameworks/base

bug:3186988
Change-Id: Ie8f69dc8c46c6329fd0b093ecdba4715f8cde2a9

AutoFill no longer needs a unique profile id native side.

AutoFill profiles now use GUIDs that they generate themselves
rather than injecting a unique int ID so we no longer need to
reach into the java class to grab the ID.

Change-Id: I26492aeddd130831e941ec79ef1f5e36271903c8

Make WebCookieJar::get() threadsafe

This should have been part of
https://android-git.corp.google.com/g/78476 which factored
WebCookieJar out of WebRequestContext.

Also, now that WebRequestContext::get() is called nly on the WebCore
thread, it does not need to be threadsafe.

Bug: 3113804
Change-Id: I4622749d1650be0b9ca0d37541361955a367110f

Merge "Factor out WebCache from WebRequestContext"

Factor out WebCache from WebRequestContext

This is the second step in allowing us to have a WebRequestContext per
WebView, which is required to allow the useragent string to be
configured per WebView.

Bug: 3113804
Change-Id: Idcd252194562d1ef91d9d6e2baf8c65b8ff088d4

Merge "Minor code cleanup in WebCookieJar"

Minor code cleanup in WebCookieJar

Change-Id: Ia68bac6bd93b0afeff049a9ac0f056191d92d2a0

Merge "Factor out CookieManager from WebRequestContext"

Factor out CookieManager from WebRequestContext

This is the first step in allowing us to have a WebRequestContext per
WebView, which is required to allow the useragent string to be
configured per WebView.

Bug: 3113804
Change-Id: I9ccf07d4a277147e74e2f3e701bd6d2166954280

Merge "No longer need to pass storage paths to WebRequestContext.cleanupPrivateBrowsingFiles()"

No longer need to pass storage paths to WebRequestContext.cleanupPrivateBrowsingFiles()

Requires the following changes to frameworks/base and packages/apps/Browser
https://android-git.corp.google.com/g/81782
https://android-git.corp.google.com/g/81780

Bug: 3232569
Change-Id: I73cabb561e206beee2bc1716f04aed8764797cc1

Merge "Fixes WebResourceRequest to correctly set load flags"

Fixes WebResourceRequest to correctly set load flags

We now correctly set the cookie load flags and the cache control HTTP
headers.

Bug: 2889880
Change-Id: I759a3131b73a8fc82432a067a31754fcc566c136

Session cookies are now persistent.

We previously checked !IsPersistent() to determine whether a
cookie is a session cookie, but now all cookies are persistent,
so we need to call the new method IsSessionCookie().

Depends on https://android-git.corp.google.com/g/81755

Bug: 2864791
Change-Id: Ia44cd0cdc5e68a4b0a61bdeebd70cdccfc8e4e1b

Set useCachedCredentials to true on first auth request.

This flag previously defaulted to false, but the Android CTS
specifies that a "sufficient" implementation should return true
on the first authentication request. (The second and subsequent
auth requests aren't tested.)

This CL sets the flag to true on the first authentication request
for a given WebRequest, but false for subsequent requests (i.e.,
if the first attempt to authenticate failed).

Using a counter rather than a flag in WebRequest for readability.
"isFirstAuthRequest" would be a bit ambiguous as a member variable,
but "authRequestCount" seems pretty clear.

Change-Id: Id9baae4c30111f743e079f90940078b07498b168

Refactoring the zoom

Change-Id: I84d44b6aab2222b52d21a65ceb74e390c677fe72

Merge "use style ignoring pending style sheets"

use style ignoring pending style sheets

If the style is not yet computed, attempting to
compute the style may blow up (e.g., the font points
to zero and is dereferenced). The webkit way around
this appears to be to ignore pending style sheets,
even though six lines down from this change, the
regular webkit makes the original styleForElement
call.

This does fix sfgate.com, so it's not a bad thing to
do, and the real style change should retrigger this
code path later on.

bug:3194102
Change-Id: Ia558e3d49c94ed817d69d528b9e914f906f62a66

Merge "allow animated gifs to skip frames"

Merge "Fix crash in JavaClassV8"

allow animated gifs to skip frames

Removing this patch causes gifs to skip frames rather than
play all intermediate frames in an attempt to catch up.

bug:2621902
Change-Id: Idc19b11991c37c685d1dc61bb5daddc6d12d30e0

Fix crash in JavaClassV8

It is possible that the m_name member of JavaClass is uninitialized as
we possibly early out from the constructor if we could not load the Java
class from the JVM. In this case when we call the dtor we try to free
uninitialized memory and crash. m_name is not actually used anywhere
so we just remove it.

Will upstream to webkit.

Bug: 3222518
Change-Id: I9354e9914157d2c4afd72dcce9122329c2486c89

Merge "Removing CRASH() from asserts in WebRequest.cpp"

Merge "Add LayoutTests http/tests/cookies at r71558"

fix radio, checkbox hit testing

Radio and checkbox input types draw only a bitmap in
their navigation bounds, and require some special treatment.

Never treat them as transparent.

Use the slop box instead of the exact box when detecting
them, since our asset may be larger than the space webkit
provides.

Also, fix a debug test if the layer is null.

bug:3215174
Change-Id: I3b573e3aa215268390db2d58435ac47115804ed7

Add LayoutTests http/tests/cookies at r71558

All tests pass or have failing test expectations.

Bug: 3227269
Change-Id: Ia0d58eb29d312c57c2f687e5431bb997b9433560

Removing CRASH() from asserts in WebRequest.cpp

These asserts have not been triggered in months, so removing the
crash (but keeping logging), and closing bug 2951985.

Change-Id: I5bb31c6e23c18e83301428582ad895b0b679ff86

reenable draw extras when GL is turned on

This captures the drawing in the UI thread, then passes
the drawing to the tile imaging thread.

The draw extras interface now takes an additional rectangle,
and each draw extra fills in the inval area formed as that
part is drawn.

The old extra implementation in GLWebViewState has been
removed. The inval portion of the setBaseLayer call has been split
out so it can be called directly.

bug:3161294
Change-Id: I28d3e6879059770b973e7c0f7c0796909f7359aa

Revert "Merge Chromium at r65505: Work around regression introduced by 64761"

This reverts commit 0c918ae77b5edfe59b726ff40a7c86dbc4a94307.

This was actually not really a regression as we were still broken
previously it's just that this change makes that breakage more visible.

The problem reproduces on tip of tree Chromium, so we can make the fix
upstream and then cherry pick into Android.

Change-Id: I4d65544b4ea137d77c00a39fcd29edb28b40aa1c

Merge "By default, provide native AutoFill with an empty profile."

By default, provide native AutoFill with an empty profile.

If no profile is set then AutoFill will not inform Java that it has
found a form that could be autofilled. We want to be able to offer
the user the option Java side to set up a profile in this case, so
we set by default an empty profile that is active until the user
syncs their real profile. This ensures that the AutoFill detection
code will not early out.

Change-Id: I474b25c5b31dae59e4ddb23c7f398d013e9d3beb

Fix the Zoom out / dirty tiles bug

Bug:3212531 Bug:3212527
Change-Id: Ie038f5b42cd34653f0a3e8fbe8c1fb1c0c3aceca

Fix case where the Texture class held a reference to a deleted Tile.

bug: 3217947
Change-Id: If75673ece459d6539f1fd990d538347e13d60d68

Clear GL inside clear view.

issue: 3215000
Change-Id: I8668a857d840084c363b13c95efffc740a1451db

Fix browser ANR when using Hardware Acceleration

Bug:3217947
Change-Id: Ic0da5c4c131026a88afff2acc254a23a4d6a42ff

Merge "Call maybeSavePassword() from Chrome HTTP stack."

Merge "Add maybeSavePassword() method to WebFrame."

Merge "Fixes the handling of unspecified mime-type"

Fixes the handling of unspecified mime-type

 Bug: 3211038 and 3211569
 If the server doesn't specify a Content-Type header in the response,
 there was no mime-type set when using the Chrome HTTP stack. This change
 fixes it so that it makes a guess based off of the URL extension.

Change-Id: I33ef90c1db54bdccb4e143795656f4f9b56572ca

Call maybeSavePassword() from Chrome HTTP stack.

This will prompt the user to save the username and password
when posting a form. Same behaviour as the Android HTTP stack.

Bug: 3060688
Test: Logged into amazon.co.uk, checked that password is saved.

Change-Id: I4185b7ef2fbb299e4441b18b50f0018f0208d264

Add maybeSavePassword() method to WebFrame.

This calls through to the new entrypoint on the Java size
(see change I3f3f84a1). I've done some modest refactoring
to minimize code duplication. The way we obtain a pointer
to the frame is still a bit messy, but I couldn't find a
really clean solution to that.

Change-Id: Ia5c12f615b8c41f1c846bbf6fd6dcfe56f3e2a51

Provide the line spacing attribute for textareas.

Bug:3085564

In order to line up the WebTextView text with the page's text,
we need to know the line spacing attribute stored in the DOM.

Requires a change in frameworks/base:
https://android-git.corp.google.com/g/#change,80646

Change-Id: I539454df318826233496ffcd140ba1d6e396a4a0

Merge Chromium at r65505: Work around regression introduced by 64761

This is a temporary workaround to fix the regression introduced on
Android by http://src.chromium.org/viewvc/chrome?view=rev&revision=64761

It should be reverted when we understand better the root cause of the
regression.

Change-Id: I6652d817254f1b8c3857e2f14b53d5d034d2f12f

Merge Chromium at r65505: Merge AutoFill change 64761

Merge in http://src.chromium.org/viewvc/chrome?view=rev&revision=64761

This is not part of the bigger autofill merge as this introdcues
a regression that we need to understand.

Change-Id: I08cfcefb93181e70a8a48d0c6da2217d3a5bbcf9

Merge Chromium at r65505: Update AutoFill files.

Bring our forked AutoFill files in line with r65505.

http://src.chromium.org/viewvc/chrome?view=rev&revision=63527
http://src.chromium.org/viewvc/chrome?view=rev&revision=63797
http://src.chromium.org/viewvc/chrome?view=rev&revision=64470
http://src.chromium.org/viewvc/chrome?view=rev&revision=64572
http://src.chromium.org/viewvc/chrome?view=rev&revision=64612
http://src.chromium.org/viewvc/chrome?view=rev&revision=64691
http://src.chromium.org/viewvc/chrome?view=rev&revision=64696

Change-Id: I34755e649d1093a79ad50de53dfd1d1f9e92a584