GitLab

This test was failing on Android Nexus 5 device due to the loading taking
too long. Simplifying its test case seems to fix the problem.

BUG=715685
TBR=wangxianzhu@chromium.org

Review-Url: https://codereview.chromium.org/2849153002
Cr-Commit-Position: refs/heads/master@{#468314}

Since we didn't install them, we can't update them.
BUG=714740

Review-Url: https://codereview.chromium.org/2847973002
Cr-Commit-Position: refs/heads/master@{#468313}

These were tests for WebSocket, XHR and fetch bandwidth. They produced a large
number of false positives and so had been disabled on the bots. Since there is
no-one available to care for these tests, remove them.

BUG=551950,549017,715500

Review-Url: https://codereview.chromium.org/2844633002
Cr-Commit-Position: refs/heads/master@{#468312}

Removing as the CHECKs were not triggered and bug was resolved.

BUG=699269

Review-Url: https://codereview.chromium.org/2850133003
Cr-Commit-Position: refs/heads/master@{#468311}

We WebViewFrameWidget keeps a RefPtr to WebViewImpl we need to make WebViewBase
ref counted. Removed the ref counted from WebViewImpl when moving it to
WebViewBase.

BUG=712963

Review-Url: https://codereview.chromium.org/2850113002
Cr-Commit-Position: refs/heads/master@{#468310}

RequestInit::header is supposed to hold a HeadersInit (ie. just a WebIDL
union) object, not a fully-fledged Headers one.

Make it so, and add a HeadersInit overload to Headers::FillWith() so all the
checks for which union type is being used are centralized in one place.
Additionally, this also makes it easier to finally define RequestInit with
an IDL file in the future.

R=tyoshino@chromium.org,yhirano@chromium.org

Review-Url: https://codereview.chromium.org/2840193002
Cr-Commit-Position: refs/heads/master@{#468309}

Production files should not be in javatest/.

BUG=None

Review-Url: https://codereview.chromium.org/2843333003
Cr-Commit-Position: refs/heads/master@{#468308}

The work continues the effort to remove direct dependencies on web/WebViewImpl.h
by moving methods to a pure virtual base class, and updating references to use
the base class rather than the implementation.

We need to break cyclic dependencies in web/ to allow us to move files up into
core/ and modules/.

BUG=712963

Review-Url: https://codereview.chromium.org/2851993002
Cr-Commit-Position: refs/heads/master@{#468307}

In verified-boot mode, ftrace trace_marker is not allowed for writting
due to security concern. As an altenative, we use a socket for processes in
Android to write trace events, and send back the trace data via Mojo
handle.

All change set includes:
- http://crrev.com/2400163003 (Chromium)
- http://ag/2028075 (Android system/core)
- http://ag/1738271 (Android device/google/cheets2)

BUG=653795
TEST=Run chrome://tracing and see events from Android.

Review-Url: https://codereview.chromium.org/2400163003
Cr-Commit-Position: refs/heads/master@{#468306}

BUG=n/a

Review-Url: https://codereview.chromium.org/2847983002
Cr-Commit-Position: refs/heads/master@{#468305}

This patch prunes a call site of the deprecated function to improve
code health.

The change is safe because the call site only needs the selection type,
not the selection itself. After this patch, it also aligns with the
code in Editor::RespondToChangedSelection.

BUG=698633
TEST=n/a; no behavioral change

Review-Url: https://codereview.chromium.org/2852913002
Cr-Commit-Position: refs/heads/master@{#468304}

Reland of Disable flaky ExperimentalAppWindowApiTest.SetIcon (patchset #1 id:1 of https://codereview.chromium.org/2855563002/ )

Reason for revert:
Build failure causing original revert seems to have been a flake.

Original issue's description:
> Revert of Disable flaky ExperimentalAppWindowApiTest.SetIcon (patchset #1 id:1 of https://codereview.chromium.org/2849073003/ )
>
> Reason for revert:
> Speculatively reverting due to build failure on Mac https://build.chromium.org/p/chromium/builders/Mac/builds/26841
>
> Original issue's description:
> > Disable flaky ExperimentalAppWindowApiTest.SetIcon
> >
> > TBR=benwells@chromium.org
> > BUG=716726
> >
> > Review-Url: https://codereview.chromium.org/2849073003
> > Cr-Commit-Position: refs/heads/master@{#468285}
> > Committed: https://chromium.googlesource.com/chromium/src/+/78552aab8b749f33cd8176a977bdff4fa7d5de50
>
> TBR=benwells@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=716726
>
> Review-Url: https://codereview.chromium.org/2855563002
> Cr-Commit-Position: refs/heads/master@{#468289}
> Committed: https://chromium.googlesource.com/chromium/src/+/dfe887bdaf1ea9fb0b3a7566dbdde620c869d42c

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

Review-Url: https://codereview.chromium.org/2856523003
Cr-Commit-Position: refs/heads/master@{#468303}

CL#1 This snackbar will be used to convey to the user the result
of any action performed on the Translate infobar.

BUG=713514
TBR=groby@chromium.org

Review-Url: https://codereview.chromium.org/2838833003
Cr-Commit-Position: refs/heads/master@{#468302}

Reland of CSS Motion Path: delete implementation of motion-offset property (patchset #1 id:1 of https://codereview.chromium.org/2856513002/ )

Reason for revert:
https://codereview.chromium.org/2841863005 was extremely unlikely to be the root cause of Mac-specific failures.

Original issue's description:
> Revert of CSS Motion Path: delete implementation of motion-offset property (patchset #2 id:40001 of https://codereview.chromium.org/2841863005/ )
>
> Reason for revert:
> Speculatively reverting due to build failure on Mac https://build.chromium.org/p/chromium/builders/Mac/builds/26841
>
> Original issue's description:
> > CSS Motion Path: delete implementation of motion-offset property
> >
> > The 'motion-offset' property has been replaced by 'offset-distance'.
> >
> > 'motion-offset' is among the properties disabled in M58.
> >
> > Intent to Remove: https://groups.google.com/a/chromium.org/d/msg/blink-dev/o1C5NzGf9Q0/sbaLF1MbAgAJ
> >
> > BUG=711937
> >
> > Review-Url: https://codereview.chromium.org/2841863005
> > Cr-Commit-Position: refs/heads/master@{#468287}
> > Committed: https://chromium.googlesource.com/chromium/src/+/612ac411762985a97166c4144ff0d049716a80db
>
> TBR=bugsnash@chromium.org,meade@chromium.org,ericwilligers@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=711937
>
> Review-Url: https://codereview.chromium.org/2856513002
> Cr-Commit-Position: refs/heads/master@{#468291}
> Committed: https://chromium.googlesource.com/chromium/src/+/d62951aca80fa961d7feb4c05fe62b0893f59bfd

TBR=bugsnash@chromium.org,meade@chromium.org,suzyh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=711937

Review-Url: https://codereview.chromium.org/2856523002
Cr-Commit-Position: refs/heads/master@{#468301}

Reland of Disable flaky CommandLineExtensionsDontLoad test (patchset #1 id:1 of https://codereview.chromium.org/2855553002/ )

Reason for revert:
Build failure causing original revert seems to have been a flake.

Original issue's description:
> Revert of Disable flaky CommandLineExtensionsDontLoad test (patchset #1 id:1 of https://codereview.chromium.org/2852933002/ )
>
> Reason for revert:
> Speculatively reverting due to build failure on Mac https://build.chromium.org/p/chromium/builders/Mac/builds/26841
>
> Original issue's description:
> > Disable flaky CommandLineExtensionsDontLoad test
> >
> > TBR=emaxx@chromium.org
> > BUG=716727
> >
> > Review-Url: https://codereview.chromium.org/2852933002
> > Cr-Commit-Position: refs/heads/master@{#468286}
> > Committed: https://chromium.googlesource.com/chromium/src/+/91f64acccbfe592af367f79ae74649d520528f0b
>
> TBR=emaxx@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=716727
>
> Review-Url: https://codereview.chromium.org/2855553002
> Cr-Commit-Position: refs/heads/master@{#468290}
> Committed: https://chromium.googlesource.com/chromium/src/+/606b2b5d25564f187a843d25e5c2998d4ad2c04e

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

Review-Url: https://codereview.chromium.org/2848223002
Cr-Commit-Position: refs/heads/master@{#468300}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/489561d2..df06cc9f

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.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

Review-Url: https://codereview.chromium.org/2852943002
Cr-Commit-Position: refs/heads/master@{#468299}

Revert of Changing default Windows compiler to VS 2017 (patchset #1 id:1 of https://codereview.chromium.org/2852433005/ )

Reason for revert:
Reverting as planned - it was just a test switch to VS 2017, which seems to have gone smoothly. I'm hopeful that canary 60.0.3086.0 is being built with VS 2017, to
complete the test.

Original issue's description:
> Changing default Windows compiler to VS 2017
>
> This is a single character change to temporarily switch Chrome to
> building with VC++ 2017. This CL is currently purely for testing
> purposes and will be landed for the weekend in order to flesh out any
> remaining bugs.
>
> This is a retry of crrev.com/2762093003 which failed due to a compiler
> warning which was missed in local testing and the try bots. That warning
> has been resolved and a local build of 'all' completed cleanly.
>
> BUG=683729
>
> Review-Url: https://codereview.chromium.org/2852433005
> Cr-Commit-Position: refs/heads/master@{#468239}
> Committed: https://chromium.googlesource.com/chromium/src/+/060d293f1d87d450a824dc46f564792272ed34a3

TBR=thakis@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=683729

Review-Url: https://codereview.chromium.org/2851143002
Cr-Commit-Position: refs/heads/master@{#468298}

This fix will set translate infobar with a translating status in the beginning, then highlighted the targeted language tab after translated.

Tested on my device

BUG=703887

Review-Url: https://codereview.chromium.org/2845013002
Cr-Commit-Position: refs/heads/master@{#468297}

- Apply color and line-height to the whole page
- Set iron-icon-width/height globally, reducing the number of places
  where specific sizing is necessary
- Import cr-shared-style into Bookmarks' shared style module, which
  removes the need to specify some button styles
- Use paper-icon-button-light in folder-node, which simplifies CSS and
  slightly improves performance
- Make item highlight colors consistent

BUG=715394
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2846673003
Cr-Commit-Position: refs/heads/master@{#468296}

AnimationEffectTiming.idl and AnimationEffectTimingReadOnly.idl
now use
attribute PlaybackDirection direction;
for consistency with the spec.

https://w3c.github.io/web-animations/#the-animationeffecttiming-interface
https://w3c.github.io/web-animations/#the-animationeffecttimingreadonly-interface

BUG=716963

Review-Url: https://codereview.chromium.org/2856503002
Cr-Commit-Position: refs/heads/master@{#468295}

The parser will in some cases create new elements in documents that
have become frame-detached. Account for that -- no execution context
due to the document having become detached -- when processing the
event listeners of an event attribute.

R=haraken
BUG=714353

Review-Url: https://codereview.chromium.org/2855443002
Cr-Commit-Position: refs/heads/master@{#468294}

Currently, when we generate non bit fields, we order them in
alphabetical order. This may result in unnecessary padding, e.g.

struct {
  char a;
  /* char padding[3] */
  int b;
  char c;
  /* char padding[3] */
};

See [1] for an explanation.

This patch defines an ordering on field types so that we can sort
non bit fields from largest alignment size to smallest. This heuristic
minimises the padding most of the time:

struct {
  int b;
  char a;
  char c;
  /* char padding[2] */
};

This is purely a memory optimisation, not speed. It does not take into
account data access patterns and cache locality. We use the data from
[2] to define the order of types.

With this heuristic, some of the generated code has changed, but it is
unlikely to have changed the size of any structs.

Diff of generated files:
https://gist.github.com/darrnshn/81d146c2696ef100d56309e5b4b529ff/revisions

[1] http://www.drdobbs.com/cpp/padding-and-rearranging-structure-member/240007649?pgno=2
[2] https://codereview.chromium.org/2841413002

BUG=628043

Review-Url: https://codereview.chromium.org/2844223002
Cr-Commit-Position: refs/heads/master@{#468293}

BUG=715387
TEST=LayoutTests/editing/selection/selectstart_detaches_frame.html

Review-Url: https://codereview.chromium.org/2848993002
Cr-Commit-Position: refs/heads/master@{#468292}

Revert of CSS Motion Path: delete implementation of motion-offset property (patchset #2 id:40001 of https://codereview.chromium.org/2841863005/ )

Reason for revert:
Speculatively reverting due to build failure on Mac https://build.chromium.org/p/chromium/builders/Mac/builds/26841

Original issue's description:
> CSS Motion Path: delete implementation of motion-offset property
>
> The 'motion-offset' property has been replaced by 'offset-distance'.
>
> 'motion-offset' is among the properties disabled in M58.
>
> Intent to Remove: https://groups.google.com/a/chromium.org/d/msg/blink-dev/o1C5NzGf9Q0/sbaLF1MbAgAJ
>
> BUG=711937
>
> Review-Url: https://codereview.chromium.org/2841863005
> Cr-Commit-Position: refs/heads/master@{#468287}
> Committed: https://chromium.googlesource.com/chromium/src/+/612ac411762985a97166c4144ff0d049716a80db

TBR=bugsnash@chromium.org,meade@chromium.org,ericwilligers@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=711937

Review-Url: https://codereview.chromium.org/2856513002
Cr-Commit-Position: refs/heads/master@{#468291}

Revert of Disable flaky CommandLineExtensionsDontLoad test (patchset #1 id:1 of https://codereview.chromium.org/2852933002/ )

Reason for revert:
Speculatively reverting due to build failure on Mac https://build.chromium.org/p/chromium/builders/Mac/builds/26841

Original issue's description:
> Disable flaky CommandLineExtensionsDontLoad test
>
> TBR=emaxx@chromium.org
> BUG=716727
>
> Review-Url: https://codereview.chromium.org/2852933002
> Cr-Commit-Position: refs/heads/master@{#468286}
> Committed: https://chromium.googlesource.com/chromium/src/+/91f64acccbfe592af367f79ae74649d520528f0b

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

Review-Url: https://codereview.chromium.org/2855553002
Cr-Commit-Position: refs/heads/master@{#468290}

Revert of Disable flaky ExperimentalAppWindowApiTest.SetIcon (patchset #1 id:1 of https://codereview.chromium.org/2849073003/ )

Reason for revert:
Speculatively reverting due to build failure on Mac https://build.chromium.org/p/chromium/builders/Mac/builds/26841

Original issue's description:
> Disable flaky ExperimentalAppWindowApiTest.SetIcon
>
> TBR=benwells@chromium.org
> BUG=716726
>
> Review-Url: https://codereview.chromium.org/2849073003
> Cr-Commit-Position: refs/heads/master@{#468285}
> Committed: https://chromium.googlesource.com/chromium/src/+/78552aab8b749f33cd8176a977bdff4fa7d5de50

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

Review-Url: https://codereview.chromium.org/2855563002
Cr-Commit-Position: refs/heads/master@{#468289}

BUG=695635

Review-Url: https://codereview.chromium.org/2844133003
Cr-Commit-Position: refs/heads/master@{#468288}

The 'motion-offset' property has been replaced by 'offset-distance'.

'motion-offset' is among the properties disabled in M58.

Intent to Remove: https://groups.google.com/a/chromium.org/d/msg/blink-dev/o1C5NzGf9Q0/sbaLF1MbAgAJ

BUG=711937

Review-Url: https://codereview.chromium.org/2841863005
Cr-Commit-Position: refs/heads/master@{#468287}

TBR=emaxx@chromium.org
BUG=716727

Review-Url: https://codereview.chromium.org/2852933002
Cr-Commit-Position: refs/heads/master@{#468286}

TBR=benwells@chromium.org
BUG=716726

Review-Url: https://codereview.chromium.org/2849073003
Cr-Commit-Position: refs/heads/master@{#468285}

Recently imported
external/wpt/WebCryptoAPI/generateKey/test_successes_RSA-OAEP.https.html
is occasionally timing out, so adding to TestExpectations.

TBR=eroman@chromium.org
BUG=716729

Review-Url: https://codereview.chromium.org/2852923002
Cr-Commit-Position: refs/heads/master@{#468284}

This adds tests to PermissionDialogTests that test Media requests with
the kUsePermissionManagerForMediaRequests feature enabled. When this
feature is enabled, media permissions can be requested in the same way
as other permissions.

Once the feature flag is removed, the old behavior that is being tested
can also be removed.

BUG=596786

Review-Url: https://codereview.chromium.org/2838873004
Cr-Commit-Position: refs/heads/master@{#468283}

IndexedDBBrowserTest.CursorTestIncognito is flaky on
linux_android_rel_ng.

TBR=jsbell@chromium.org
BUG=716728

Review-Url: https://codereview.chromium.org/2853753002
Cr-Commit-Position: refs/heads/master@{#468282}

This CL fixes an issue where drag and drop wouldn't work on Mac because
the drop data was being cleared by the bookmark manager API's drop
handler before the bookmark manager's handler had handled the drop. This
has been fixed by adding a setTimeout to clearing the data, which was
also in the old bookmark manager, giving the bookmark manager a chance
to deal with the drop.

BUG=716257
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2845243003
Cr-Commit-Position: refs/heads/master@{#468281}

added a headlessresourcedispatcherhostdelegate so that we do not crash when a website requests a client cert store

BUG=714534

Review-Url: https://codereview.chromium.org/2852633002
Cr-Commit-Position: refs/heads/master@{#468280}

When we generate ComputedStyle, we differentiate between properties
(which are meant to correspond to CSS properties) and nonproperties
(everything else). However, this terminology is confusing:

- Some nonproperties are derived from real CSS properties (e.g.
  hasSimpleUnderline).
- The generator code contains a lot of "for property_ in nonproperties"
  which is quite confusing.
- The name does not reveal the actual difference between properties
  and nonproperties in terms of code generation.

This patch removes the field role of 'nonproperty' in favour of the
parameter 'has_custom_compare_and_copy'. So there are only two field
roles: 'property' (which includes flags like 'unique' and 'emptyState')
and 'inherited_flag'.

BUG=628043

Review-Url: https://codereview.chromium.org/2830983003
Cr-Commit-Position: refs/heads/master@{#468279}

Clusterfuzz found a sample where
CompositeEditCommand::MoveParagraphWithClones() moves |before_paragraph|
out of the DOM tree after moving the paragraph. This patch uses
RelocatablePosition to track |before_paragraph| and |after_paragraph| so
that the positions we track are always connected.

Note: CompositeEditCommand::MoveParagraphs() is already using
RelocatablePosition to track positions before and after the moved
paragraphs.

BUG=712510
TEST=ApplyBlockElementCommandTest.IndentHeadingIntoBlockquote

Review-Url: https://codereview.chromium.org/2850773003
Cr-Commit-Position: refs/heads/master@{#468278}

This patch prunes some call sites of the deprecated function to improve
code health. The change is safe because layout has been updated a few
lines before each call site.

BUG=698633
TEST=n/a; no behavioral change

Review-Url: https://codereview.chromium.org/2847813005
Cr-Commit-Position: refs/heads/master@{#468277}

Add a reftest using an additional axis VM2B in the variabletest_box
font. The VM2B axis shifts the mark anchor to the left, leading to the
combining mark below box shifting right when the axis parameter is
adjusted from 0 to 1000. As the reference in the ref test, the font now
contains two additional glyphs N and O, which have fixed base anchors at
the middle and right position. A rendering of the M character showing
"m2b pos", followed by COMBINING SQUARE BELOW with VM2B 0, 500, 1000
applied must thus be equal to renderings of the N and O glyph with no
VM2B variation values applied.

BUG=693065
R=behdad, kojii

Review-Url: https://codereview.chromium.org/2850743002
Cr-Commit-Position: refs/heads/master@{#468276}

This CL resolves TODO(kouhei) in ScriptModuleResolverImpl to rethrow instantiation error when
resolved module has failed to instantiate.

TEST=ScriptModuleResolverImplTest.ResolveInstantiationFailedModule
BUG=594639

Review-Url: https://codereview.chromium.org/2837253002
Cr-Commit-Position: refs/heads/master@{#468275}