GitLab

Failures on all platforms:
  virtual/video-surface-layer/media/controls/modern/
    tap-to-hide-controls.html

Timeouts:
  fast/events/hr-timestamp/input-events.html

Bug: 810437, 831720
Change-Id: I538493dc8a815e04effe5090ad432ad79fa477e8
Tbr: steimel@chromium.org
Tbr: skyostil@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1054849Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557747}

Mark these two tests as failing:
  deqp/functional/gles3/shaderoperator/binary_operator_01.html
  deqp/functional/gles3/shaderoperator/unary_operator_01.html

Bug: 756537
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I08a3d1b7ba3e240aec2347a92e8b52156780c510
Tbr: rjkroege@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1054773Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557746}

Validate response headers of an HTTP/2 pushed stream against the client
request headers even if the response headers only arrive after the
pushed stream has been claimed by a request.  Cancel stream and retry
request in HttpNetworkTransaction if headers do not match.

Bug: 554220
Change-Id: I5e1a42583b941faef5529b50178fe00d8c864976
Reviewed-on: https://chromium-review.googlesource.com/1053429Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Bence Béky <bnc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557745}

Bug: 790520
Change-Id: I9612375c0c479e4b0219cd2c315e895c0185afe6
Reviewed-on: https://chromium-review.googlesource.com/1054208Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Zhiling Huang <hzl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557744}

Adds circular inkdrops for the toolbar and pill-shaped ones for entries
in the bookmarks bar for Refresh. This removes the recent move to 8dp
squircles for Refresh and Touch and instead unifies them as rounded.

Also tones down EMPHASIS_HIGH for non-touchable and non-refresh as they
are not ready for rounded corners.

Bug: chromium:822069, chromium:822072
Change-Id: Ie619f40834127ebfde59621a86fae42e7d8c4f23
Reviewed-on: https://chromium-review.googlesource.com/1053379
Commit-Queue: Peter Boström <pbos@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557743}

Currently, the search providers all have static prepopulated favicon
URLs. These can fall out of date.

This CL updates the search providers' favicons as the user
browses. Specifically, when the user actually lands on a URL matching
the a search provider URL template, and we get a favicon that does not
match the stored favicon URL, we update the stored favicon URL.

Bug: 88243, 823535
Change-Id: I43f72ad449515d5a19e5ec4b32ca996b9d32eb11
Reviewed-on: https://chromium-review.googlesource.com/1048174
Commit-Queue: Tommy Li <tommycli@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557742}

This is a reland of 52e03fa8

Mac compile failure should be fixed with USE_AURA #ifdef around test.

Original change's description:
> AppShell: Suppress context menu in webviews
>
> AppShell doesn't support context menus, but still needs to provide a
> WebViewGuestDelegate to handle the context menu. Otherwise, we reach a
> NOTREACHED() in WebContentsViewChildFrame::ShowContextMenu().
>
> Also removes a WebViewGuestDelegate function that is unused (see
> https://codereview.chromium.org/2836973002).
>
> Bug: 820604
> Change-Id: I0760d397384908a490704f23388e11c4eef113a5
> Reviewed-on: https://chromium-review.googlesource.com/957596
> Commit-Queue: Michael Giuffrida <michaelpg@chromium.org>
> Reviewed-by: Lucas Gadani <lfg@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#544136}

Bug: 820604
Change-Id: Ib44c00f70a89f88b13f0ba934d725d10353bff86
Reviewed-on: https://chromium-review.googlesource.com/1053181Reviewed-by: Lucas Gadani <lfg@chromium.org>
Commit-Queue: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557741}

Transient downloads are meant not to show up in download home. However
notifications must be shown during the download. This was broken
recently due to a change which affected background fetch notifications
to get lost. Restoring this behavior.

Bug: 841344
Change-Id: I034b9761b0bd33275e23a298691b49e0db7f6d3f
Reviewed-on: https://chromium-review.googlesource.com/1052891Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557740}

Bug: 775830
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I3da04418bac37e0e03b63df4a8d2407296948e77
Reviewed-on: https://chromium-review.googlesource.com/1045492
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557739}

Refactor BoxPainterBase and BoxModelObjectPainter to supply flowbox size
as parameter to PaintFillLayer instead of storing it as a private field.

This will in turn help with code re-use between InlineFlowBoxPainter and
the LayoutNG equivalent.

