GitLab

dcheck_always_on=1 has the same effect now.

BUG=411008

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

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

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

TBR=esprehn@chromium.org

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

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

https://codereview.chromium.org/511253003/ replaced applyScrollAndScale
with applyViewportDeltas. This method can now be removed from the Blink
API.

BUG=333143

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

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

This patch enables background html parser for documents with no frames,
which are XMLHttpRequest response documents. HTML imports and
inspector/DOMPatchSupport will still to use the synchronous mode as the
current code expect the change to happen synchronously.

Before this change, XMLHttpRequest immediately set its state to DONE after
receiving the last data. After this change, XMLHttpRequest will wait for
|Document::finishedParsing| before changing its state to DONE if it has
responseType == "document". This is needed because |DocumentParser::finish()|
do not synchronously give the complete document, but we have to wait until all
tokens from the background parser is processed.

BUG=409461

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

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

Removed an unused CompositionUnderline constructor.

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

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

to keep figcation infomation on accessibility tree.

BUG=None

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

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

All of these measure stable features except CSSPropertyWebkitCursorVisibility which
is actually removed.

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

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

BUG=411864
TEST=blink unit test Canvas2DLayerBridgeTest.testPrepareMailboxAndLoseResource

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

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

This reverts https://src.chromium.org/viewvc/blink?revision=173614&view=revision

Breaks going in and out of compositing mode for layers that are floated. There
is logic that is reading the self painting layer bit when computing the overflow
of elements inside layout and also when computing if floats should paint. My
original patch was generally wrong, so many parts of layout are reading
hasSelfPaintingLayer() inside layout.

BUG=411825

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

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

BUG=404355

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

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

BUG=412478

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

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

This is a few little cleanups which I left behind as FIXMEs in previous
changes that were designed to be merge-worthy.  I don't believe any of
these changes should have an observable effect in real usage.
They are a pre-requisite, however, for additional fixes I intend to
make.

BUG=398914

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

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

BUG=408910

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

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

Also rename it v8_conversion_needs_exception_state, which is exactly what
it means.  The old name is somewhat misleading, since other conversions
can typically also raise exceptions in practice; the difference is whether
the conversion function does so via (and thus expects) an ExceptionState
argument.

Defining it in v8_types.py makes sense since this is a property of the
V8 bindings layer's conversion API for the type, not of the type itself.

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

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

