- 18 Oct, 2018 40 commits
-
-
Rune Lillesveen authored
Just confusing to have separate group for core owners. Change-Id: I8d63d3a47e88c42d00fdaceac9c417ebdb0af711 Reviewed-on: https://chromium-review.googlesource.com/c/1288393Reviewed-by:
Anders Ruud <andruud@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#600719}
-
Kenichi Ishibashi authored
According to the comment on DiskCache::ReadData[1], it's not guaranteed that the completion callback is called when the read request immediately succeeds. We should use TRACE_EVENT_ASYNC_BEGIN1() macro only after ERR_IO_PENDING because otherwise async events never finish. [1] https://cs.chromium.org/chromium/src/net/disk_cache/disk_cache.h?l=301&rcl=93f9ef9762b0c9158dd8acb0ceb52dbd14a308a3 Change-Id: Ie70730b8dfffd0777a68eabd9970216054cdc7ab Bug: 896604 Reviewed-on: https://chromium-review.googlesource.com/c/1288336 Commit-Queue: Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#600718}
-
Gauthier Ambard authored
This CL creates a TableView item for handling a footer which would display a link in a TextView. This is designed as a replacement for CollectionViewFooterItem. Bug: 894791 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Id4b84e5b17920237016314447bed9a25140ab30c Reviewed-on: https://chromium-review.googlesource.com/c/1286416 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#600717}
-
Anders Hartvoll Ruud authored
For custom properties registered with <integer>, it should according to CSS Values and Units Level 4 be possible to apply non-integral values, and then have the value round to the nearest integer computed value time. To do this, this CL ...: * Changes the parsing of <integer> values such that non-integral calc() expressions are allowed. * Recognizes such calc()-expressions computed-value time, resolves them, and rounds them to the nearest integer. * Wraps values such as CSS.number(1.5) in calc() if necessary, when setting such values using Typed OM. R=futhark@chromium.org Bug: 641877 Change-Id: I2d4d7c4d72c9b2069f2fe10b1be1b4b94d5900e2 Reviewed-on: https://chromium-review.googlesource.com/c/1280663 Commit-Queue: Anders Ruud <andruud@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#600716}
-
Colin Blundell authored
This is possible now that we have infrastructure that glues an IdentityTestEnvironment instance to a Profile, and it enables removing the test's knowledge of SigninManager. Bug: 896180 Change-Id: I074c0d2886561b264612ce2d34c5ca41b81de3be Reviewed-on: https://chromium-review.googlesource.com/c/1286810Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#600715}
-
Christian Dullweber authored
The HiDPI version of the test is flaky on windows bots as well. TBR: kenrb@chromium.org Bug: 896609, 882458 Change-Id: Ifc8fdb8f33ce319398566b22737fcfe7f8f3d972 Reviewed-on: https://chromium-review.googlesource.com/c/1288532 Commit-Queue: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#600714}
-
Sam McNally authored
This is a reland of 9ecd2358 that: - Updates FakeDriveFs to create a .Trash directory. - Move files directly instead of delegating to MoveFileLocal. - Leave (non-recursive) DeleteDirectory unchanged. Original change's description: > Turn DriveFS directory deletes into moves into a trash directory. > > Change NativeFileUtil to allow moving a directory to an existing > directory. > > Bug: 829696 > Change-Id: I6e233fa2a9ad161d1457d06fc6b6eda3ab3955d7 > Reviewed-on: https://chromium-review.googlesource.com/c/1278624 > Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org> > Reviewed-by: Sergei Datsenko <dats@chromium.org> > Commit-Queue: Sam McNally <sammc@chromium.org> > Cr-Commit-Position: refs/heads/master@{#600283} Bug: 829696 Change-Id: Ied7836010a00a8f3226b8f65e02568b3f2c2a47d Reviewed-on: https://chromium-review.googlesource.com/c/1286025 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Reviewed-by:
Sergei Datsenko <dats@chromium.org> Cr-Commit-Position: refs/heads/master@{#600713}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/519565187c85..3d878160972c git log 519565187c85..3d878160972c --date=short --no-merges --format='%ad %ae %s' 2018-10-18 cbruni@chromium.org [telemetry] Support unicode keys in KeyPressAction 2018-10-18 bpastene@chromium.org devil: Exit w/ a non-zero code when unlock_bootloader.py fails. 2018-10-17 nharper@chromium.org netlog_viewer: match QUIC sessions table rows with headers 2018-10-17 mlippautz@chromium.org V8 GC metrics: Add GC cycle time 2018-10-17 nharper@chromium.org Fix path for serving netlog_viewer locally in README.md Created with: gclient setdep -r src/third_party/catapult@3d878160972c The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:878390,chromium:843903 TBR=sullivan@chromium.org Change-Id: Iaa3306106064794f6b10421d28eaddff58126f5c Reviewed-on: https://chromium-review.googlesource.com/c/1288024Reviewed-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@{#600712}
-
Kent Tamura authored
https://google.github.io/styleguide/cppguide.html#Namespace_Names > Namespace names are all lower-case. This CL has no behavior changes. Bug: 889726 Change-Id: I2efd3b383325c3df5996fd9dda2e7ab5da01c5aa Reviewed-on: https://chromium-review.googlesource.com/c/1288517Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#600711}
-
Jordan Demeulenaere authored
Bug: 806868 Change-Id: I3948803d43dfd91ccec1e6fe0fa57b4d197de1e6 Reviewed-on: https://chromium-review.googlesource.com/c/1286148 Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Reviewed-by:
Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#600710}
-
David Jean authored
Video: https://drive.google.com/open?id=16GEAtsIQLbSjYlrIC3a1hby8gVxjgYAz Bug: 891299 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I8601ac0eb934e42dbe6ca6389b91ed35d3228964 Reviewed-on: https://chromium-review.googlesource.com/c/1264383 Commit-Queue: David Jean <djean@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#600709}
-
Michael Lippautz authored
This reverts commit 2018a9ef. Reason for revert: Reverting as this was fixed on the V8 side and this restores a consistent state where all callbacks assume non-null ThreadState. Original change's description: > bindings: Fix second pass callbacks fired during teardown sequence > > Work around the problem where second pass callbacks are fired during > teardown. > > Previously, these callbacks where just dropped which is not correct for > other reason, e.g., when components keep a state per process and not > just per Isolate. > > Bug: chromium:890631 > Change-Id: I1fd66f9ffee86c5c7bff053303c741c4dedefdb0 > Reviewed-on: https://chromium-review.googlesource.com/c/1253782 > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Commit-Queue: Michael Lippautz <mlippautz@chromium.org> > Cr-Commit-Position: refs/heads/master@{#597525} TBR=haraken@chromium.org,mlippautz@chromium.org,clemensh@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:890631 Change-Id: I8d6f55fe926093ba776d182d9c154ee0b6538ed1 Reviewed-on: https://chromium-review.googlesource.com/c/1288270Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#600708}
-
Takuto Ikuta authored
This CL is part of https://chromium-review.googlesource.com/c/chromium/src/+/1286016 This CL was uploaded by git cl split. R=haraken@chromium.org Bug: 753973 Change-Id: Ife13d645bede991ae8959759be878ed5bdf3733e Reviewed-on: https://chromium-review.googlesource.com/c/1288156Reviewed-by:
-
Gauthier Ambard authored
This CL adds a class to ease testing of subclasses of ChromeTableViewController, in the same way CollectionViewControllerTest is easing the testing of subclasses of CollectionViewController. Bug: 894791 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I159849612b16d5475d581c79601f93de04e6b705 Reviewed-on: https://chromium-review.googlesource.com/c/1283025 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
-
Marc Treib authored
In some circumstances, ProfileSyncService wants to fully reset SyncServiceCrypto's state. Currently, it destroys and recreated the whole object. That works, but it's problematic if we want to pass pointers to the SyncServiceCrypto object elsewhere. Instead, this CL adds a Reset() method to SyncServiceCrypto. For convenient implementation, and to make sure *everything* gets reset, all the mutable state gets wrapped into a new struct. Bug: 884159 Change-Id: Ia2590301a6236461e4df20d7f83fee5599e5483c Reviewed-on: https://chromium-review.googlesource.com/c/1286424 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Cr-Commit-Position: refs/heads/master@{#600705}
-
Sergio Villar Senin authored
Currently using the SigninManager API to get the primary (authenticated) account. Migrated it to IdentityManager so it could eventually use the identity service. Bug: 890772 Change-Id: I22cadc9256589885c002a0cabe1e3a174b0504ad Reviewed-on: https://chromium-review.googlesource.com/c/1282973Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Commit-Queue: Sergio Villar <svillar@igalia.com> Cr-Commit-Position: refs/heads/master@{#600704}
-
Zhuoyu Qian authored
https://google.github.io/styleguide/cppguide.html#Namespace_Names > Namespace names are all lower-case. This CL has no behavior changes. Bug: 889726 Change-Id: Ie72fbbaa5ffae8a15fb07356d0e190fd3d140e5c Reviewed-on: https://chromium-review.googlesource.com/c/1288111Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Zhuoyu Qian <zhuoyu.qian@samsung.com> Cr-Commit-Position: refs/heads/master@{#600703}
-
https://crbug.com/889567Makoto Shimazu authored
I'm suspecting that CHECK(context_) will hit. Bug: 889567 Change-Id: I1e839185dc86f33ed9d8da6fb93c0ecf63a3d2a5 Reviewed-on: https://chromium-review.googlesource.com/c/1288097 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#600702}
-
Takuto Ikuta authored
This CL is part of https://chromium-review.googlesource.com/c/chromium/src/+/1286017 This CL was uploaded by git cl split. R=yosin@chromium.org Bug: 681136 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I6bd5a1a0e8a2050c579cf459e243c5b63185b1f6 Reviewed-on: https://chromium-review.googlesource.com/c/1288515Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#600701}
-
Daniel Vogelheim authored
.mjs is apparently being used as an extension for JavaScript ES6 modules. (Example: https://developers.google.com/web/fundamentals/primers/modules) Change-Id: I0cb88cf01cb468214e0ccbd8fab89ea5040c9c84 Reviewed-on: https://chromium-review.googlesource.com/c/1286132Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org> Cr-Commit-Position: refs/heads/master@{#600700}
-
Anders Hartvoll Ruud authored
Unfortunately, the existing test for this does too much behind a single green square. If even a single assertion were to fail, it would hide the result of all the other rests. At the same time, adding 40something tests is perhaps a bit excessive, so I've tried to reduce that number somewhat (mostly by only testing one of the initial value cases, which I think is reasonable). The aspect of the deleted test which verified that values were observable from all API entry points (has, get, getAll, size and iterator) is now a separate test as well. R=ikilpatrick@chromium.org Bug: 641877 Change-Id: Ie9316583ccef0ee4ddcaf688b7c1fac422649a39 Reviewed-on: https://chromium-review.googlesource.com/c/1286855Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Anders Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#600699}
-
Anders Hartvoll Ruud authored
R=ikilpatrick@chromium.org Bug: 641877 Change-Id: Ic0403289f9deaf97369962d6901bafb58dc46931 Reviewed-on: https://chromium-review.googlesource.com/c/1283018Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/48ee86f818fb..dd254486280d git log 48ee86f818fb..dd254486280d --date=short --no-merges --format='%ad %ae %s' 2018-10-18 skia-bookmaker@skia-swarming-bots.iam.gserviceaccount.com Update markdown files 2018-10-18 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update go_deps asset Created with: gclient setdep -r src/third_party/skia@dd254486280d 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 TBR=egdaniel@chromium.org Change-Id: Ic35e86e6b58e135c21009ff512cd4d26667d3ff7 Reviewed-on: https://chromium-review.googlesource.com/c/1288021Reviewed-by:
-
Tatsuhisa Yamaguchi authored
Some third-party archivers generate ZIP files which claim file names are coded in CP437 but actually put file names in other coding system. This change attempts to guess which encoding system they are, and decodes the file and directory names. Not all detected encodings are supported by the TextDecoder. There is also coding name difference between CED and TextDecoder. By this reason the set of encodings to be handled is whitelisted and translated by a mapping. We can simply add mappings to support other encodings. We also assume that all the files are coded in the uniform coding system, so as to stabilize the automated coding system detection. Bug: 846195 Test: browser_tests --gtest_filter='*ZipFiles*FilesAppBrowserTest*' --gtest_also_run_disabled_tests Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: I284c189431d4fbac5554fb62507f1646517d279d Reviewed-on: https://chromium-review.googlesource.com/c/1251684 Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Reviewed-by:
Anand Mistry <amistry@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#600696}
-
Colin Blundell authored
Forgot to do this in the CLs converting away from these utils. TBR=sdefresne@chromium.org Bug: 896180 Change-Id: Ifa1766efc5ec91c5313658d65fdfc0fc48bc9fe1 Reviewed-on: https://chromium-review.googlesource.com/c/1288529Reviewed-by:
-
Hirokazu Honda authored
This enables VideoFrameLayout (VFL) to be created always by factory functions. They return base::Optional. With invalid configured values, they should return base::nullopt. However, VFL is often created in VideoFrame's factory functions. Some of VideoFrame factory function's input rule is unclear. It may leads to pass invalid values to VFL and returns nullptr as VideoFrame. Becuase VideoFrame is widely used on Chrome video stack on any platform, it is dangerous to change a logic at one time. In this CL, VFL's factory functions always create VFL. Therefore, a logic in VideoFrame is unchanged. This also changes buffer_sizes is empty if it is not specified. BUG=896135 TEST=media_unittests on Linux TEST=VEA and VDA unittest on eve 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: Ibf4191f2e8674ccff084ac8fb4d7e48f89594d6c Reviewed-on: https://chromium-review.googlesource.com/c/1250682 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Alexandre Courbot <acourbot@chromium.org> Cr-Commit-Position: refs/heads/master@{#600694}
-
Alexandr Ilin authored
This CL replaces the deprecated base::SharedMemoryHandle in gfx::GpuMemoryBufferHandle with the base::UnsafeSharedMemoryRegion. 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;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I527a84876f8b87ddc652cf2f780a2893cb419c46 Bug: 854594, 863011 Reviewed-on: https://chromium-review.googlesource.com/c/1106340 Commit-Queue: Alexandr Ilin <alexilin@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#600693}
-
sangwoo.ko authored
Replace old API with new API. This CL is a refactor and has no intended behavior change. Bug: 842194 Change-Id: I51591bad6d084f2200fca07deb6b0e90a6e45ba2 Reviewed-on: https://chromium-review.googlesource.com/c/1246841 Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#600692}
-
Takuto Ikuta authored
This CL is part of https://chromium-review.googlesource.com/c/chromium/src/+/1286017 This CL was uploaded by git cl split. R=peria@chromium.org Bug: 681136 Change-Id: I9dce2f01375277b9f471e65d30479ec91689e478 Reviewed-on: https://chromium-review.googlesource.com/c/1288496Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#600691}
-
Takuto Ikuta authored
This CL is part of https://chromium-review.googlesource.com/c/chromium/src/+/1286017 This CL was uploaded by git cl split. R=jdoerrie@chromium.org Bug: 681136 Change-Id: I948c064bed03c801088e96007b641be2412bc6ee Reviewed-on: https://chromium-review.googlesource.com/c/1288509 Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#600690}
-
Kinuko Yasuda authored
Especially on Android. Wasn't able to find a strong culprit, disabling it for now. TBR=ahest@chromium.org Bug: 896466 Change-Id: I3e063094b2fb3888c10a3366d24d6ea9145fddf0 No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/1288331Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#600689}
-
Matt Falkenhagen authored
I suspect the errors were that we were getting no preload request and two fallback requests, which seems plausible based on the existing comment. Loosen the test expectations accordingly. Bug: 896230 Change-Id: I539b56343d5057975352f74b482b61583174fb13 Reviewed-on: https://chromium-review.googlesource.com/c/1288330 Commit-Queue: Matt Falkenhagen <falken@chromium.org> Reviewed-by:
-
chrome://flagsMichael Lippautz authored
There should be no reason to override this from the UI anymore. Also fix flag toggle. The toggle was only working from 'default false' -> 'enabled' but could not be used to explicitly disable the feature. Tbr: jochen@chromium.org Bug: 757440 Change-Id: I732c508503692437f9780d745f43d9bc4f09a611 Reviewed-on: https://chromium-review.googlesource.com/c/1286857 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Yoshifumi Inoue authored
This patch renames ContainsOnlyWhitespace() to ContainsOnlyWhitespaceOrEmpty() to avoid ambiguity of return value of empty string for improving readability. This patch is done by global replace and manual changes fro removing |IsEmpty()| from |str.IsEmpty() || str.ContainsOnlyWhitespaceOrEmpty()| in * third_party/blink/renderer/core/css/properties/css_parsing_utils.cc * third_party/blink/renderer/core/editing/selection_controller.cc This patch is a preparation of the patch[1]. [1] http://crrev.com/c/1288176 Make WhitespaceAttacher to ignore empty Text node Bug: 894674 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I144ef97f2b016a1bb39e6c3ef5fc5d850865d009 Reviewed-on: https://chromium-review.googlesource.com/c/1288157Reviewed-by:
-
Christopher Cameron authored
It is no longer the case that a gfx::NativeView/Window refers to an NSView/Window in the same process. Because of RemoteMacViews, they may refer to an NSView/Window that is in another process. Make gfx::NativeView/Window be a pass-by-value struct, and make converting from a gfx::NativeView/Window to an NSView/Window require an explicit conversion function. Mechanically add this conversion as needed. Also define a gfx::kNullNativeView/Window to match other types. Allow some temporary sloppiness to keep the patch size manageable: - Provide a non-explicit constructor that takes a NSView/Window. This will be removed in a follow-on patch (which will mechanically add constructors). - Provide a bool-cast operator to allow if-statements to still compile. These statements can later be changed to comparison with the gfx::kNullNativeView/Window. R=avi, ellyjones TBR=jochen (OWNER of remaining files) Bug: 893719 Change-Id: I39ccac1fd117fe0660fcc21bcb276c8d90dbe550 Reviewed-on: https://chromium-review.googlesource.com/c/1270343 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#600685}
-
Colin Blundell authored
This change is possible now that there is infrastructure for glueing an IdentityTestEnvironment to a Profile. Bug: 896180 Change-Id: I444b847636b4f709c9003ac3047bfc8e848bc049 Reviewed-on: https://chromium-review.googlesource.com/c/1286459 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#600684}
-
Maksim Sisov authored
Wayland does not come with mus. Thus, update the doc respectively. What is more, add a note about minigbm usage with Ozone/Wayland. Change-Id: I66194efccfef75d4496a2f74a7d8c3ec9fe53ee2 Reviewed-on: https://chromium-review.googlesource.com/c/1282066 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Cr-Commit-Position: refs/heads/master@{#600683}
-
chrome://media-internalsHenrik Grunell authored
* Adds capability of showing general audio information on chrome://media-internals in a key-value format. * Shows states of AudioService features (enabled/disabled). * Fixes JS presubmit errors: Use $(id) instead of getElementById(id), and use camelCase for variables. Bug: 895741 Change-Id: I35db3a3ab175586aa7e493ca1acccb1ac7ffa824 Reviewed-on: https://chromium-review.googlesource.com/c/1283020Reviewed-by:
Olga Sharonova <olka@chromium.org> Reviewed-by:
-
Daniel Murphy authored
R=mek@chromium.org Bug: 781870 Change-Id: Ic1b3846812f99520300dbad5e4082921043e673a Reviewed-on: https://chromium-review.googlesource.com/c/1286204 Commit-Queue: Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#600681}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/0f3d2a60105b..48ee86f818fb git log 0f3d2a60105b..48ee86f818fb --date=short --no-merges --format='%ad %ae %s' 2018-10-18 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 c20ead32ad6d..25843dd65181 (5 commits) 2018-10-18 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 54e397425efa..b83d4911baae (1 commits) 2018-10-18 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-10-18 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-10-18 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 847fc0b278f8..54e397425efa (1 commits) 2018-10-18 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-10-18 nigeltao@google.com Have EncodeBench avoid superfluous setup 2018-10-17 nigeltao@google.com Rename EncoderBench.cpp to EncodeBench.cpp 2018-10-17 mtklein@google.com clean up SK_LEGACY_4F_XFERMODES 2018-10-17 fmalita@chromium.org [skottie] Clang-tidy fixes 2018-10-17 halcanary@google.com SkPDF: be smarter about checking to see if a new font is needed. 2018-10-17 jvanverth@google.com Fix NV12 and NV21 images in non-DDL 2018-10-17 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-10-17 kjlubick@google.com [canvaskit] Add perf jobs 2018-10-17 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skia/third_party/skcms c851ac96663d..f3e50cfac5bc (1 commits) 2018-10-17 mtklein@google.com reject non-3x3 gamut matrices 2018-10-17 caryclark@skia.org fix pathops timeouts 2018-10-17 mtklein@google.com disable legacy shaders in flutter Created with: gclient setdep -r src/third_party/skia@48ee86f818fb 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:895763 TBR=egdaniel@chromium.org Change-Id: Ifba7c1e3f72463f9a519d9fb401827ae5aaf08c2 Reviewed-on: https://chromium-review.googlesource.com/c/1288019Reviewed-by:
-