Bug: 714962
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I342c51c87eb253e2d6420f2b11440b77c564f67d
Reviewed-on: https://chromium-review.googlesource.com/1054133Reviewed-by: Aleks Totic <atotic@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557738}

R=dpapad@chromium.org
BUG=841000

Change-Id: I4b0c6ae08a6097df28aed3743fbe82f211c2e023
Reviewed-on: https://chromium-review.googlesource.com/1053425
Commit-Queue: Dan Beam (no longer on Chrome) <dbeam@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557737}

Introduces an LRU cache for prelaunching renderers. This class creates a
pool of prelaunched renderers for the N most recently closed pages.

Bug: internal b/77879457
Test: cast_shell_unittests
Change-Id: I0500f24b65377360d0c74ea390b476e1b879708a
Reviewed-on: https://chromium-review.googlesource.com/1036730
Commit-Queue: Sean Topping <seantopping@chromium.org>
Reviewed-by: Stephen Lanham <slan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557736}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/0aae1c1fc30a..3c311c15709b

$ git log 0aae1c1fc..3c311c157 --date=short --no-merges --format='%ad %ae %s'
2018-05-09 manojgupta cbuildbot: Disable unit tests in fuzzer builds.
2018-04-20 bmgordon build_stages: Use snapshot to clean up chroot.
2018-05-04 yunlian generic_stages: temporarily ignore all target prebuilts.
2018-05-08 guocb auto_updater: filter more directories when copy devserver to DUT

Created with:
  roll-dep src/third_party/chromite
BUG=chromium:841512,chromium:829665,chromium:730144,chromium:826418,chromium:840975,chromium:824580


The AutoRoll server is located here: https://chromite-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=chrome-os-gardeners@chromium.org

Change-Id: I0abebd960beb44b1560aca995781fed1b91f453f
Reviewed-on: https://chromium-review.googlesource.com/1054619
Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#557735}

This reverts commit 13077433.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> Revert "Cancel parallel download request if server doesn't send partial response"
> 
> This reverts commit 189d1033.
> 
> Reason for revert: "ParallelDownloadTest.NoPartialResponse" is flaky. See https://crbug.com/841666.
> 
> Original change's description:
> > Cancel parallel download request if server doesn't send partial response
> > 
> > If the server doesn't support partial responses, Chrome should cancel
> > parallel download requests.
> > These requests result in the wrong write offset and makes the file larger
> > than the original
> > 
> > BUG=838627
> > 
> > Change-Id: I1fa878854579a1572e6e2f90f3bb932b59b333ba
> > Reviewed-on: https://chromium-review.googlesource.com/1048130
> > Commit-Queue: Min Qin <qinmin@chromium.org>
> > Reviewed-by: Xing Liu <xingliu@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#557252}
> 
> TBR=qinmin@chromium.org,xingliu@chromium.org
> 
> Change-Id: I995c3c48fe39c535275a353130b10204009c5e30
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 838627, 841666
> Reviewed-on: https://chromium-review.googlesource.com/1053470
> Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
> Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#557489}

TBR=yhirano@chromium.org,qinmin@chromium.org,xingliu@chromium.org

Change-Id: I47a2d4131efa8b863d9868eeca74e99d7beeaa8e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 838627, 841666
Reviewed-on: https://chromium-review.googlesource.com/1054887Reviewed-by: Min Qin <qinmin@chromium.org>
Commit-Queue: Min Qin <qinmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557734}

The RemoteMediaPlayerWrapper's call to RMP.load() never completes, and
all subsequent API calls fail. This is because messages from the
CastDevice are not being properly routed back to RemoteMediaPlayer.

This CL register the RemotingCastSession as a MessageReceivedCallback
for the media namespace, and forwards those messages to the
RemoteMediaPlayerWrapper.

Bug: 790766
Change-Id: Ibb514b14d4559fd280c6bafac871781c7fb1ac99
Reviewed-on: https://chromium-review.googlesource.com/1044969Reviewed-by: Zhiqiang Zhang <zqzhang@chromium.org>
Commit-Queue: Thomas Guilbert <tguilbert@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557733}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/e877b1776a14..2191308678e4

