GitLab

The NTP (New tab page) renderer is a semi-priveleged process having access to
some special APIs. But currently, network requests made by an NTP renderer can
be intercepted by extensions using the web request API.

This CL hides network requests made by an NTP renderer from extensions. This
works since it's guaranteed that the NTP won't share the renderer process with
any other site.

BUG=797461

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Id8523aa7cbb4e5fb74256fa6cfc180709849568a
Reviewed-on: https://chromium-review.googlesource.com/1018724
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553265}

There is a pre-existing bug on Mac where Control + A performs "Select
all" instead of Command + A. The equivalent context menu item just sends
the keystroke to perform "Select all". As such, when it tried to do the
right thing and use the Command modifier, "Select all" does not actually
work.

To fix this quickly, just make the context menu item send Control + A.
Of course this is not ideal, but making Command + A perform "Select all"
is harder than it appears.

BUG=834735,836074

Change-Id: I2c8201772d7e901984c427d0977a1fb899f802b5
Reviewed-on: https://chromium-review.googlesource.com/1025403Reviewed-by: dsinclair <dsinclair@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553264}

Its use of feature flags causes data races.

The first attempt in https://chromium-review.googlesource.com/1026462
got incorrectly reverted by FindIt.

BUG=836359
TBR=xidachen@chromium.org
NOTRY=true

Change-Id: Icfeaf23ccb69b3b1e80f6df10556f5c75e663a71
Reviewed-on: https://chromium-review.googlesource.com/1026557Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553263}

Bug: 834899
Change-Id: I160ebe1b8e830851291e89fdc8b2a860881637f7
Reviewed-on: https://chromium-review.googlesource.com/1025651Reviewed-by: Adrienne Porter Felt <felt@chromium.org>
Commit-Queue: Edward Jung <edwardjung@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553262}

This change just show the default penguin icon on the shelf for all
running Crostini apps. Subsequent CLs will change to use real app
specific icons.

BUG=chromium:819444
TEST=Manually tested on an eve device.

Change-Id: Ic5001edc1cf4a51acd54ea99ab43bbc1afa69dcf
Reviewed-on: https://chromium-review.googlesource.com/1023292Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Nicholas Verne <nverne@chromium.org>
Commit-Queue: Tim Zheng <timzheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553261}

Audio service runs in an unsandboxed utility process behind a flag.
Not all the content dependencies on AudioManager are removed yet,
so:

***when Audio process is enabled***, the state is weird:
1) 2 instances of AudioManager are running:
* one hosted in the BrowserMainLoop (provides audio streams),
* another living in the audio process (provides audio system
information and debug recording interfaces).
2) It also means that debug audio recording not
working (because there are no audio
streams going through the audio process).

When Audio process is disabled (default), Audio service runs embedded
in content and everything works properly (the CL has no effect).

The goal of this CL(if it is landable) is to simplify local manual
testing when developing the audio service.

Change-Id: I5382ba60efa924a314f77803fed119e3d3ebda70
BUG: 672469
Reviewed-on: https://chromium-review.googlesource.com/1016983
Commit-Queue: Olga Sharonova <olka@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553260}

The test failed in debug bots because we direct reparent the HWND to
another shell and skip full window reparent logic then shell close also
overwrite some state.

In this patch, we moved shell close to the end of test body.

Bug: 833661
Change-Id: I06c639a1c1620fe3a06af2eb6606970a6a86c78c
Reviewed-on: https://chromium-review.googlesource.com/1025119Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Jianpeng Chao <chaopeng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553259}

BUG=internal b/78359662
TEST= cast_bluetooth_unittests

Change-Id: Idd10f3edcc161b964cfda90826058564d55c6a6a
Reviewed-on: https://chromium-review.googlesource.com/1022953
Commit-Queue: Bailey Forrest <bcf@chromium.org>
Reviewed-by: Stephen Lanham <slan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553258}

