GitLab

DOMFormData was an implementatio of FormData IDL interface. New name matches to
the inteface.

This CL has no behavior changes.

BUG=528840
R=keishi@chromium.org

Review URL: https://codereview.chromium.org/1325893005 .

git-svn-id: svn://svn.chromium.org/blink/trunk@202021 bbb929c8-8fbe-4397-9dbb-9b2b20218538

These were mostly for the convenience of layout tests, rarely used in the
prod code. Let's keep TimelineModel.Record() a thin wrapper that just adds
hierarchy to trace events and keep timeline-specific logic away from it,
since it has to work for bare trace events most of the time.

BUG=

Review URL: https://codereview.chromium.org/1315373005

git-svn-id: svn://svn.chromium.org/blink/trunk@202020 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This change adds use counters for the following methods on window:
 - alert()
 - confirm()
 - prompt()
 - print()

Review URL: https://codereview.chromium.org/1325193007

git-svn-id: svn://svn.chromium.org/blink/trunk@202018 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=529956
R=pfeldman

Review URL: https://codereview.chromium.org/1315643009

git-svn-id: svn://svn.chromium.org/blink/trunk@202017 bbb929c8-8fbe-4397-9dbb-9b2b20218538

The Chrome-side security handler now sends a SecurityStyleChanged event upon being enabled, so it is not necessary for SecurityModel to do anything to keep track of security state anymore.

BUG=

Review URL: https://codereview.chromium.org/1309953003

git-svn-id: svn://svn.chromium.org/blink/trunk@202016 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Selectors with unknown namespace prefixes should be dropped.
Added changes to check if a prefix encountered is unknown,
and if it is so ignoring the selector.

BUG=396945

Review URL: https://codereview.chromium.org/1326403002

git-svn-id: svn://svn.chromium.org/blink/trunk@202015 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Fixed-background under scrolled container has been broken when we
enabled ScrollRecorder. Because the drawing coordinates are in the
scrolled content, while fixed-background should be drawn as if it
is in the viewport, we should add the accumulated scroll offset to
the target rect of the fixed-background.

BUG=528226
TEST=paint/overflow/fixed-background-scroll.html
TEST=paint/overflow/fixed-background-scroll-window.html
TEST=paint/overflow/fixed-background-scroll-in-frame.html

Review URL: https://codereview.chromium.org/1332643002

git-svn-id: svn://svn.chromium.org/blink/trunk@202014 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Current:
 mode: "no-cors"
 credentials: "same-origin"
 redirect: "follow"

With this patch:
 mode: "same-origin"
 credentials: "include"
 redirect: "manual"

Spec Issue: https://github.com/whatwg/fetch/issues/106
BUG=522873

Review URL: https://codereview.chromium.org/1320023005

git-svn-id: svn://svn.chromium.org/blink/trunk@202013 bbb929c8-8fbe-4397-9dbb-9b2b20218538

It has been deprecated a few months ago and it's usage is near 0.

Deprecation CL:
https://codereview.chromium.org/1194963002

Counter:
https://www.chromestatus.com/metrics/feature/timeline/popularity/852

BUG=502275

Review URL: https://codereview.chromium.org/1306683007

git-svn-id: svn://svn.chromium.org/blink/trunk@202012 bbb929c8-8fbe-4397-9dbb-9b2b20218538

http://src.chromium.org/viewvc/blink?view=revision&revision=202008

BUG=492871,459055
TBR=eae@chromium.org

Review URL: https://codereview.chromium.org/1333723002 .

git-svn-id: svn://svn.chromium.org/blink/trunk@202011 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=525284

Review URL: https://codereview.chromium.org/1315643008

git-svn-id: svn://svn.chromium.org/blink/trunk@202010 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This avoids an uninitialized read in
FontCache::fallbackFontForCharacter(), which assumes the script is set.
Found by DrMemory in
http://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Content%20Browser%20%28DrMemory%20full%29%20%284%29/builds/5885/steps/memory%20test%3A%20content_browsertests/logs/stdio.

Review URL: https://codereview.chromium.org/1327143002

git-svn-id: svn://svn.chromium.org/blink/trunk@202009 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=459055
TBR=mohsen@chromium.org

Review URL: https://codereview.chromium.org/1318093004 .

git-svn-id: svn://svn.chromium.org/blink/trunk@202008 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=529471

Review URL: https://codereview.chromium.org/1314853006

git-svn-id: svn://svn.chromium.org/blink/trunk@202006 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This CL updates callers in InlineFlowBox.

BUG=499321

Review URL: https://codereview.chromium.org/1328723002

git-svn-id: svn://svn.chromium.org/blink/trunk@202005 bbb929c8-8fbe-4397-9dbb-9b2b20218538

TBR=oilpan-reviews
BUG=488628
NOTRY=true

Review URL: https://codereview.chromium.org/1327193002

git-svn-id: svn://svn.chromium.org/blink/trunk@202004 bbb929c8-8fbe-4397-9dbb-9b2b20218538

... so that Device Pixel Ratio does fit.

BUG=529744
TBR=caseq