$ git log e877b1776..219130867 --date=short --no-merges --format='%ad %ae %s'
2018-05-10 jbudorick Revert "gclient: Get rid of parsed_url."
2018-05-10 ehmaldonado bot_update: Assert no syncing of refs/remotes/*

Created with:
  roll-dep src/third_party/depot_tools
BUG=chromium:841936,chromium:840445


The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=agable@chromium.org

Change-Id: I52a88ffeda4b56309156e3fc377eebdbc3ae8138
Reviewed-on: https://chromium-review.googlesource.com/1054635Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#557732}

This reverts commit 7ebf529c.

Reason for revert: Breaks many browser_tests in official builds.

Google-internal official builder:
https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/win%20trunk/builds/84099

Failures are visible publicly here:
https://ci.chromium.org/buildbot/chromium.clang/ToTWin/1489

Original change's description:
> Revert "Revert "Disable Channel ID on all platforms except Android""
> 
> This reverts commit 3c79bfc0.
> 
> In addition to the original CL bd3d1ea0, this includes a fix to the crash
> when relaunching after changing about:flags.
> 
> Bug: 841006
> Change-Id: I04c82074bb83da7ba5854b783862ec689d7a36e0
> Reviewed-on: https://chromium-review.googlesource.com/1050976
> Reviewed-by: Bo <boliu@chromium.org>
> Reviewed-by: Ryan Hamilton <rch@chromium.org>
> Commit-Queue: Nick Harper <nharper@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#557307}

TBR=boliu@chromium.org,rch@chromium.org,nharper@chromium.org,dschuyler@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 841006
Change-Id: I2900a883c01307f3c91bcd91698ef8487acfac44
Reviewed-on: https://chromium-review.googlesource.com/1054205Reviewed-by: Reid Kleckner <rnk@chromium.org>
Commit-Queue: Reid Kleckner <rnk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557731}

This is a reland of 3c6230c7

Previous revert is caused by merge conflict with another CL that
happened to land around the same time.

This reland is a simply a rebase of the original CL.

Original change's description:
> Refactor HashPasswordManager to accept multiple password hashes
>
> This CL doesn't change existing behavior, but only refactor
> HashPasswordManager class such that it can keep track of multiple
> password hashes. This CL introduces a new struct PasswordHashData to
> replace SyncPassswordData.
>
> Migration from SyncPasswordData to PasswordHashData is in place so that
> we'll not lose already captured sync password. IsSyncPasswordHashSaved
> histogram enum is added to track this migration. SyncPasswordData
> will be deprecated after the transition period.
>
> Bug: 830998
> Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
> Change-Id: I50b52adcd1cf14a6bfddc74838019f3c7d287a99
>
> TBR=jochen@chromium.org
>
> Change-Id: I50b52adcd1cf14a6bfddc74838019f3c7d287a99
> Reviewed-on: https://chromium-review.googlesource.com/1036464
> Commit-Queue: Jialiu Lin <jialiul@chromium.org>
> Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
> Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#557646}

TBR=dvadym@chromium.org, jochen@chromium.org

Bug: 830998
Change-Id: I195fbea0c53f2b3a2000e53b37f3eeaa33d5b846
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/1054354Reviewed-by: Jialiu Lin <jialiul@chromium.org>
Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Jialiu Lin <jialiul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557730}

Add the UI strings for NTP custom backgrounds,
custom links, and fakebox.

Bug: 837798, 839152, 837784
Change-Id: I8c605ec41923eced800579f8c15901caf93e93f4
Reviewed-on: https://chromium-review.googlesource.com/1052667Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557729}

Also doing the same for kEasyUnlockLearnMoreUrl, which is only used in CrOS.

Bug: 621723
Change-Id: Ieada2f363c886e9321016de45a2157a3d35ac1fb
Reviewed-on: https://chromium-review.googlesource.com/1053155
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557728}

Instead of having MojoMediaClient creating a RendererFactory, ask
MojoMediaClient to create a Renderer directly. This helps us to
eliminate the audio and video renderer sink from the interface, which
isn't interesting to most Renderers hosted by MojoRendererService.

This also helps to simplify how we pass |audio_device_id| into the
Renderer directly (not having to go through the audio sink).

Bug: 586211
Test: All existing tests pass.
Change-Id: Ifcb0708a063da1958ccf495ef08ab8f98902617d
Reviewed-on: https://chromium-review.googlesource.com/1053295Reviewed-by: Stephen Lanham <slan@chromium.org>
Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557727}

https://skia.googlesource.com/skia.git/+log/024615e9752d..959fc245a92a

$ git log 024615e97..959fc245a --date=short --no-merges --format='%ad %ae %s'
2018-05-10 brianosman Add SkColorSpace::toProfile
2018-05-10 bsalomon Add GrContextThreadSafeProxy and remove most friends of GrContextThreadSafeProxy
2018-05-10 mtklein non-linear blending first steps

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
TBR=egdaniel@chromium.org

Change-Id: I5fc690ffbb0c016a92b25ff07cd7e152c9d477d0
Reviewed-on: https://chromium-review.googlesource.com/1054550
Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#557726}

Viz hit-test doesn't need LocalSurfaceId in targeting/ transforming
requests, HitTestQuery has that information; and we don't currently
have a valid one in RenderWidgetHostViewGuest and
RenderWidgetHostViewChildFrame in OOP-D. We could add it in RWHVG/
RWHVCF, but since we don't need it anyway, skip this validation for
Viz hit-test.

Bug: 840893
Change-Id: Ic9bbf9f29fc2016f9c838f781f8cf554c180d61e
Reviewed-on: https://chromium-review.googlesource.com/1050440Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Ria Jiang <riajiang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557725}

The ImageData might get destroyed while the paints are still
pending. Typically, the paints are then cancelled thereafter so
no harm comes from the dangling references, but this patch avoids
creating them in the first place.

The remaining changes are consequences of ProgressivePaint
becoming non-POD, and converting to protected members. Also
use scoped FPDF classes while we're at it.

Bug: 838886
Change-Id: Ic933350ac9e981da58f1f841968dc89fb8518974
Reviewed-on: https://chromium-review.googlesource.com/1054502Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557724}

Bug: 841974
Change-Id: Ie847933ad8090024a14807cc4db629a837266092
Reviewed-on: https://chromium-review.googlesource.com/1054638Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Alexander Alekseev <alemate@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557723}

Turns ContentVideoView/ContentVideoViewEmbedder public interfaces,
and removes the corresponding DEPS exceptions.

Bug: 617324
Change-Id: I2e1fdd5b48e6a11e204c3d145458a58561dc0fb8
Reviewed-on: https://chromium-review.googlesource.com/1043305Reviewed-by: Stephen Lanham <slan@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557722}

Bug: 840898
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I5180a32f2453547d915ca5867190facc30e63e00
Reviewed-on: https://chromium-review.googlesource.com/1053115Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557721}

This CL adds even more startup traces to diagnose what causes the 8 ms
gap. My main suspicion is that this is caused by class-loading
WebViewContentsClientAdapter (locally, I see a 0.5 ms gap between the
start of insideCreateWebViewContentsClientAdapter &
beginningOfConstructor).

The goal is to land these on trunk and view the bot to see what takes
the majority of the time. Then we can look into optimizing this.

Bug: 838702
Test: run benchmark locally, observe the events are logged.
Change-Id: Id1976e80e69dbb1073ed2197b60ca0652ad15356
Reviewed-on: https://chromium-review.googlesource.com/1054637Reviewed-by: Changwan Ryu <changwan@chromium.org>
Commit-Queue: Nate Fischer <ntfschr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557720}

- Get rid of redundant .c_str() for std::string,
- Get rid of redundant .get() for std::unique_ptr,
- Get rid of redundant member initializations
- Replace NULL -> nullptr.

R=blundell@chromium.org, caitkp@chromium.org, jochen@chromium.org, meacer@chromium.org, sdefresne@chromium.org

Change-Id: Ie334879428001915e2684dfd106bc5f7e4950718
Reviewed-on: https://chromium-review.googlesource.com/1046050
Commit-Queue: Christopher Thompson <cthomp@chromium.org>
Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
Reviewed-by: Varun Khaneja <vakh@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Rachel Blum <groby@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557719}

https://chromium.googlesource.com/catapult.git/+log/f24714c46c60..a75468d15a3e

$ git log f24714c46..a75468d15 --date=short --no-merges --format='%ad %ae %s'
2018-05-10 nednguyen Remove all Telemetry profilers

Created with:
  roll-dep src/third_party/catapult


The AutoRoll server is located here: https://catapult-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=sullivan@chromium.org

Change-Id: Ib8051cd218ab5f646b8dd392da0c01b6e739771f
Reviewed-on: https://chromium-review.googlesource.com/1054430Reviewed-by: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#557718}

When ParentWindowAllocator changed to default generate a valid
LocalSurfaceId, it broke an implicit assumption that Aura's
WindowPortLocal was relying on.

aura::WindowPortLocal used the default invalid LocalSurfaceId to know it
had not generated a valid ID as the result of embedding external
content.

This patch fixes that by keeping the LocalSurfaceId allocator in a
base::Optional and checking that to determine external content
embedding.

BUG=840246

Change-Id: Ie5475e239cc237caaf3a68b149f92812746a7129
Reviewed-on: https://chromium-review.googlesource.com/1046378
Commit-Queue: Chris Blume <cblume@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557717}

This allows Cocoa and Views to use the same pasteboard notification.
This does not hook up views Find Bar updates to the pasteboard.

BUG=826478,827189

Change-Id: I038fe4b3acb9e3884ca928d9cbc5106a1a324913
Reviewed-on: https://chromium-review.googlesource.com/1053548
Commit-Queue: Robert Liao <robliao@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557716}

When we get a child's intrinsic size we need to make sure to use the correct
writing mode. This patch only fixes the case where the child has a size
specified.

Makes a few more tests pass (see the test expectations change). Makes one test
fail, but that one passed more by coincidence than anything else.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ia14edd6c85970ccb17c8735dbb3a307e657015a6
Reviewed-on: https://chromium-review.googlesource.com/1053885
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Aleks Totic <atotic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557715}

Driven by follow up of https://crrev.com/c/1042825.

BUG=773295

Change-Id: I8778d225f411c84f431867b2845301f3da770d6d
Reviewed-on: https://chromium-review.googlesource.com/1050705
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557714}

Bug: 841966
Change-Id: I7e14a67b0f89d1ba713c582d07ba2229c7ecf25d
Reviewed-on: https://chromium-review.googlesource.com/1054226Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557713}

Change-Id: I0b716112f233b2790ec0ee5af33760438aea9218
Reviewed-on: https://chromium-review.googlesource.com/1054422Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: Artur Khachatryan <khachatryan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557712}

Layout tests were disabled on Fuchsia, so they bitrotted in the last
couple of month. This CL includes the following changes to make layout
tests runnable on Fuchsia again:
1. Content shell package is installed with "pm install" command.
2. Fonts no longer need to be copied - they are part of the content_shell
   package as data deps.
3. Now SSH can used to forward TCP connections instead of
   layout_test_proxy.
4. libosmesa.so can be loaded from the package, so it no longer needs
   to be copied to /system/lib .

Bug: 778467
Change-Id: Ie16f7ad6b9088153450194b280a6f69e9cb25328
Reviewed-on: https://chromium-review.googlesource.com/1054332Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557711}

There is a follow up CL to add an ActionView class which is a child of
DialogPlate that I want to keep grouped in the same directory.

Doing the DialogPlate move here in a separate CL because adding the
ActionView in the same CL introduced enough changes where git didn't
track the DIFF but instead treated the changes as a file delete and
new file add.

Follow up CL is:
https://chromium-review.googlesource.com/c/chromium/src/+/1053160

Screenshot in bug.

Bug: b:79473532
Change-Id: Ifcd7a0db2c67c2c7b4f61244b5107ff3bc4fb42b
Reviewed-on: https://chromium-review.googlesource.com/1054149
Commit-Queue: David Black <dmblack@google.com>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557710}

This was generated by luci_check.  This:
* Adds new buildbot builders
* Remove migrated buildbot builders
* Adds luci equivalent builders

Known issue: Main console is not updated.  That will come later.

Change-Id: Ic6bb08ce57e0a42933181f1ad9a12a95f0739b5e
Reviewed-on: https://chromium-review.googlesource.com/1054004Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Ryan Tseng <hinoka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557709}

Presently, the NetworkContext has three public constructors, one
private constructor, and a public static method, all which, in some
way, configure a URLRequestContext based on NetworkContextParams.

However, one of these constructors in unused except for in an
(uncalled) CreateForTesting() method, two are 'transitional'
constructors that only exist as part of the Network S13N effort to
keep things running, and the static method has implicit and explicit
circular dependencies with the URLRequestContextMojo.

This adds documentation to that effect, removes the unused code,
updates the existing code to follow the 'best practice' for new
code, and tries to hide the footguns from plain sight.

BUG=none

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: If5710d84cbb8e144798da9ac3a5235f92b65105e
Reviewed-on: https://chromium-review.googlesource.com/1053962
Commit-Queue: Ryan Sleevi <rsleevi@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557708}