BUG=693090
TBR=zmo@chromium.org

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: I722b61b1c7276e6957b39c9d141224fd48db8498
Reviewed-on: https://chromium-review.googlesource.com/1024472Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Geoff Lang <geofflang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553257}

Use GetTargetVisibility instead of IsVisible to determine whether we should
hide or show the backdrop according to visibility of the app list. Since we
can get the app list visibility change notification immediately through
GetTargetVisibility regardless of the animation.

Bug: 760359
Change-Id: I8b1494144e541b4a646026c285151f524f0cb91b
Reviewed-on: https://chromium-review.googlesource.com/1025215Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Min Chen <minch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553256}

This base::RefCountedMemory subclass owns a
base::ReadOnlySharedMemoryMapping instance.

To demonstrate the usefulness of this new class, convert PdfCompositor
from mojo::ScopedSharedBufferHandle to base::ReadOnlySharedMemoryRegion.
Use base::RefCountedSharedMemoryMapping to wrap memory regions and avoid
data copies.

While changing PdfCompositor, turn some DCHECKs into actual checks,
since there is no guarantee shared memory options will always succeed.

BUG=826213

Change-Id: I3135235ea350f58c83606e530b33ed6aa7be3cb5
Reviewed-on: https://chromium-review.googlesource.com/989429Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Wei Li <weili@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553255}

More fuzzers need to be added to Zucchini for launch including
adding support for protobuf based fuzzers. To facilitate this a new
fuzzers/ subdirectory will help to separate Zucchini from its fuzz
related infrastructure.

Bug: 835341
Change-Id: Ib18bfe9bb0b0050e94fa7bdca22fb99c735d9141
Reviewed-on: https://chromium-review.googlesource.com/1026475Reviewed-by: Samuel Huang <huangs@chromium.org>
Commit-Queue: Calder Kitagawa <ckitagawa@google.com>
Cr-Commit-Position: refs/heads/master@{#553254}

https://pdfium.googlesource.com/pdfium.git/+log/90e03b9b3f45..6453a67d84dc

$ git log 90e03b9b3..6453a67d8 --date=short --no-merges --format='%ad %ae %s'
2018-04-24 dsinclair Remove CFX_BlockBuffer
2018-04-24 hnakashima Fix edit text rendered with wrong offsets after deletion.
2018-04-24 dsinclair Cleanup CFX_XMLParser entity conversion
2018-04-24 hnakashima Fix caret moving to next line on click in multiline XFA edit.

Created with:
  roll-dep src/third_party/pdfium
BUG=chromium:835924,chromium:832293


The AutoRoll server is located here: https://pdfium-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=dsinclair@chromium.org

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

As mentioned in the spec (https://webaudio.github.io/web-audio-api/#dynamicscompressornode),
the channelCountMode defaults to "clamped-max", not "max".  Make it so.

Bug: 835946
Test: webaudio/constructor/dynamicscompressor.html
Change-Id: I8aee3336a89ff5662ecbe7afee1793cb94184169
Reviewed-on: https://chromium-review.googlesource.com/1024791
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553252}

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I48ac2749a45e134d69f7c6dd8cd0408dc87fcce7
Reviewed-on: https://chromium-review.googlesource.com/1025375
Commit-Queue: Eric Roman <eroman@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553251}

Verify DM token fetch and store status.
Also add an observer for the enrollment process. This will be used
by upcoming browser launch block during enrollment feature.


Bug: 812641
Change-Id: I8d9df9ecc6cdbb3f6616253d597a82d3d9414810
Reviewed-on: https://chromium-review.googlesource.com/1020417
Commit-Queue: Owen Min <zmin@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: Roger Tawa <rogerta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553250}

Currently, it's possible for a |RenderFrameHost| to be deleted between
callbacks to |AuthenticatorImpl|, and then for |AuthenticatorImpl| to
try and use a dangling pointer in |render_frame_host_|.

