GitLab

Oilpan is enabled on ToT, and is reading from TestExpectations and
has no Expectations file of its own (for now).

Mark the test as a flake and place it in TestExpectations "Oilpan"
area of that file.

BUG=581974
TBR=kojii@chromium.org
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372066}

Move shape-outside/webkit-clip-path property
handling from LegacyCSSPropertyParser into CSSPropertyParser.
To make this possible I converted the code for
parsing <shape-box> and <basic-shape>.

Also add code that was missing for handling
repeating-linear-gradient().

BUG=499780

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

Cr-Commit-Position: refs/heads/master@{#372065}

Oilpan is enabled on ToT, and is reading from TestExpectations and
has no Expectations file of its own (for now).

Mark the test as a flake and place it in TestExpectations "Oilpan"
area of that file.

TBR=haraken@chromium.org,ksakamoto@chromium.org
BUG=299010
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372064}

This strips out a lot of dated content, focuses more
on practical usage, and significantly shortens the doc.

Less yawning for everyone!

BUG=None
TBR=benwells@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#372063}

TBR=haraken@chromium.org
NOPRESUBMIT=true
NOTRY=true
BUG=581987

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

Cr-Commit-Position: refs/heads/master@{#372062}

Revert of [mojo] Enable ServiceRegistryImpl to override remote services (patchset #12 id:220001 of https://codereview.chromium.org/1470153002/ )

Reason for revert:
Causing Oilpan leak bot failure

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Leak/builds/17113

Original issue's description:
> [mojo] Enable ServiceRegistryImpl to override remote services
>
> This adds a new AddServiceOverride method to ServiceRegistryImpl.
> The purpose of this is to allow testing code to intercept remote
> service connections and opaquely provide a mock implementation of
> any service.
>
> The new interface is also added to the ServiceRegistry's JS
> bindings.
>
> Committed: https://crrev.com/9b0d7173af550086d5688137752a6f70109e9c98
> Cr-Commit-Position: refs/heads/master@{#371736}

TBR=jam@chromium.org,reillyg@chromium.org,rockot@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

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

Cr-Commit-Position: refs/heads/master@{#372061}

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

Cr-Commit-Position: refs/heads/master@{#372060}

virtual/threaded/printing/ellipsis-printing-style.html
virtual/threaded/printing/iframe-print.html
printing/ellipsis-printing-style.html
printing/iframe-print.html

Started happening after the rebasleines for the associated
CL [1] landed.

[1] https://code.google.com/p/chromium/issues/detail?id=390040#c14

TBR=wkorman@chromium.org,chrishtr@chromium.org
BUG=390040
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372059}

Revert of Let notifyScriptLoadError() handle already detached ScriptLoaders. (patchset #3 id:40001 of https://codereview.chromium.org/1644483002/ )

Reason for revert:
The change here assumes PendingScripts are separately allocated objects, an M50 change. This makes for more difficult backporting. Unnecessarily so.

Hence reverting and will reland a variation ( https://codereview.chromium.org/1642863002/ ) that works for older branches.

Original issue's description:
> Let notifyScriptLoadError() handle already detached ScriptLoaders.
>
> If a ScriptRunner has been disposed of already, allow ScriptLoaders
> to notify of their failure without asserting.
>
> R=haraken
> BUG=570012
>
> Committed: https://crrev.com/e7bf58190483dffac8e78506884170720165b198
> Cr-Commit-Position: refs/heads/master@{#371772}

TBR=haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570012

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

Cr-Commit-Position: refs/heads/master@{#372058}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/e85d78f6..bbcc7be6

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#372057}

This reverts commit 13eeea65.

The original CL was suspected of causing an 18.4% regression in
dromaeo.domcoreattr at 370280:370316 on Android Galaxy s5, so reverting
this to see if the bots recover.

reverts: https://codereview.chromium.org/1602843003

BUG=580981

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

Cr-Commit-Position: refs/heads/master@{#372056}

Add WebFont.FontDisplayValue histogram that measures which fallback mode
is specified in font-display @font-face descriptor.

BUG=564447

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

Cr-Commit-Position: refs/heads/master@{#372055}

This change adds trace events to ThreadedDataProvider. Even though the
feature is still disabled by default, having trace events helps us
investigate the behavior locally.

BUG=581992, 277124
R=kinuko@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#372054}

The image encoder runs on a background worker thread, which is not attached to Oilpan.
Thus CanvasAsyncBlobCreator::m_callback touched by the image encoder must use CrossThreadPersistent.

BUG=581877

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

Cr-Commit-Position: refs/heads/master@{#372053}

Disable the test because it fails on Linux. See bug for details.

TBR=kelvinp@chromium.org
BUG=582005
notry=true

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

Cr-Commit-Position: refs/heads/master@{#372052}

On iOS, we know whether the page has displayed insecure content; this CL
propagates that information to SecurityStateModel::VisibleSecurityState.

As far as I can tell, the resulting value isn't used by anything on the iOS
port, so this CL will not result in any visible change at the current time.

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

Cr-Commit-Position: refs/heads/master@{#372051}

Revert of Add deprecation message for -webkit-background-composite (patchset #11 id:200001 of https://codereview.chromium.org/1585383003/ )

Reason for revert:
Broke fast/backgrounds/webkit-background-composite-deprecated.html most bots.  Example:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.6/builds/48981

Original issue's description:
> Add deprecation message for -webkit-background-composite
>
> This patch is two fold:
> 1. Adds the capability of adding deprecation messages for CSSPropertyIDs
> as they are parsed.
> 2. Adds the relevant deprecation message for
> -webkit-background-composite. The reasons for which are highlighted here
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/thxrbc8XOmc
>
> BUG=498588
>
> Committed: https://crrev.com/6d2d7c3b60d3e5cd9b2efd7120cae737ad1e4329
> Cr-Commit-Position: refs/heads/master@{#372040}

TBR=alancutter@chromium.org,timloh@chromium.org,nainar@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=498588

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

Cr-Commit-Position: refs/heads/master@{#372050}

The assert that is firing is very concerning and is still being
investigated.

BUG=577121

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

Cr-Commit-Position: refs/heads/master@{#372049}

This patch fixs the issue that Chrome URL bar's overlay in SurfaceFlinger
becomes fullscreen after pull to refresh. It will reduce memory bandwidth.

BUG=581626

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

Cr-Commit-Position: refs/heads/master@{#372048}

Cr-Commit-Position: refs/heads/master@{#372047}

The CVDisplayLink is tied to a ScopedTypeRef data member of DisplayLinkMac.
However, there may be other references. Since there's no guarantee that
~DisplayLinkMac() will invoke the last CVDisplayLinkRelease(..) and clear
the callback automatically, clear it explicitly in ~DisplayLinkMac.

BUG=564780

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

Cr-Commit-Position: refs/heads/master@{#372046}

There are two element style recalc counters, one in Document, and one
in StyleResolver. The one in StyleResolver includes pseudo elements in
that count. We are unifying thesie counts keeping pseudo elements
included which means that the element recalc count will be higher for
tracing and the inspector, but unchanged for
internals.updateStyleAndReturnAffectedElementCount().

Also the StyleResolverStats is moved from StyleResolver to StyleEngine.
This means the stats will survive a clearResolver().

This change is motivated around item 6 and 9 in the design document for
crbug.com/401359.

R=dstockwell@chromium.org
BUG=401359

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

Cr-Commit-Position: refs/heads/master@{#372045}

Originally only sysex permission requests are exposed to the
content layer, but with this series of patches, all Web MIDI requests
will be exposed to the content layer so that each chromium embedder
can decide how to decide several permissions.

First of all, expose all requests outside Blink.
Changes for the permission service will follow as a separate patch.

BUG=535181

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

Cr-Commit-Position: refs/heads/master@{#372044}

Note: not adding a seperate Oilpan entry, just mark it flaky while
Oilpan is enabled on ToT.

TBR=eae@chromium.org,harken@chromium.org
BUG=581990
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372043}

As per the CPP style guide, variables declared constexpr or const, and whose
value is fixed for the duration of the program, are named with a leading "k"
followed by mixed case. Since objects don't have a constant value for the
duration of the program, refactor "kTestString" to "test_string".

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

Cr-Commit-Position: refs/heads/master@{#372042}

BUG=580928

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

Cr-Commit-Position: refs/heads/master@{#372041}

This patch is two fold:
1. Adds the capability of adding deprecation messages for CSSPropertyIDs
as they are parsed.
2. Adds the relevant deprecation message for
-webkit-background-composite. The reasons for which are highlighted here
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/thxrbc8XOmc

BUG=498588

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

Cr-Commit-Position: refs/heads/master@{#372040}

Notification type SIMPLE is for the web notification actually and it triggers some hacks which causes the problem (see the issue).

This patch changes the type to BASE_FORMAT without hacks.

BUG=b/26831435

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

Cr-Commit-Position: refs/heads/master@{#372039}

TBR=rockot@chromium.org
BUG=581987
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372038}

I2S: https://groups.google.com/a/chromium.org/d/msg/blink-dev/_nu6HlbNQfo/XzaLNb1bBgAJ

BUG=471199

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

Cr-Commit-Position: refs/heads/master@{#372037}

Currently dialog boxes do not take main status from the parent window.
Change this so that any child window does not become a main window.

Also, use the same trick that Cocoa bubbles use to avoid the traffic
lights dimming, but only do this for child windows that don't have their
own traffic lights.

BUG=543689

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

Cr-Commit-Position: refs/heads/master@{#372036}

We store a flag for finding ::before or ::after in
InvalidationSetFeatures, yet they are not added as features to
invalidation sets. That means we need to handle *::before as a universal
selector and cause subtree invalidations.

R=ericwilligers@chromium.org
BUG=581675

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

Cr-Commit-Position: refs/heads/master@{#372035}

Revert of DevTools: remove V8ScriptRunner use from InjectedScript*. (patchset #2 id:20001 of https://codereview.chromium.org/1641933002/ )

Reason for revert:
http/tests/inspector/resource-tree/resource-tree-non-unique-url.html started failing

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%2032/bui...

Reverting to see if we can get that test to pass again.

Original issue's description:
> DevTools: remove V8ScriptRunner use from InjectedScript*.
>
> BUG=580337
>
> Committed: https://crrev.com/ec279b9d63c3a96d237495967faba97be76597eb
> Cr-Commit-Position: refs/heads/master@{#372020}

TBR=dgozman@chromium.org,jochen@chromium.org,pfeldman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=580337

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

Cr-Commit-Position: refs/heads/master@{#372034}

cwallez@ pointed out the file was unmaintainable. Generate it from a
script instead. Effort was expended to minimize the initial changes. The
only changes are:

1. webgl_conformance_tests and webgl2_conformance_tests are swapped.
2. The D3D9 and GL versions of gles2_conform_test on the Windows testers
   come first.
3. content_unittests was missed on several testers, and is now present.

This script will be generalized later to create the forthcoming
chromium.gpu.json.

BUG=542370
TBR=dpranke@chromium.org
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372033}

Currently, Mac10.7 & Linux & Linux Oilpan [ Debug ] to be precise.

Note not adding a seperate Oilpan entry, but be aware this test is
failing on Linux Oilpan Debug after the related change [1].

[1] https://codereview.chromium.org/1623073003

TBR=haraken@chromium.org,fmalita@chromium.org
BUG=581978
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372032}

This matches behavior of ImageView::SetImage.

BUG=581965

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

Cr-Commit-Position: refs/heads/master@{#372031}

Before this patch, the serviceworker-internals shows a render process host ID in
"Renderer process ID" entry. This is not really helpful for blink/chromium
developers to debug errors related to renderer process activity.

After this patch, the internal page shows a real render process ID in the entry.

BUG=544760

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

Cr-Commit-Position: refs/heads/master@{#372030}

Oilpan is enabled on ToT, and is reading from TestExpectations and
has no Expectations file of its own (for now).

Mark the test as a flake and place it in TestExpectations "Oilpan"
area of that file.

TBR=haraken@chromium.org,ksakamoto@chromium.org
BUG=299010
NOPRESUBMIT=true
NOTRY=true

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

Cr-Commit-Position: refs/heads/master@{#372029}

BUG=514857,581604
TBR=bpastene@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#372028}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/0085d5c64509..a7e38c146fe9

$ git log 0085d5c64..a7e38c146 --date=short --no-merges --format='%ad %ae %s'

CQ_INCLUDE_TRYBOTS=
TBR=catapult-sheriff@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#372027}