Review URL: https://codereview.chromium.org/1310873005

git-svn-id: svn://svn.chromium.org/blink/trunk@202003 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=504537

Review URL: https://codereview.chromium.org/1327593005

git-svn-id: svn://svn.chromium.org/blink/trunk@202002 bbb929c8-8fbe-4397-9dbb-9b2b20218538

http://src.chromium.org/viewvc/blink?view=revision&revision=201990

BUG=492871
TBR=eae@chromium.org

Review URL: https://codereview.chromium.org/1323313005 .

git-svn-id: svn://svn.chromium.org/blink/trunk@202001 bbb929c8-8fbe-4397-9dbb-9b2b20218538

http://src.chromium.org/viewvc/blink?view=revision&revision=201968

TBR=mkwst@chromium.org

Review URL: https://codereview.chromium.org/1329063004 .

git-svn-id: svn://svn.chromium.org/blink/trunk@202000 bbb929c8-8fbe-4397-9dbb-9b2b20218538

http://src.chromium.org/viewvc/blink?view=revision&revision=201961

TBR=mkwst@chromium.org

Review URL: https://codereview.chromium.org/1310513012 .

git-svn-id: svn://svn.chromium.org/blink/trunk@201999 bbb929c8-8fbe-4397-9dbb-9b2b20218538

http://src.chromium.org/viewvc/blink?view=revision&revision=201962

BUG=529530
TBR=tkent@chromium.org

Review URL: https://codereview.chromium.org/1322933007 .

git-svn-id: svn://svn.chromium.org/blink/trunk@201998 bbb929c8-8fbe-4397-9dbb-9b2b20218538

http://src.chromium.org/viewvc/blink?view=revision&revision=201983

BUG=492871
TBR=szager@chromium.org

Review URL: https://codereview.chromium.org/1311543007 .

git-svn-id: svn://svn.chromium.org/blink/trunk@201997 bbb929c8-8fbe-4397-9dbb-9b2b20218538

When https://codereview.chromium.org/1329863002 landed the flakyness has
been fixed. This CL is updating the expectations accordingly.

BUG=528110

Review URL: https://codereview.chromium.org/1305933008

git-svn-id: svn://svn.chromium.org/blink/trunk@201996 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Disable SVG tests that exercise zero length path segments
with round or square end caps that change the drawing
when Skia is patched to perform the same actions natively.

R=fmalita@chromium.org
BUG=529845

Review URL: https://codereview.chromium.org/1315883003

git-svn-id: svn://svn.chromium.org/blink/trunk@201995 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This patch extends compositions to be able to span multiple nodes.  This involves
two main changes: first, compositions are switched to being stored as a Range, not
a Node plus two offsets.  Second, the underline painting now uses DocumentMarkerController
(which is already multi-node) instead of a custom composition-specific path.

As a result, we can remove the hack in setCompositionFromExistingText that clobbered
the text to shoehorn it into a single node, which would cause style loss.  This
is now able to span the existing text regardless of how complicated it is.

TEST=webkit_unit_tests --gtest_filter=DocumentMarkerControllerTest.UpdateRenderedRectsForComposition
TEST=webkit_unit_tests --gtest_filter=WebViewTest.SetCompositionFromExistingTextInRichText
BUG=488628

Review URL: https://codereview.chromium.org/1325563002

git-svn-id: svn://svn.chromium.org/blink/trunk@201994 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This change is about being more strict about applying style changes in tree
order: first adjust the relationship to the ancestry, THEN adjust the children.

This order is important if a multicol container has a child out-of-flow
multicol container with a spanner, and this child is changed to become in-flow
at the same time as it ceases to be a multicol container, and instead becomes a
spanner. If we change it from multicol to spanner first (instead of making it
part of the outer multicol container first), the outer multicol container is
going to believe that it contains the inner spanner, and we'd end up with a
spanner inside another spanner, which isn't allowed.

#a - multicol
  #b - abspos multicol, changing it to static spanner
    #c - spanner (but it should become a regular block once #b becomes a spanner)

The effect of this fix is that we swap the ordering of notifying the flow thread
about descendant style changes (flowThreadDescendantStyleWillChange(),
flowThreadDescendantStyleDidChange()), compared to when handling style changes
locally on the object (styleWillChange(), styleDidChange()) takes place. More
specifically, we need to get to flowThreadDescendantStyleDidChange() first
(which registers or unregisters descendants in the flow thread - i.e. updates
the LayoutMultiColumnSet / LayoutMultiColumnSpanner placeholder structure), and
THEN to evacuateAndDestroy() (via LayoutBlockFlow::styleDidChange() and 
createOrDestroyMultiColumnFlowThreadIfNeeded()), instead of the other way around.
This way we register #b (now a spanner) in #a first. That will prevent #a from
seeing anything inside #b (spanners are rather opaque).

Since we're now notifying the flow thread from LayoutBox instead of
LayoutObject, we can change the style change notification methods to take
LayoutBox instead of any kind of LayoutObject. The flow thread only cares about
LayoutBox or better here anyway. This allows for some cleanup in the
notification methods, since we no longer need to worry about computed style
weirdness on text layout objects.

