GitLab

Runtime ShadowDOM flag was already set to true at r169201. This change shouldn't have any effects.

CSSSelector still uses this flag. I'll clean up CSSSelector in another patch.

BUG=336121

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

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

If you delete the documentElement then that also means all <style> and
<link> elements must have been removed from the document so the
StyleResolver should already be reset or cleared and we don't need to
call clearStyleResolver() from inside Document::childrenChanged.

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

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

This removes 6 FIXMEs, and removes duplicate calls of info.GetIsolate().

This updates the license block to the new form (since this is *generated* code),
and makes it automatically use the correct module name.
I didn't make the year update automatically, as otherwise r-b-t will break
every new year and this detail doesn't seem worth the hassle.

I also moved one long comment into a template comment, so it's not in the
generated code. This seems clearer (it's in the template), and reduces
duplication and noise.

Overall these reduce the size of the generated code and tests
by about 6% (generated code from 17 MB from 16 MB)!
Not a big deal, but makes them a bit faster to compile and easier to read.

R=haraken
BUG=345503

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

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

To conform the latest W3C spec, add sysexEnabled attribute.
http://www.w3.org/TR/webmidi/#idl-def-MIDIAccess

MIDIAccess originally has a method sysExEnabled().
To avoid having two functions for the same operation, this change
renames it to sysexEnabled(). This change affects other caller
implementations.

To rename everything in blink, it modifies blink API.
So this change is also the first step of blink API change.

Here are a series of changes.
1. this change
2. https://codereview.chromium.org/210003002/
3. https://codereview.chromium.org/208423016/

BUG=349538
TEST=LayoutTests/webmidi

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

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

This CL removes a histgram from GraphicsLayerUpdater::rebuildTree. This
histogram accounts for 9.2% of the time spent in
GraphicsLayerUpdater::rebuildTree and 1.6% of the total compositing update in
Polymer's calculator.

R=ojan@chromium.org

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

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

Perl cruft: Python compiler assumes a header file exists.
This removes a special case and simplifies CG logic by having stub header
files if needed.

R=haraken
BUG=341748

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

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

This is another attempt to investigate crashes in the wild.
This change deploys more RELEASE_ASSERT() to see if/where the
wrongly-killed Resource objects appear.

BUG=352043
TEST=none
R=eseidel, japhet@chromium.org, kouhei

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

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

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

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

Delete WebSocketHandle::connect(..., const WebString&, ...), which is replaced by
WebSocketHandle::connect(..., const WebSerializedOrigin&, ...).

BUG=339373

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

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

Replace hide() in ExternalPopupMenu:show() with
m_webExternalPopupMenu->close(). It means we remove
PopupMenuClient::popupDidHide() call.

We should not call it because the client changes its state to "no popup" state
by the popupDidHide call, and it won't call PopupMenu::hide() even if it's
necessary.

BUG=30550,145658

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

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

Linear easing was being supplied instead of the timing function specified by the user.

Example for testing: http://jsbin.com/sihif/1/quiet

Cubic bezier keyframe easings were being calculated without knowledge of the iteration duration. This led to inconsistency in the accuracy being requested.

We now always use the iteration duration when choosing the accuracy for timing function evaluation.

Without this, the transitions layout tests opacity-transform-transitions-inside-iframe.html and
cubic-bezier-overflow-length.html would have needed rebaselining.

BUG=352919

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

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

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

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

BUG=335345
R=eseidel@chromium.org

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

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

* Splits up large tests into smaller files.
* Extracts common key data to its own file
* Reduces use of shouldRejectPromiseWithNull()

BUG=245025

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

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

R=alancutter@chromium.org, jochen@chromium.org

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

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

Attempt 2.

Notry because we don't have an XP trybot.
NOTRY=TRUE
BUG=354849

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

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

The last consumer of PostAttachCallbacks is layer updating, this patch
switches away from the generic "call me after recalc style" system that
was PostAttachCallbacks to a very specialized system that only marks
elements as needing a style recalc for the purposes of creating and
destroying layers.

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

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

http/tests/serviceworker/registration.html
* was waiting on CR side to land

virtual/serviceworker/http/tests/serviceworker/registration.html
* was crashy but seems okay now (?)

virtual/serviceworker/http/tests/serviceworker/indexeddb.html
virtual/serviceworker/http/tests/serviceworker/postmessage.html
* were waiting on CR side to land

BUG=350103,352703,285976
TBR=alecflett

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

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

When the root renderer is fully collapsed with a background
image, the solid color background painting path would assume
that the background image would fill the viewport, but the
background image code would, correctly, only use the actual
height of the root and therefor not fill anything. The
result is the viewport isn't filled at all.

This is a regression from my early patch (r166582) that avoids
implicitly painting the viewport background when the root
itself will do so as an optimization to avoid rasterizing the entire
viewport twice. Catching the case in the solid color background
painting path. Behavior matches FireFox and Safari.

BUG=349936

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

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

Remove the SkiaFontWin.cpp and SkiaFontWin.h files and all references
to them as the paintSkiaText method provided by SkiaFontWin.h is no
longer used now that the Uniscribe code path has been removed.

R=schenney@chromium.org
BUG=313423

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

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

This attribute is already present on Element.idl. This change avoids extra bindings code
being written.

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

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

With Repaint-After-Layout we would repaint the SVG Root element a lot more
then needed. This ports the check for when to repaint the SVG Root element
into the LayoutRectRecorder to better match what the LayoutRepainting was doing
originally.

BUG=320139

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

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

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

This patch removes a compositing query assert in
FrameView::scrollbarExistenceDidChange because no compositing state is
queried through scrollbarExistenceDidChange.

No new tests.

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

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

Verify that a very large shape-margin value combined with a very small
shape-outside image defines a shape that matches the shape-outside
element's margin-box.

This is a merge of http://trac.webkit.org/changeset/166019 by
Hans Muller <hmuller@adobe.com>.

BUG=354917

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169768

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

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

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

Revert of Update image data upon changing the img.crossorigin attribute. (https://codereview.chromium.org/201143002/)

Reason for revert:
The re-issuing of another image load upon setting 'crossorigin' has been reported to cause a site problem (a Google internal site/service.)

Until diagnosed and understood fully, let's back this one out.

Original issue's description:
> Update image data upon changing the img.crossorigin attribute.
> 
> The spec tells us that setting the crossorigin attribute on an img
> element should cause the "update the image data" steps to be invoked,
> 
>  http://www.whatwg.org/specs/web-apps/current-work/#update-the-image-data
> 
> Do so here, along with updating the underlying test's expectation.
> 
> R=
> BUG=341047
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169382

TBR=adamk@chromium.org,abarth@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=341047

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

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

http://www.whatwg.org/specs/web-apps/current-work/#attr-script-type

  The type attribute gives the language of the script or format of the
  data. If the attribute is present, its value must be a valid MIME
  type. The charset parameter must not be specified. The default,
  which is used if the attribute is absent, is "text/javascript"."

Blink is less strict than other browsers. Removing the optional type
attribute enables tests to be run in other browsers, notably Firefox.

BUG=308992
NOTRY=true

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

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

These map to the eventnames 'beginEvent', 'endEvent' and 'repeatEvent'
respectively.

Spec:
https://svgwg.org/svg2-draft/animate.html#InterfaceSVGAnimationElement

BUG=338288
NOTRY=true

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

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

We were early'ing out, but not calling cache->setScrollParent. The result: the
cache thought it was dirty, but it wasn't. This CL fixes that.

BUG=TBD

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

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

Mark the following layout test as flaky on windows debug builds:
  virtual/threaded/animations/animation-hit-test-transform.html
  virtual/threaded/transitions/blendmode-transitions.html
  virtual/threaded/transitions/cubic-bezier-overflow-svg-length.html
  virtual/threaded/animations/compositor-start-event-timing.html

BUG=248938
TBR=ajuma@chromium.org

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

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

Remove paintSkiaText wrapper function that is no longer needed now that we always have SkTypeface.

R=bungeman@chromium.org

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

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

These files do not appear to be referenced in the build and can therefore be
deleted.

R=adamk@chromium.org

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

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

Remove has3DContent, layerHas3DContent
The last vestiges of these functions were removed with
https://src.chromium.org/viewvc/blink?revision=167695&view=revision
when Abarth removed isSoftwareRenderable.

Remove computeFixedViewportConstraints, computeStickyViewportConstraints
Jamesr removed the last callers in
https://src.chromium.org/viewvc/blink?view=rev&revision=147653

Remove clearMappingForAllRenderLayers, clearMappingForRenderLayerIncludingDescendants
These functions were added (under a different name) by Simon Fraser in
http://trac.webkit.org/changeset/90484. Jamesr removed the caller
in https://codereview.chromium.org/13818029.

Remove setCompositingParent, removeCompositedChildren
This code predates written history and possibly humankind.

Remove updateLayerTreeGeometry
Abarth removed the implementation but left the function definition in
https://src.chromium.org/viewvc/blink?view=rev&revision=168244

Remove updateGraphicsLayersMappedToRenderLayer
Abarth removed the implementation but left the function definition in
https://src.chromium.org/viewvc/blink?view=rev&revision=167697

Remove hasAnyAdditionalCompositedLayers
Shawn Singh removed the implementation but left the function definition
in https://src.chromium.org/viewvc/blink?view=rev&revision=163720

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

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

Before the change 'WTF::copyChars' contained an algorithm that was used
instead of 'memcpy' if the number of characters had been less than 20.

This however was unnecessary as 'memcpy' produces more efficient asm code
than the home-brewed algorithm did as it uses 'movq' (instead of 'movl')
doing the copying within the less number of asm commands (tested with
GCC 4.6.3).

So, this patch removes the home-brewed algorithm from 'WTF::copyChars'
and makes it always rely on 'memcpy'.

A simple testing programm doing the following code

    for (int i = 10000000; --i;) {
        // Re-create 'src' to prevent from optimizing cycle iterations out.
        char* src = new char[15];
        copyChars(dest, src, 15);
        copyChars(dest, src, 14);
        delete[] src;
    }

takes:

0m0.324s before the change
0m0.235s after the change (using 'memcpy').

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

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

BUG=353991
NOTRY=true

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

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

BUG=304341

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

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

Mark the virtual/threaded/animations/3d/transform-perspective.html test
as timing out on all debug bots as per flakiness dashboard data.

BUG=248938
TBR=ajuma@chromium.org

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

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

This patch updates User-Agent for Samsung Galaxy S4.

BUG=354920
TBR=pfeldman, apavlov

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

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

The user can search styles by property name, value, or rule selector.
Computed style properties are filtered, so that only matching ones are visible.

R=eustas, pfeldman, vsevik
BUG=278852

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

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