Revert of Revert of HTMLConstructionSite: avoid n^2 running time for large scripts. (patchset #1 id:1 of https://codereview.chromium.org/555223002/)

Reason for revert:
Not a culprit. Reverted to bring back the original patch.

Original issue's description:
> Revert of HTMLConstructionSite: avoid n^2 running time for large scripts. (patchset #5 id:80001 of https://codereview.chromium.org/494993002/)
> 
> Reason for revert:
> Speculative revert.
> 
> It may break the following blink sheriff bot:
> http://build.chromium.org/p/chromium.webkit/builders/Android%20Tests%20%28dbg%29/builds/21403
> 
> 
> 
> Original issue's description:
> > HTMLConstructionSite: avoid n^2 running time for large scripts.
> > 
> > Every time background parser sends chunk, tree is flushed.
> > 
> > If page contains very large script, then script node content is updated
> > many times. Every update is causes string concatenation.
> > 
> > Solution: do not flush pending text until it is mandatory.
> > 
> > Test: https://codereview.chromium.org/500363002
> > Test depends on: https://codereview.chromium.org/544453004/
> > 
> > BUG=410790
> > 
> > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181635
> 
> TBR=eseidel@chromium.org,kouhei@chromium.org,eustas@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=410790
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181664

TBR=eseidel@chromium.org,kouhei@chromium.org,eustas@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=410790

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

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

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

BUG=408075
TBR=eae@chromium.org

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

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

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

BUG=408075
TBR=eae@chromium.org

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

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

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

BUG=408075
TBR=eae@chromium.org

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

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

Revert of HTMLConstructionSite: avoid n^2 running time for large scripts. (patchset #5 id:80001 of https://codereview.chromium.org/494993002/)

Reason for revert:
Speculative revert.

It may break the following blink sheriff bot:
http://build.chromium.org/p/chromium.webkit/builders/Android%20Tests%20%28dbg%29/builds/21403



Original issue's description:
> HTMLConstructionSite: avoid n^2 running time for large scripts.
> 
> Every time background parser sends chunk, tree is flushed.
> 
> If page contains very large script, then script node content is updated
> many times. Every update is causes string concatenation.
> 
> Solution: do not flush pending text until it is mandatory.
> 
> Test: https://codereview.chromium.org/500363002
> Test depends on: https://codereview.chromium.org/544453004/
> 
> BUG=410790
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181635

TBR=eseidel@chromium.org,kouhei@chromium.org,eustas@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=410790

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

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

BUG=397889

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

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

Follow up r181646 and make PromiseTracker work better with Oilpan.

TBR=oilpan-reviews
BUG=340522
NOTRY=true

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

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

This CL removes the old code needed to make the blink roll happen properly.

This is part 3 of 3, depends on:
https://codereview.chromium.org/518583003/

BUG=408426, 406336

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

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

In r174431 from May this year we changed the underline position and
thickness logic to respect the underline data specified in the font.
While this improved rendering for certain fonts and scripts it did cause
a regression for others. This change reverts to the old behavior while
we figure out how to handle fonts with inaccurate or undesirable
metrics.

BUG=408075
TBR=keishi@chromium.org

Committed as r181591 and reverted in r181607 due to missing XP
expectations.

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

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

BUG=none
TBR=ager,glider
NOTRY=true

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

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

Removes unused ModuleProxy, EventInterfaces.h, etc.

BUG=235436

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

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

The other media control painting methods do handle a null node in
toParentMediaElement (as can happen with a pseudo-element, such as
::-webkit-scrollbar). The slider thumb painting should too.

There are currently unfounded assertions that this node is not null,
but author style can make it null, so a proper check is appropriate.

A layout test has been added for this case.

BUG=412154

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

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

Those two files were split apart and redundantly copied
for non-Mac and Mac platforms. This is an attempt an unifying
them, as a preparation for removing the CoreText shaper.
We can merge the implementations in FontPlatformDataHarfBuzz.cpp
and FontPlatformData.cpp as a next step.

This is a reland after the change got reverted again
in 7443ae0830de1c3483 due to Windows build breakge.

BUG=334269, 411287
R=eae,arv

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

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

R=ager@chromium.org,haraken@chromium.org
BUG=340522

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

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

Depends on https://codereview.chromium.org/508873005/.

BUG=406969
R=ojan@chromium.org

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

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

Relies on on async magnitude-perf API:
https://codereview.chromium.org/544453004/

Tests fix:
https://codereview.chromium.org/494993002/

BUG=410790

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

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

BUG=373885

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

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

Drive-by fixes:
1) Fix constants documentation detection
2) Fix inherited Function methods problem

BUG=391593

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

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

Although I cannot reproduce the crash, as far as I see the crash report,
it seems that we are missing a handle scope in ScriptController::shouldBypassMainWorldCSP.

BUG=411773
TEST=None, I cannot reproduce

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

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

BUG=None

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

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

Revert "Revert of [oilpan]: optimize the way we allocate persistent handles in wrappers. (patchset #6 id:100001 of https://codereview.chromium.org/52535 3002/)"

Reland the wrapper persistent change using placement new for the constructor and no use of destructors. Also when constructing the object we make no assumption about the content of the slot.

This reverts commit 1738b5c841f11dd45dadefcd7b3e8ab6bab4eccb.

R=ager@chromium.org, fdegans@chromium.org, haraken@chromium.org, oilpan-reviews@chromium.org, zerny@chromium.org
BUG=411240

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

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

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

BUG=344873
TBR=habib.virji@samsung.com

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

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

Also remove the logSetter() overload taking an old value parameter from
the WebDOMActivityLogger interface, since there are no calls to it.

BUG=388287

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

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

1) The results were skewed by scripts from the same resource being ran several
times (so we saw times > 10 s).

2) Script run time was included in the measurement, and it shouldn't be.

WebCore.Scripts.Deferred.TimeBetweenLoadedAndCompiled is removed for now (to be
fixed later, when it's really needed). Tracking whether a deferred script was
already loaded when requested is more complicated.

BUG=

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

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

Reason: As discussed with @haraken over IRC, we need to migrate to using chromium
resource, grd, system. In that we may need to think over how to use the include directive.

One of the harmony modules, [1], describes about using the 'import "module-name"' directive.
We could consider using that as well. As this needs some discussion, we decided to revert
this patch first and reconsider landing this once the formal decision is made about it.

R=haraken@chromium.org, yosin@chromium.org

[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import

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

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

BUG=391593

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

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