BUG=516532
R=eae@chromium.org,jchaffraix@chromium.org,leviw@chromium.org

Review URL: https://codereview.chromium.org/1320843005

git-svn-id: svn://svn.chromium.org/blink/trunk@201993 bbb929c8-8fbe-4397-9dbb-9b2b20218538

libxml 2.9.0 (we updated from 2.7.x to 2.9.2 for M45) added a
default-on abort on > 10,000,000 byte documents which is a bit
arbitrary and is causing problems for users. The _HUGE option
makes it not do this. My assumption is that this is reasonable
behaviour security-wise as: 1) we were doing it this way until
recently; and 2) it shouldn't be any worse in the renderer than
just doing `for (;;) x+='y';`

(There's a big trivial xml file in
LayoutTests/http/tests/xmlhttprequest/resources/big.xml
that goes with the test, but git cl upload won't upload it. I
guess I'll dcommit just that file first?)

R=dominicc@chromium.org
BUG=528078,463958
TEST=LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-big-document.html

Review URL: https://codereview.chromium.org/1316673007

git-svn-id: svn://svn.chromium.org/blink/trunk@201992 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This CL updates all callers in InlineFlowBox.

BUG=499321

Review URL: https://codereview.chromium.org/1312713005

git-svn-id: svn://svn.chromium.org/blink/trunk@201991 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=492871
TBR=szager@chromium.org

Review URL: https://codereview.chromium.org/1329713003 .

git-svn-id: svn://svn.chromium.org/blink/trunk@201990 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=

Review URL: https://codereview.chromium.org/1329823002

git-svn-id: svn://svn.chromium.org/blink/trunk@201989 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This CL updates callers in LayoutText.cpp.

BUG=499321

Review URL: https://codereview.chromium.org/1330433003

git-svn-id: svn://svn.chromium.org/blink/trunk@201988 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Modifies V8WrapperInstantiationScope::SecurityCheck to only call BindingSecurity::shouldAllowAccessToFrame if m_context's world is the main world.

BUG=529667

Review URL: https://codereview.chromium.org/1323453005

git-svn-id: svn://svn.chromium.org/blink/trunk@201987 bbb929c8-8fbe-4397-9dbb-9b2b20218538

TBR=szager@chromium.org
BUG=

Review URL: https://codereview.chromium.org/1309313011 .

git-svn-id: svn://svn.chromium.org/blink/trunk@201986 bbb929c8-8fbe-4397-9dbb-9b2b20218538

We will fix the removed expectations after the chromium
side change (crrev.com/1317733006) is in.

BUG=529807

Review URL: https://codereview.chromium.org/1320863005

git-svn-id: svn://svn.chromium.org/blink/trunk@201985 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Intent to implement: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/MyINCevjAJA

The NetInfo spec changed as follows:

1) wimax is now a connection type
2) connection.ontypechange has moved to onchange (we will deprecate ontypechange once onchange is shipped)
3) connection.downlinkMax has been added

This CL adds Blink support for those changes. ontypechange is deprecated in favor of onchange. Tests are updated to use onchange. downlinkMax support is added but defaults to 0.0 until the browser process starts sending real values.

Patch 1) https://crrev.com/1308943005 (this)
Patch 2) https://crrev.com/1306423004
Patch 3) https://crrev.com/1309903006

BUG=412741

Review URL: https://codereview.chromium.org/1308943005

git-svn-id: svn://svn.chromium.org/blink/trunk@201984 bbb929c8-8fbe-4397-9dbb-9b2b20218538

This also gets rid of a bug in clampScrollOffset that sometimes caused it to use a maximum offset that was one pixel too small, so a few tests need to be rebaselined.

BUG=492871
R=skobes@chromium.org

Review URL: https://codereview.chromium.org/1223173006

git-svn-id: svn://svn.chromium.org/blink/trunk@201983 bbb929c8-8fbe-4397-9dbb-9b2b20218538

The old implementation was "allocate or die".  It's not good for
ImageData, which may try to allocate a huge memory block.  This CL
makes ImageData gracefully returns nullptr optionally raising an
exception in case of out of memory.

BUG=528898

Review URL: https://codereview.chromium.org/1328343002

git-svn-id: svn://svn.chromium.org/blink/trunk@201982 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Follows the same approach as BMPImageDecoder: Use a
FastSharedBufferReader to avoid consolidation.

Add an optional offset to PNGImageDecoder for ICOImageDecoder, so that
PNGImageDecoder can read a PNG subimage in an ICO without copying the
data. This eliminates another call to SharedBuffer::data(), as well as
a copy to create the separate SharedBuffer for PNG.

BUG=467772
BUG=528625

Review URL: https://codereview.chromium.org/1316203008

git-svn-id: svn://svn.chromium.org/blink/trunk@201981 bbb929c8-8fbe-4397-9dbb-9b2b20218538

BUG=529776

Review URL: https://codereview.chromium.org/1330163005

git-svn-id: svn://svn.chromium.org/blink/trunk@201980 bbb929c8-8fbe-4397-9dbb-9b2b20218538