- 24 Oct, 2018 40 commits
- 
- 
Marc Treib authoredThis reverts commit 538e9529. Reason for revert: "Windows deterministic" has been consistently failing on the main waterfall: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Windows%20deterministic First failing build: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Windows%20deterministic/10487 Error message from the lookup_GN_args step looks like this: Traceback (most recent call last): File "C:\b\swarming\w\ir\kitchen-workdir\src\tools\mb\mb.py", line 63, in Main ret = self.args.func() File "C:\b\swarming\w\ir\kitchen-workdir\src\tools\mb\mb.py", line 332, in CmdLookup vals = self.Lookup() File "C:\b\swarming\w\ir\kitchen-workdir\src\tools\mb\mb.py", line 614, in Lookup config = self.ConfigFromArgs() File "C:\b\swarming\w\ir\kitchen-workdir\src\tools\mb\mb.py", line 696, in ConfigFromArgs (self.args.builder, self.args.master, self.args.config_file)) MBErr: Builder name "Windows deterministic" not found under masters[chromium.fyi] in "C:\b\swarming\w\ir\kitchen-workdir\src\tools\mb\mb_config.pyl" Original change's description: > Move 'Windows deterministic' from FYI to the main waterfall. > > Bug: 897980,330260 > Change-Id: I71bb304c6398484d5af5a981fdbc09314d69dd22 > Reviewed-on: https://chromium-review.googlesource.com/c/1295272 > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Commit-Queue: Nico Weber <thakis@chromium.org> > Cr-Commit-Position: refs/heads/master@{#602198} TBR=thakis@chromium.org,dpranke@chromium.org,martiniss@chromium.org Change-Id: Ib9aff7c076f1f0e3192788f10da80036b4262ed1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 897980, 330260 Reviewed-on: https://chromium-review.googlesource.com/c/1297139Reviewed-by: Marc Treib <treib@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#602297} 
- 
v8-ci-autoroll-builder authoredSummary of changes available at: https://chromium.googlesource.com/v8/v8/+log/2e57da0a..a6bda76c 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I2a4e866ed3607f9c73e359d311496fb69971b04c Reviewed-on: https://chromium-review.googlesource.com/c/1297723Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602296} 
- 
Mikel Astiz authoredMost if not all sync integration tests output this red-herring and annoying trace: "Skipping unexpected progress marker for non-enabled type <type>". The reason is that, prior to this patch, the loopback server returned progress markers for all datatypes that the server knows about (i.e. at least one entity exists). Instead, let's mimic the real server and return progress markers for types that have been requested by the client. Bug: None Change-Id: I7354c0f38afbd71cf06ab3e7e7c4113f44db6ee2 Reviewed-on: https://chromium-review.googlesource.com/c/1296545 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#602295} 
- 
chromium-internal-autoroll authoredhttps://chrome-internal.googlesource.com/chrome/src-internal.git/+log/0b9c09194c6a..624fe7f9b061 Created with: gclient setdep -r src-internal@624fe7f9b061 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: I8cf0eadd802c752c0eef02ac29afb3876987266b Reviewed-on: https://chromium-review.googlesource.com/c/1297335Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602294} 
- 
Friedrich Horschig authoredBefore this CL, several places (esp. in browser/) would override the instance of the keyboard delegate in ui/. This caused an inconsistent, hard to track state of the KeyboardVisibilityDelegate. With this CL, the ui/ takes control of setting the delegate and specific browser-side implementations are called by using the appropriate factory methods. For that, the KeyboardVisibilityDelegate (brief: KVD) uses sublasses with very clearly split responsibilities: KeyboardVisibilityDelegate (ui/; public interface for all clients) <-- ActivityKVD (ui/; propagates visibility changes to listeners) <-- SingleWindowKVD (browser/; prevents multi-window use) <-- ChromeKVD (browser/ separates extensions/native) <-- FakeKeyboard (javatests/; mocks native keyboard) The CL actually has no functional changes, it performs these moves: - WindowAndroid Listeners moved into KeyboardVisibilityDelegate - AcivityWindowAndroid callbacks moved into ActivityKVD - ProcessInitializationHandler delegate became SingleWindowKVD - ManualFillingTestHelper's FakeKeyboard became a new, public class Functional changes follow with https://crrev.com/c/1286426 and mainly affect the ChromeKVD. Purpose of isAndroidSoftKeyboardShowing(...) and hideAndroidSoftKeyboardKeyboard(...): - Template methods to simplify mocking the calls to InputMethodManager - Ensures access to keyboard without extensions in subclasses (e.g. Chrome KVD) - prevents use of @CallSuper Change-Id: I06cf50f48625d2d883efc8cd66a3d062204e51b2 Reviewed-on: https://chromium-review.googlesource.com/c/1286649 Commit-Queue: Friedrich Horschig [EDT] <fhorschig@chromium.org> Reviewed-by:Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#602293} 
- 
chromium-autoroll authoredhttps://webrtc.googlesource.com/src.git/+log/41ed3e083ed5..700b4a4e65c2 git log 41ed3e083ed5..700b4a4e65c2 --date=short --no-merges --format='%ad %ae %s' 2018-10-24 peah@webrtc.org AEC3: Allow limiting dominant nearend to the non-initial phase Created with: gclient setdep -r src/third_party/webrtc@700b4a4e65c2 The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-chromium-autoroll 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:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng BUG=chromium:898273 TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I3ae8759bf93d26497f457efeef579669fbc32ee0 Reviewed-on: https://chromium-review.googlesource.com/c/1297332Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602292} 
- 
Marc Treib authoredBefore SyncAuthManager::RegisterForAuthNotifications(), the active/authenticated account isn't initialized, so the histogram would always record "not signed in". So make sure we record the histogram only after registering for auth. Bug: 898430 Change-Id: I00064541cefe6cb65d5a246f12c7a8eaef469c28 Reviewed-on: https://chromium-review.googlesource.com/c/1297420Reviewed-by: Mikel Astiz <mastiz@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#602291} 
- 
chromium-autoroll authoredhttps://android.googlesource.com/platform/external/perfetto.git/+log/ea62998cdf18..a1e7b1c440e1 git log ea62998cdf18..a1e7b1c440e1 --date=short --no-merges --format='%ad %ae %s' 2018-10-24 hjd@google.com Merge "perfetto-ui: A number of small improvements" Created with: gclient setdep -r src/third_party/perfetto@a1e7b1c440e1 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll 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=perfetto-bugs@google.com Change-Id: I8cec73ea3005cfbf43ed5edcc696af4bb01803a8 Reviewed-on: https://chromium-review.googlesource.com/c/1297333Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602290} 
- 
Marc Treib authoredThis reverts commit af460e9a. Reason for revert: Seems to have broken paint/invalidation/table/cached-69296.html from webkit_layout_tests on Mac 10.11 - currently failed 7/7 times after this landed: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests First failing build: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/30030 Original change's description: > [CI] Combine GetBackgroundPaintLocation of PaintLayer and LayoutBoxModelObject > > For fixing SlimmingPaintV2 scrolling background painting and > invalidation, we need a single place to get the accurate value > of the background paint location. > > Bug: 732611 > > Change-Id: I0ad696395d73c9c19078e13e9a32cf9dbd0c628f > Reviewed-on: https://chromium-review.googlesource.com/c/1297586 > Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> > Reviewed-by: Robert Flack <flackr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#602250} TBR=flackr@chromium.org,wangxianzhu@chromium.org Change-Id: I279520fc9deafd9891dd3a5457e42d6fa0b201aa No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 732611 Reviewed-on: https://chromium-review.googlesource.com/c/1297362Reviewed-by: Marc Treib <treib@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#602289} 
- 
Joel Hockey authoredDepends on https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1287389 https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1287390 Bug: 878324 Bug: 890649 Change-Id: I20b57cc7900727fa15ef2de12d247052dd040bff Reviewed-on: https://chromium-review.googlesource.com/c/1288095 Commit-Queue: Joel Hockey <joelhockey@chromium.org> Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org> Reviewed-by: Nicholas Verne <nverne@chromium.org> Reviewed-by: Sam McNally <sammc@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#602288} 
- 
chromium-internal-autoroll authoredhttps://chrome-internal.googlesource.com/chrome/src-internal.git/+log/84fbe1444f3c..0b9c09194c6a Created with: gclient setdep -r src-internal@0b9c09194c6a The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: I5a8e2aae26c599ffbe67598f524b037e66e7863d Reviewed-on: https://chromium-review.googlesource.com/c/1297331Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602287} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium 950fb63e. With Chromium commits locally applied on WPT: f79c6fe7 "Reimplementing getDatabaseNames() as databases() for indexeddb." 57cc1d42 "Added 304 CSP test and removed "content-*" prefix from headers to ignore on 304" 89bcea96 "[run_web_tests] Remove extra baselines after crrev.com/c/1277654" 36e64bc7 "Updating tests for Window onerror is not triggered bug fix." Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/26627 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=kyleju No-Export: true Change-Id: Iaf4edd35b07965fff3160503d3140be9a40950c8 Reviewed-on: https://chromium-review.googlesource.com/c/1297720 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#602286} 
- 
Nico Weber authoredhttps://chromium-review.googlesource.com/c/1295272 moved the bot on the console and updated mb_config.pyl. But once the bot had synced in the mb_config.pyl change, it started failing with lookup GN args Failed to extract GN args from output of "mb lookup" The lookup step still passes in `-m chromium.fyi`, and due to the mb_config.pyl change that no longer works. My best guess is that the mastername comes from the cr-buildbucket.cfg file, so change the master here too. (Seems like a nice thing to do in general.) TBR=dpranke Bug: 897980 Change-Id: I95fe0b7470ec39eb8ebfd0dc5a57980d598b0e46 Reviewed-on: https://chromium-review.googlesource.com/c/1297262Reviewed-by: Nico Weber <thakis@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#602285} 
- 
Nico Weber authoredBug: 324079 Change-Id: I6a98e713094a8a158d7ef227efd77e75a79be810 Reviewed-on: https://chromium-review.googlesource.com/c/1296681Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by: Asanka Herath <asanka@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#602284} 
- 
Olivier Robin authoredBug: 897341 Change-Id: I311db2e8f685bae5aa2b218750ddaf6b7a4c02f0 Reviewed-on: https://chromium-review.googlesource.com/c/1296170Reviewed-by: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by: Mike Dougherty <michaeldo@chromium.org> Commit-Queue: Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#602283} 
- 
Gauthier Ambard authoredThis CL fixes the way the utils are checking if the preferred content size category is an accessibility category for iOS 10. Even if iOS 10 support will be dropped, the code should be correct. Bug: 895804 Change-Id: I4ba1066b7eef712534da876777013476513c060a Reviewed-on: https://chromium-review.googlesource.com/c/1297417Reviewed-by: Mark Cogan <marq@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#602282} 
- 
Gauthier Ambard authoredNew files are needed by the MDC repo. Bug: 898170 Change-Id: Ib48862834df9d677db459e5efccee761269f49a9 Reviewed-on: https://chromium-review.googlesource.com/c/1296500Reviewed-by: Mark Cogan <marq@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#602281} 
- 
Leszek Swirski authoredThis reverts commit 42aaff47. Reason for revert: Blocking a revert for https://crbug.com/894796 Original change's description: > [blink] Remove work stealing > > Work stealing is deprecated by script streaming on the task scheduler, > and broken anyway by ScriptResource-controlled script streaming. > > Bug: 557466 > Bug: 761977 > Bug: 865098 > Bug: 866868 > Change-Id: Ie01d239b052ad7499e9884687bf95376a5cf6f9a > Reviewed-on: https://chromium-review.googlesource.com/c/1258168 > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Reviewed-by: Kouhei Ueno <kouhei@chromium.org> > Reviewed-by: Daniel Vogelheim <vogelheim@chromium.org> > Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org> > Reviewed-by: Jochen Eisinger <jochen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#601921} TBR=hiroshige@chromium.org,kouhei@chromium.org,vogelheim@chromium.org,leszeks@chromium.org,jochen@chromium.org Change-Id: I0017e6580c01f5da934a6a13af0b8b01641c0c39 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 557466, 761977, 865098, 866868 Reviewed-on: https://chromium-review.googlesource.com/c/1297421Reviewed-by: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#602280} 
- 
chromium-internal-autoroll authoredhttps://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f426f506dc36..84fbe1444f3c Created with: gclient setdep -r src-internal@84fbe1444f3c The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: Id7eeb57a1f3e93921463de2a450e70b3b1663f71 Reviewed-on: https://chromium-review.googlesource.com/c/1297330Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602279} 
- 
Eric Willigers authoredThe functional pseudo-class like :matches() with 0 specificity is now named :where(). https://drafts.csswg.org/selectors/#zero-matches BUG=898215 Change-Id: Ic4d3feccc2fca566e54bb0076b4248b7958788c2 Reviewed-on: https://chromium-review.googlesource.com/c/1296606Reviewed-by: Rune Lillesveen <futhark@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#602278} 
- 
v8-ci-autoroll-builder authoredSummary of changes available at: https://chromium.googlesource.com/v8/v8/+log/130258cf..2e57da0a 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I374ca00ae3b01c20a26ff02b7ab611dcab9a7fbc Reviewed-on: https://chromium-review.googlesource.com/c/1297737Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602277} 
- 
Fumitoshi Ukai authoredBug: 835732 Change-Id: I04f6e2cbb3808355d4d9d86b99dbffa34ecd6f74 Reviewed-on: https://chromium-review.googlesource.com/c/1290711Reviewed-by: Shinya Kawanaka <shinyak@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Fumitoshi Ukai <ukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#602276} 
- 
chromium-internal-autoroll authoredhttps://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4257a53431af..f426f506dc36 Created with: gclient setdep -r src-internal@f426f506dc36 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: I2b936396d9882af849bb14996c55febde71a44ab Reviewed-on: https://chromium-review.googlesource.com/c/1296988Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602275} 
- 
Alexander Hendrich authoredWith network service we can't modify cookie headers anymore to store the start/end times of GAIA redirects to/from SAML IdP. Instead we will only use the domain based filtering (*google.<TLD> or *youtube.<TLD>) to filter out GAIA cookies. Bug: 887061 Change-Id: Iac1c2a9b267b61a788269931ab8318c13a7a0666 Reviewed-on: https://chromium-review.googlesource.com/c/1296539Reviewed-by: John Abd-El-Malek <jam@chromium.org> Commit-Queue: Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#602274} 
- 
chromium-autoroll authoredhttps://skia.googlesource.com/skia.git/+log/49353683320d..70bd0b72f3f8 git log 49353683320d..70bd0b72f3f8 --date=short --no-merges --format='%ad %ae %s' 2018-10-24 mtklein@google.com Revert "Some more clean-up of YUVA code" 2018-10-24 csmartdalton@google.com ccpr: Harden the path cache 2018-10-23 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-10-23 csmartdalton@google.com ccpr: Don't use the GrContext id for the path cache id 2018-10-23 jvanverth@google.com Some more clean-up of YUVA code Created with: gclient setdep -r src/third_party/skia@70bd0b72f3f8 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel BUG=chromium:897510,chromium:897413,chromium:897245,chromium:897507,chromium:897510,chromium:897413,chromium:897245,chromium:897507 TBR=brianosman@chromium.org Change-Id: I681acc9f101d006ae523d155a7c0975098f166c9 Reviewed-on: https://chromium-review.googlesource.com/c/1297829Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602273} 
- 
Jan Krcal authoredThis CL makes the old Directory controller react to the toggle in Payments methods as well. Bug: 898141 Change-Id: I09918c9aace663376f7beb0ed0cd5f2ef3854b0e Reviewed-on: https://chromium-review.googlesource.com/c/1296598 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by: Florian Uunk <feuunk@chromium.org> Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#602272} 
- 
Gauthier Ambard authoredThis CL moves the util files living in ios/c/b/ui to the correct folder ios/c/b/ui/util. Bug: none Change-Id: Icbd9b1cfe17b8af434f48647446071843e6e37ea Reviewed-on: https://chromium-review.googlesource.com/c/1296534 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by: Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#602271} 
- 
Ramin Halavati authoredTraffic annotation auditor binaries for Windows and Linux are updated after the change crrev.com/c/1291813. Bug: 876855 Change-Id: I7490b23e192e4c02ca753cae5f3762758deccfb2 TBR: nicolaso@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/1297358Reviewed-by: Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#602270} 
- 
Sergey Ulanov authoredFuchsia's mediacodec API uses NV12 to encode decoded output frames, so it's useful to support them in VideoFrame and the renderer. Bug: 876519 Change-Id: I4863cfbf47fb9e2ba034b239eaa7f485e1f87dd4 Reviewed-on: https://chromium-review.googlesource.com/c/1297601Reviewed-by: Dan Sanders <sandersd@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#602269} 
- 
Alexey Baskakov authoredThis unifies the API for both old and new implementations of InstallManager interface. Bug: 891172 Change-Id: I95086c897c4fe3d6f58bf8e8e88050b0e5692118 Reviewed-on: https://chromium-review.googlesource.com/c/1295616 Commit-Queue: Alexey Baskakov <loyso@chromium.org> Reviewed-by: Ben Wells <benwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#602268} 
- 
Joel Hockey authoredchrome.fileManagerPrivate.sharePathWithCrostini changed name to chrome.fileManagerPrivate.sharePathsWithCrostini and takes array of Entry in order to simplify case where multiple paths are being shared at a single time. Requires a custom barrier closure in CrostiniSharePath to capture the first instance of an error and respond with it when done. Bug: 878324 Change-Id: I1f308028abda9db92990cc73ad8d7598aee118fe Reviewed-on: https://chromium-review.googlesource.com/c/1293094 Commit-Queue: Joel Hockey <joelhockey@chromium.org> Reviewed-by: Nicholas Verne <nverne@chromium.org> Reviewed-by: Ben Wells <benwells@chromium.org> Reviewed-by: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#602267} 
- 
chromium-autoroll authoredThis CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 Please note that, despite rolling to chrome/android, this profile is used for both Linux and Android. The AutoRoll server is located here: https://autoroll.skia.org/r/afdo-chromium-autoroll 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=gbiv@chromium.org Change-Id: Iad5c8952daf2805026e23a5340e558861fd79710 Reviewed-on: https://chromium-review.googlesource.com/c/1296987Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602266} 
- 
Mikel Astiz authoredThese features are enabled by default with M71 and seem effective according to experiments on canary&dev, so there's no need to keep these feature toggles around. These features may also be the cause for certain unrelated test flakiness on TSAN, presumably because test-only feature overrides are not thread safe. Bug: 887068,892550 Change-Id: Ia57e1f161f726db05e2322568da51d3681df8751 Reviewed-on: https://chromium-review.googlesource.com/c/1296504Reviewed-by: Jan Krcal <jkrcal@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#602265} 
- 
chromium-internal-autoroll authoredhttps://chrome-internal.googlesource.com/chrome/src-internal.git/+log/10ebffa1d5f3..4257a53431af Created with: gclient setdep -r src-internal@4257a53431af The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: Iea121ce6bf45664306eccb27f72ef601214e90e5 Reviewed-on: https://chromium-review.googlesource.com/c/1296986Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#602264} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium 50fe66c6. With Chromium commits locally applied on WPT: f79c6fe7 "Reimplementing getDatabaseNames() as databases() for indexeddb." 57cc1d42 "Added 304 CSP test and removed "content-*" prefix from headers to ignore on 304" 89bcea96 "[run_web_tests] Remove extra baselines after crrev.com/c/1277654" 36e64bc7 "Updating tests for Window onerror is not triggered bug fix." Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/26619 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=markdittmer No-Export: true Change-Id: Ia321142a2749c26fbce00751f4f2d3bff01314c4 Reviewed-on: https://chromium-review.googlesource.com/c/1297736 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#602263} 
- 
Alan Cutter authoredMark virtual/scroll_customization/fast/scroll-behavior/smooth-scroll/keyboard-scroll.html as timing out This is a follow up to https://chromium-review.googlesource.com/c/1297733 to also cover the virtual version of this test. TBR=sahel@chromium.org NOTRY=true Bug: 898378 Change-Id: I2c9748f00f6e58b287c0f50c2b68f312d104dd2c Reviewed-on: https://chromium-review.googlesource.com/c/1297750Reviewed-by: Alan Cutter <alancutter@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#602262} 
- 
David Jacobo authoredReceive usage metrics for sharing files feature in ARC, then store them via UMA. Bug: b/112469474 Change-Id: I4f34475ab0c38bae2ed9a2bb4e02a0654553a814 Reviewed-on: https://chromium-review.googlesource.com/c/1289516 Commit-Queue: David Jacobo <djacobo@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Reviewed-by: Yusuke Sato <yusukes@chromium.org> Reviewed-by: Greg Kerr <kerrnel@chromium.org> Cr-Commit-Position: refs/heads/master@{#602261} 
- 
Alan Cutter authoredTBR=pdr@chromium.org Bug: 898394 Change-Id: I0286d9374cd941546798de6f6507c8f81f436c7d Reviewed-on: https://chromium-review.googlesource.com/c/1297770Reviewed-by: Alan Cutter <alancutter@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#602260} 
- 
Tetsui Ohkubo authoredIn order to look visually consistent with UnifiedSystemTray, notification popup ordering should be inverted on Chrome OS. To implement that, this CL adds MOVE_UP_FOR_INVERSE animation state to MessagePopupCollection. TEST=MessagePopupCollectionTest BUG=876769 Change-Id: I0dcb5c20c8bc3e76f44b32c989b7c935d0e2f676 Reviewed-on: https://chromium-review.googlesource.com/c/1290450 Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Cr-Commit-Position: refs/heads/master@{#602259} 
- 
Antonio Gomes authoredBUG=890771 Change-Id: I9fbb028db7c7e4c259885aa804ef475bb79deb47 Reviewed-on: https://chromium-review.googlesource.com/c/1296872Reviewed-by: Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#602258} 
 
-