GitLab

There is a crash in DeviceDataManagerX11::UpdateDeviceList() that could
occur if there is no valuator information for the deviceid.

BUG=716422

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

This converts SlideOutView to SlideOutController, which provides all the
same functionality but can be added to any view rather than just those
that extend it.

It allows the target view to control the layer that's sliding out. For
toasts, that is the widget's layer. For notifications inside the message
center, that will continue to be the NotificationView itself.

BUG=716429

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

Revert of Implementing Binary glTF reader for the VR controller model (patchset #3 id:40001 of https://codereview.chromium.org/2852533004/ )

Reason for revert:
Test added in this patch fails on downstream bot (see http://crbug.com/717088)

BUG=717088

Original issue's description:
> Implementing Binary glTF reader for the VR controller model
>
> The binary glTF format enables us to deliver 3D models on a single file with minimal size.
>
> Binary glTF Specification in: https://github.com/KhronosGroup/glTF/tree/master/extensions/Khronos/KHR_binary_glTF
>
> BUG=644562
>
> Review-Url: https://codereview.chromium.org/2852533004
> Cr-Commit-Position: refs/heads/master@{#468198}
> Committed: https://chromium.googlesource.com/chromium/src/+/b416391e0ae62217afb7ea5ffeb04ab2640a96ed

TBR=mthiesse@chromium.org,bajones@chromium.org,acondor@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=644562

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

And also add a trace event that matches the non-network service path.

BUG=598073,694688

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

https://skia.googlesource.com/skia.git/+log/53f77bd4fdd7..489530e06cb7

$ git log 53f77bd4f..489530e06 --date=short --no-merges --format='%ad %ae %s'
2017-05-01 benjaminwagner Fix merge conflict.
2017-05-01 scroggo Remove unnecessary SkFILEStream::getMemoryBase
2017-05-01 msarett Only store width and height on SkPixelRef (part 2)
2017-04-28 benjaminwagner Omit shapes_mixed_10000_32x33 on GTX960 ANGLE Release
2017-04-28 benjaminwagner Omit HalfFloatAlphaTextureTest on CommandBuffer bot.
2017-04-30 benjaminwagner Add Swarming bot id and task id to recipe expectations.
2017-05-01 egdaniel Revert "Revert "Plumb the use of GrBackendRenderTarget throughout Skia""

Created with:
  roll-dep src/third_party/skia


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


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=scroggo@chromium.org

Change-Id: I4c26c247c484d17dd5b77af9db392553db895821
Reviewed-on: https://chromium-review.googlesource.com/491306Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#468336}

With out-of-process iframes, it is not safe to assume that an
unresponsive event comes from the main frame's widget/process.

BUG=623228
TEST=See bug for repro steps.

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

Add a new pattern to detect and scrub serial numbers logged in
/var/log/messages

BUG=714802
TEST=components_unittests --gtest_filter=AnonymizerToolTest.AnonymizeCustomPatterns

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

BUG= 716597

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

We add the following log messages:
* Error message when one of tracingCategories & traceEventsToMeasure is specified
but not the other.
* Hint users to use tools/perf/run_benchmark to get tracing based metrics

BUG=701059

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

The browser code contains partial support for using multiple zygote
processes, which exists to support Flash component updates. However,
at present the browser process only creates a single zygote, on startup,
and uses that for everything. Clean up the code to make that more clear.

* Remove probably-unused lazy zygote creation code.
* Always use ZygoteHandle instead of ZygoteHandle*. ZygoteHandle is always
a pointer under the hood.
* Add CHECKs to validate the assumption there is only one zygote.

This came up while investigating a zygote check failure on Chrome OS.
I'm concerned that the lazy-launch code could be starting a zygote during
browser process shutdown. See crbug.com/692227

BUG=692227,569191
TEST=manually load web pages, extensions and nacl apps to trigger child
process creation, verify they load properly

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

Revert of Photo Picker Dialog: Use sandboxed utility process for decoding images. (patchset #11 id:240001 of https://codereview.chromium.org/2816733002/ )

Reason for revert:
generate_orderfiles.py fails due to AndroidManifest.xml change (see http://crbug.com/717071 )

BUG=717071

Original issue's description:
> Photo Picker Dialog: Use sandboxed utility process for decoding images.
>
> BUG=656015
>
> Review-Url: https://codereview.chromium.org/2816733002
> Cr-Commit-Position: refs/heads/master@{#468209}
> Committed: https://chromium.googlesource.com/chromium/src/+/83c9f3f4faeb35a3a2607446def65f1a8da7aa49

TBR=twellington@chromium.org,yusufo@chromium.org,tedchoc@chromium.org,finnur@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=656015

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

SkPixelRef no longer stores an SkImageInfo

BUG=skia:6535
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

The |hit_test_bounds_override_inner_| member of
aura::Window is no longer used, so remove it.
No functional changes.

BUG=344924
TEST=aura_unittests
TBR=varkha@chromium.org

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

This CL removes the following files from bindings/core/v8 and updates
all include sites to use their platform/bindings version:

- ActiveScriptWrappable.h
- ConditionalFeatures.h
- DOMDataStore.h
- DOMWrapperMap.h
- DOMWrapperWorld.h
- Microtask.h
- ScopedPersistent.h

BUG=682322
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_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

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

Part 2/2.

BUG=713847

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

This uses the new DelayedPersistentAllocation to avoid allocating the
full "counts" array when only one value is being stored.  That one
value is kept in the HistogramSamples metadata.  When a second value
is recorded, the counts-array is created and the single-sample is
moved; it is there-after unused.

This change only affects SampleVector.  A future change may add the
same support for PersistentSampleMap.

A quick test shows persistent memory use is reduced by 44% (from 924
KiB to 512 KiB) after a clean start and the loading of three pages:
  chrome://histograms
  https://www.google.com/
  chrome://histograms  (again)

BUG=705342

Review-Url: https://codereview.chromium.org/2811713003
Cr-Original-Commit-Position: refs/heads/master@{#466401}
Committed: https://chromium.googlesource.com/chromium/src/+/4162bafec82ea11c98d4dc2572337a039b79fdb9
Review-Url: https://codereview.chromium.org/2811713003
Cr-Commit-Position: refs/heads/master@{#468325}

They should not come close to one hour.

BUG=717028
TEST=test only change
R=jonross@chromium.org
TBR=jonross@chromium.org

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

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a
doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340

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

The command buffer client handles all the unpack parameters but
alignment for non-compressed texture uploads from client data.
Resetting the unpack state was handled transparently on the
service-side. This implements the resetting in the passthrough cmd
decoder too.

This makes the following tests pass with the passthrough command buffer:
 - conformance2/textures/misc/tex-unpack-params
 - deqp/functional/gles3/texturespecification/teximage2d_unpack_params
 - deqp/functional/gles3/texturespecification/teximage3d_unpack_params
 - deqp/functional/gles3/texturespecification/texsubimage2d_unpack_params
 - deqp/functional/gles3/texturespecification/texsubimage3d_unpack_params

BUG=chromium:602688
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_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

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

BUG=715729

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

BUG=705865
TBR=zmo@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_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

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

Decreased the number of missing header files by 4.

BUG=661774

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

https://pdfium.googlesource.com/pdfium.git/+log/ce8e51e6c444..72b715d51383

$ git log ce8e51e6c..72b715d51 --date=short --no-merges --format='%ad %ae %s'
2017-05-01 npm Add PDF reference to CFX_Matrix comment

Created with:
  roll-dep src/third_party/pdfium


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


TBR=dsinclair@chromium.org

Change-Id: I9b0e04c24e07ab8ba754d96b22859e1cb8297f3a
Reviewed-on: https://chromium-review.googlesource.com/491049
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#468318}

iOS has a custom method for determining which omnibox suggestion types
are search/query types (as opposed to URL types). This change removes it
in favor of the cross-platform method used on other platforms. This has
useful effects on how we display navsuggest and calculator suggestions
and eliminates the need to maintain separate logic.

See the bug for a detailed description of the motivation for this change
and an analysis of its effects.

BUG=716047

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

BUG=715060

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

https://skia.googlesource.com/skia.git/+log/0c5cf5d7a260..53f77bd4fdd7

$ git log 0c5cf5d7a..53f77bd4f --date=short --no-merges --format='%ad %ae %s'
2017-04-28 fmalita Initial clip-mask-layer support
2017-04-28 jvanverth Optimizations for analytic shadows.

Created with:
  roll-dep src/third_party/skia


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


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=scroggo@chromium.org

Change-Id: I1783644d20be8d8950df6db24daf042d277590c3
Reviewed-on: https://chromium-review.googlesource.com/490838Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#468315}

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}