This was caught by the TSAN bot on
https://chromium-review.googlesource.com/c/chromium/src/+/1007637, but
that change only made it clearer—it could happen already.

Change-Id: Id5227d7a84844d5195a33f99ef504ae5883446d8
Reviewed-on: https://chromium-review.googlesource.com/1025223
Commit-Queue: Adam Langley <agl@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553249}

For printers with only 1 option for color, the unavailable value is
used for the print ticket. This CL updates this to reflect the printer's
only color option instead of always defaulting to false.

Also add validation of the value set by default to the Color
SettingsSectionTest.

Bug: 773928
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ib59cb766a8590e159cd23bcceb4b45214dc76f74
Reviewed-on: https://chromium-review.googlesource.com/1025016Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553248}

Change-Id: Iccb180e333b1ec276965ea53e87857c3c40636ec
Reviewed-on: https://chromium-review.googlesource.com/1024793
Commit-Queue: Eric Roman <eroman@chromium.org>
Reviewed-by: David Benjamin <davidben@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553247}

https://chromium.googlesource.com/catapult.git/+log/921cf8305ce5..0d6f84866778

$ git log 921cf8305..0d6f84866 --date=short --no-merges --format='%ad %ae %s'
2018-04-23 ehmaldonado pinpoint: Make DEPS parsing forwards compatible.
2018-04-24 simonhatch Dashboard - Remove mapreduce.yaml from __init__.py

Created with:
  roll-dep src/third_party/catapult
BUG=chromium:834968


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: I11cf5e67d2fadba50168fd19576c19e302bb2d4e
Reviewed-on: https://chromium-review.googlesource.com/1026161Reviewed-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@{#553246}

This CL is a refactor with no intended behavior change.

This CL updates TabStripModel::AppendWebContents to take a std::unique_ptr
rather than a raw pointer.

Change-Id: Ie04a8bc9299108d65449a765e5cffaebf9ca91ff
Bug: 832879
Reviewed-on: https://chromium-review.googlesource.com/1024296Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553245}

Power.PowerButtonPressInLaptopMode for press power button in laptop mode.
Power.PowerButtonPressInTabletMode for press power button in tablet mode.

Bug: 796315
Change-Id: Ibeaebbc39d3d5186552bd2a4f988d0e0144fc84c
Reviewed-on: https://chromium-review.googlesource.com/1022492Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Qiang Xu <warx@google.com>
Reviewed-by: Dan Erat <derat@chromium.org>
Commit-Queue: Min Chen <minch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553244}

Remove the IsNavigationMojoResponseEnabled() function and every mention
of "NavigationMojoResponse" in the code.

Other CLs:
  1) https://chromium-review.googlesource.com/c/chromium/src/+/1015005
  2) https://chromium-review.googlesource.com/c/chromium/src/+/1021631
* 3) https://chromium-review.googlesource.com/c/chromium/src/+/1022790
  4) https://chromium-review.googlesource.com/c/chromium/src/+/1024038

Bug: 705744
Change-Id: Ia755a96901d08819d0b10417965e292821415c44
Reviewed-on: https://chromium-review.googlesource.com/1022790
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Varun Khaneja <vakh@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553243}

Change-Id: If02c85c71b0092eeb631dcbe9aefb765a67d5752
Reviewed-on: https://chromium-review.googlesource.com/1025327Reviewed-by: Ryan Tseng <hinoka@chromium.org>
Commit-Queue: Alan Bram <flyboy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553242}

This reverts commit da17415d.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 553230 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2RhMTc0MTVkMTg4ODRmYjQ3NDUxYmIyN2ZjOWYzMDg3ZmRiMzc5MTUM

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.linux/Linux%20Builder/101940

Sample Failed Step: compile

Original change's description:
> Disable NotificationImageReporterTest.
> 
> Its use of feature flags causes data races.
> 
> BUG=836359
> TBR=xidachen@chromium.org
> NOTRY=true
> 
> Change-Id: I5bc92a6fc9b9010b9863b363dc3033b95f777762
> Reviewed-on: https://chromium-review.googlesource.com/1026462
> Reviewed-by: Lei Zhang <thestig@chromium.org>
> Commit-Queue: Lei Zhang <thestig@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#553230}

No-Presubmit: true
No-Tree-Checks: true
No-Try: true
BUG=836359

Change-Id: I748cc7ee9c5dd97a74d10dfdd77161966cff86b8
Reviewed-on: https://chromium-review.googlesource.com/1026433
Cr-Commit-Position: refs/heads/master@{#553241}

This reverts commit 7f7c86d8.

Reason for revert: <INSERT REASONING HERE>
Causing compile error:
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Fuchsia%20ARM64%20Cast%20Audio/10534

Original change's description:
> Revert "Reland "Remove HttpStreamFactory interface.""
> 
> This reverts commit ea3e3f22.
> 
> Reason for revert: Failure at
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.10%20Tests/31554
> 
> Original change's description:
> > Reland "Remove HttpStreamFactory interface."
> >
> > Reland https://crrev.com/c/1000487.
> >
> > Remove HttpStreamFactory interface, also rename HttpStreamFactoryImpl
> > to
> > HttpStreamFactory.
> >
> > Bug: 829429
> > Change-Id: I298879817785174d1d7290520f889f0252074e14
> > Reviewed-on: https://chromium-review.googlesource.com/1023321
> > Commit-Queue: Bence Béky <bnc@chromium.org>
> > Reviewed-by: Ryan Hamilton <rch@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#553180}
> 
> TBR=rch@chromium.org,bnc@chromium.org
> 
> Change-Id: I9fab232c4219d9e11d5df54f57c13dfd50540b2f
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 829429
> Reviewed-on: https://chromium-review.googlesource.com/1026491
> Commit-Queue: Bence Béky <bnc@chromium.org>
> Reviewed-by: Bence Béky <bnc@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#553233}

TBR=rch@chromium.org,bnc@chromium.org

Change-Id: I1a33380942c2fc0d3aed903a4a505f0615e24f8c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 829429
Reviewed-on: https://chromium-review.googlesource.com/1026492Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553240}

This reverts commit 4aa787aa.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> Reland "Reland "Remove HttpStreamFactory interface.""
> 
> This reverts commit 7f7c86d8.
> 
> Reason for revert: <INSERT REASONING HERE>
> Causing compile error here:
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Fuchsia%20ARM64%20Cast%20Audio/10534
> 
> Original change's description:
> > Revert "Reland "Remove HttpStreamFactory interface.""
> > 
> > This reverts commit ea3e3f22.
> > 
> > Reason for revert: Failure at
> > https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.10%20Tests/31554
> > 
> > Original change's description:
> > > Reland "Remove HttpStreamFactory interface."
> > >
> > > Reland https://crrev.com/c/1000487.
> > >
> > > Remove HttpStreamFactory interface, also rename HttpStreamFactoryImpl
> > > to
> > > HttpStreamFactory.
> > >
> > > Bug: 829429
> > > Change-Id: I298879817785174d1d7290520f889f0252074e14
> > > Reviewed-on: https://chromium-review.googlesource.com/1023321
> > > Commit-Queue: Bence Béky <bnc@chromium.org>
> > > Reviewed-by: Ryan Hamilton <rch@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#553180}
> > 
> > TBR=rch@chromium.org,bnc@chromium.org
> > 
> > Change-Id: I9fab232c4219d9e11d5df54f57c13dfd50540b2f
> > No-Presubmit: true
> > No-Tree-Checks: true
> > No-Try: true
> > Bug: 829429
> > Reviewed-on: https://chromium-review.googlesource.com/1026491
> > Commit-Queue: Bence Béky <bnc@chromium.org>
> > Reviewed-by: Bence Béky <bnc@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#553233}
> 
> TBR=rch@chromium.org,bnc@chromium.org
> 
> Change-Id: I55ada6185c0a923d0c87b5986c15d1fc5f06d461
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 829429
> Reviewed-on: https://chromium-review.googlesource.com/1026530
> Reviewed-by: Xida Chen <xidachen@chromium.org>
> Commit-Queue: Xida Chen <xidachen@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#553238}

TBR=rch@chromium.org,bnc@chromium.org,xidachen@chromium.org

Change-Id: Ic8ecc1e9d34d964d4b825863ef1b725f4e48ad90
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 829429
Reviewed-on: https://chromium-review.googlesource.com/1026531Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553239}

This reverts commit 7f7c86d8.

Reason for revert: <INSERT REASONING HERE>
Causing compile error here:
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Fuchsia%20ARM64%20Cast%20Audio/10534

Original change's description:
> Revert "Reland "Remove HttpStreamFactory interface.""
> 
> This reverts commit ea3e3f22.
> 
> Reason for revert: Failure at
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.10%20Tests/31554
> 
> Original change's description:
> > Reland "Remove HttpStreamFactory interface."
> >
> > Reland https://crrev.com/c/1000487.
> >
> > Remove HttpStreamFactory interface, also rename HttpStreamFactoryImpl
> > to
> > HttpStreamFactory.
> >
> > Bug: 829429
> > Change-Id: I298879817785174d1d7290520f889f0252074e14
> > Reviewed-on: https://chromium-review.googlesource.com/1023321
> > Commit-Queue: Bence Béky <bnc@chromium.org>
> > Reviewed-by: Ryan Hamilton <rch@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#553180}
> 
> TBR=rch@chromium.org,bnc@chromium.org
> 
> Change-Id: I9fab232c4219d9e11d5df54f57c13dfd50540b2f
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 829429
> Reviewed-on: https://chromium-review.googlesource.com/1026491
> Commit-Queue: Bence Béky <bnc@chromium.org>
> Reviewed-by: Bence Béky <bnc@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#553233}

TBR=rch@chromium.org,bnc@chromium.org

Change-Id: I55ada6185c0a923d0c87b5986c15d1fc5f06d461
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 829429
Reviewed-on: https://chromium-review.googlesource.com/1026530Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553238}

Depending on who runs generate_gni.sh we get different
formatting from clang-format. Force it to -style=Chromium

Change-Id: Iee48618ee7e43ae956d3a65e5b08a6babc14c83e
Reviewed-on: https://chromium-review.googlesource.com/1024436Reviewed-by: Tom Finegan <tomfinegan@chromium.org>
Commit-Queue: Johann Koenig <johannkoenig@google.com>
Cr-Commit-Position: refs/heads/master@{#553237}

To help us figure out how to prioritize the various picture buffer
types in the new D3D11VideoDecoder, this UMA records how often
each is used in the DXVA VDA.

Bug: 835972
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: I39b41000b54b932ce8cc592893bef1826f458308
Reviewed-on: https://chromium-review.googlesource.com/1024927Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553236}

The ash service was being started twice because it was not marked as
a "singleton" service. Service connections with different user ids
could result in the service starting twice in the same process.

This caused CHECK failures in bluez::BluezDBusManager::Initialize
and desktopui_MashLogin to fail in the autotest lab.

Fix by marking the ash and ui services as singlton services, which
they should have been in the first place.

Bug: 836311
Test: existing autotest desktopui_MashLogin
Change-Id: I066692b19bb5d89c75e46c79675024740f42f844
Reviewed-on: https://chromium-review.googlesource.com/1026202Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553235}

Does not implement padding in spec.

BUG=822063
TEST=none

Change-Id: Ifcbcb1f7b6e4385a6c0318624d95e6201b5d5a4a
Reviewed-on: https://chromium-review.googlesource.com/1024666Reviewed-by: Allen Bauer <kylixrd@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553234}

This reverts commit ea3e3f22.

Reason for revert: Failure at
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.10%20Tests/31554

Original change's description:
> Reland "Remove HttpStreamFactory interface."
>
> Reland https://crrev.com/c/1000487.
>
> Remove HttpStreamFactory interface, also rename HttpStreamFactoryImpl
> to
> HttpStreamFactory.
>
> Bug: 829429
> Change-Id: I298879817785174d1d7290520f889f0252074e14
> Reviewed-on: https://chromium-review.googlesource.com/1023321
> Commit-Queue: Bence Béky <bnc@chromium.org>
> Reviewed-by: Ryan Hamilton <rch@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#553180}

TBR=rch@chromium.org,bnc@chromium.org

Change-Id: I9fab232c4219d9e11d5df54f57c13dfd50540b2f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 829429
Reviewed-on: https://chromium-review.googlesource.com/1026491
Commit-Queue: Bence Béky <bnc@chromium.org>
Reviewed-by: Bence Béky <bnc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553233}

https://chromium.googlesource.com/angle/angle.git/+log/66410530a78d..33e05babbc69

$ git log 66410530a..33e05babb --date=short --no-merges --format='%ad %ae %s'
2018-04-23 lucferron Vulkan: Bugfix in TextureVk::setSubImage and DynamicBuffer

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-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.


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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/8fecf0ef..034c66d8

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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

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

Change-Id: Ib1166868b004c6767f8e8db66b692ce975691d4a
Reviewed-on: https://chromium-review.googlesource.com/1026056Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553231}

Its use of feature flags causes data races.

BUG=836359
TBR=xidachen@chromium.org
NOTRY=true

Change-Id: I5bc92a6fc9b9010b9863b363dc3033b95f777762
Reviewed-on: https://chromium-review.googlesource.com/1026462Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553230}

This is a reland of be5a2a38

TBR=vadimsh@chromium.org, nodir@chromium.org, sergeyberezin@chromium.org

Original change's description:
> [cr-buildbucket] Clear 'cores' dimension for ios-* builders.
>
> ios-* builders may run on a mix of VMs with cores:8 and Mac Mini's with cores:4.
>
> Bug: 731546
> Change-Id: I056fbb5bd80ca50ef8005aa8bb99643853acd923
> Reviewed-on: https://chromium-review.googlesource.com/1024729
> Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
> Reviewed-by: Nodir Turakulov <nodir@chromium.org>
> Commit-Queue: Sergey Berezin <sergeyberezin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#553160}

Bug: 731546
Change-Id: I97848d9fccbc3c4d527d9a205150c7f6ffd920dd
Reviewed-on: https://chromium-review.googlesource.com/1025183Reviewed-by: Shuotao Gao <stgao@chromium.org>
Commit-Queue: Shuotao Gao <stgao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553229}

Bug: 810176
Change-Id: Ic78c5e6ef5557067679f3f4ed2dcf6b76f1ee8e2
Reviewed-on: https://chromium-review.googlesource.com/1014348
Commit-Queue: Erik Luo <luoe@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553228}

Change-Id: I77a57afaa10c8b5e8306c96f375c8f644e876d13
Reviewed-on: https://chromium-review.googlesource.com/1026095Reviewed-by: Matt Reynolds <mattreynolds@chromium.org>
Commit-Queue: Conley Owens <cco3@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553227}

Update SandboxedUnpackerTests to manifest v2 and remove the allowance
for manifest v1 extensions.

Repackage all the crx's used in the sandboxed unpacker tests.

Additionally, git doesn't allow empty directories to be added to the
repository, which appears to have caused problems with some of the
unpacked versions of these extensions (where the empty directory is
necessary for the test). Add the missing directories, and put in a
dummy readme file to appease git.

Bug: 816679
Change-Id: I7130bbe89079437b8ee9090ba120039e2dbe9672
Reviewed-on: https://chromium-review.googlesource.com/1024910Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553226}