- 06 Dec, 2018 40 commits
-
-
Huanzhong Huang authored
Remove part of the description of BrowsingDataCounter's |pref_| that's no longer true and irrelevant. Bug: none Change-Id: Ibfb47522b33b5ca92f9e29191c79b7070d425e38 Reviewed-on: https://chromium-review.googlesource.com/c/1358459Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Commit-Queue: Huanzhong Huang <huanzhong@chromium.org> Cr-Commit-Position: refs/heads/master@{#614324}
-
David Roger authored
This configuration is no longer enabled on any platforms. The code can now be removed. TBR=sky Bug: 777774 Change-Id: Id92ef784e3ae933b21d96f8fd39deba1a4fa67a6 Reviewed-on: https://chromium-review.googlesource.com/c/1355189 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#614323}
-
Alex Clarke authored
This test is currently flaky because it relies upon the ordering of delayed and non delayed tasks to quit a run loop before the very short (0.01s) animation delay is up. This patch deflakes it by temporarily freezing time to prevent the animation from completing before the test expects it to. TBR=wzang@chromium.org Bug: 863341, 891670, 910493 Change-Id: I34cf2824c387c380134eb38b3375e6ab79de4be8 Reviewed-on: https://chromium-review.googlesource.com/c/1365244 Commit-Queue: Alex Clarke <alexclarke@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#614322}
-
Carlos Caballero authored
ThreadManager instances are created on the new thread and thus their thread_manager_ member might not be initialized when GetThreadManagerFor is called. This patch addresses that issue by having two different containers. One to keep track of threads and the other for keeping track of all the ThreadManager instances. This code could use some nice refactoring, this patch just tries to do the minimum changes to get rid of the race. Bug: 906468 Change-Id: Ie39f5a6a18111248c4f83264f6adcdaef4befa23 Reviewed-on: https://chromium-review.googlesource.com/c/1365442 Commit-Queue: Carlos Caballero <carlscab@google.com> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/924d5cbc6627..285509d0e41e git log 924d5cbc6627..285509d0e41e --date=short --no-merges --format='%ad %ae %s' 2018-12-06 dtu@chromium.org [pinpoint] Add cpython to CIPD packages for Swarming tasks. Created with: gclient setdep -r src/third_party/catapult@285509d0e41e 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:911511 TBR=sullivan@chromium.org Change-Id: If17883c4c319c61a837806e819bd0b6c4bdbee28 Reviewed-on: https://chromium-review.googlesource.com/c/1365412Reviewed-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@{#614320}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/c60a77731d68..518add35a8f3 git log c60a77731d68..518add35a8f3 --date=short --no-merges --format='%ad %ae %s' 2018-12-06 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 68c6b982..d6dec397 (614157:614285) 2018-12-05 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 48afe1fc..68c6b982 (614046:614157) Created with: gclient setdep -r src/third_party/webrtc@518add35a8f3 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:None,chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: Ie8e88fd34dcfea01fdf9936668312881d8648d7e Reviewed-on: https://chromium-review.googlesource.com/c/1365490Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/ed9aa80dbd6e..ddbf9b06e56d Created with: gclient setdep -r src-internal@ddbf9b06e56d 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: Ib4c1d177947f216078337b7a96776a364e186e1e Reviewed-on: https://chromium-review.googlesource.com/c/1365491Reviewed-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@{#614318}
-
Dániel Bátyai authored
Arm64 builds for chromeos do not currently use gold when linking. This change enables gold, which results in smaller binary size. Bug: 878565 Change-Id: I70f24490ac010ce58bcc6fd1441ade63288147ad Reviewed-on: https://chromium-review.googlesource.com/c/1341913Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Peter Gal <pgal.u-szeged@partner.samsung.com> Cr-Commit-Position: refs/heads/master@{#614317}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/6d59568196c0..7c4a9b7c7e0e git log 6d59568196c0..7c4a9b7c7e0e --date=short --no-merges --format='%ad %ae %s' 2018-12-06 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update go_deps asset 2018-12-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 96bd8fdf36e5..8f1b7a66a5e3 (3 commits) 2018-12-06 herb@google.com Embed SkPath in SkGlyph::PathData Created with: gclient setdep -r src/third_party/skia@7c4a9b7c7e0e 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-blink-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 TBR=jcgregorio@chromium.org Change-Id: I498a50fb1a2fc65424f50214ad9a19d1a004788c Reviewed-on: https://chromium-review.googlesource.com/c/1365171Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium 8fea47aa. 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=raphael.kubo.da.costa No-Export: true Change-Id: I1f6ad756eb6eafe1416332c08f6f9edd7b030f93 Reviewed-on: https://chromium-review.googlesource.com/c/1365077Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#614315}
-
Robbie Gibson authored
This CL disables dragging text out of the omnibox on iPhone because there is nowhere to drag the text to. Bug: 834778 Change-Id: Ic5c731b8c5400c40e7357c5ff25e5a8e45f3e544 Reviewed-on: https://chromium-review.googlesource.com/c/1363194Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Commit-Queue: Robbie Gibson <rkgibson@google.com> Cr-Commit-Position: refs/heads/master@{#614314}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/c7cc237f9582..924d5cbc6627 git log c7cc237f9582..924d5cbc6627 --date=short --no-merges --format='%ad %ae %s' 2018-12-06 benjhayden@chromium.org Add alerts-section for v2spa. Created with: gclient setdep -r src/third_party/catapult@924d5cbc6627 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 TBR=sullivan@chromium.org Change-Id: I3ce19b927d52a7efb70f97d51f37cdb8bb4a68d0 Reviewed-on: https://chromium-review.googlesource.com/c/1365170Reviewed-by:
-
Victor Costan authored
This CL removes the following patches: * patches/0006-fts3-Fix-uninit-variable-in-fts3EvalDeferredPhrase.patch was superseded by https://www.sqlite.org/src/info/6f4b0d0494446d46 * patches/0009-Fix-ossfuzz.c-to-compile-and-run-with-our-config.patch was superseded by https://www.sqlite.org/src/info/d343f7d6b05865c2 * patches/0010-Backport-Windows-VFS-mmap-fix.patch was a backport of 3.26.0 change https://sqlite.org/src/info/ffce4aac18dacbf2 This CL adds a workaround to our wrapper for SQLite's main header. The workaround is needed by the change at https://www.sqlite.org/src/info/d584a0cb51281594 which fixes a bug that was preventing us from building the SQLite shell on Windows. The most attractive feature in this version is the introduction of the SQLITE_DBCONFIG_DEFENSIVE config flag [1]. The flag will be enabled in a follow-up CL, after the upgrade proves stable. [1] https://www.sqlite.org/c3ref/c_dbconfig_defensive.html Bug: 910906 Change-Id: I046dc5a1419188bb93be24fe0edaf536411d100d Reviewed-on: https://chromium-review.googlesource.com/c/1357358 Commit-Queue: Victor Costan <pwnall@chromium.org> Reviewed-by:
Chris Mumford <cmumford@google.com> Cr-Commit-Position: refs/heads/master@{#614312}
-
Huanzhong Huang authored
To allow checks for |buttonText|. Change-Id: I367876e36e669b56a2dc4cd436665bd600be398e Reviewed-on: https://chromium-review.googlesource.com/c/1356716 Commit-Queue: Huanzhong Huang <huanzhong@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#614311}
-
Hiroki Nakagawa authored
Bug: 911415 Change-Id: If442d03c009fd934dc4f6700b63dc435af381093 Reviewed-on: https://chromium-review.googlesource.com/c/1364993Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#614310}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/e2f948469c66..bdd887fce76c git log e2f948469c66..bdd887fce76c --date=short --no-merges --format='%ad %ae %s' 2018-12-06 kainino@chromium.org Revert "Create new src/tests BUILD.gn file." 2018-12-05 senorblanco@chromium.org Create new src/tests BUILD.gn file. Created with: gclient setdep -r src/third_party/dawn@bdd887fce76c The AutoRoll server is located here: https://autoroll.skia.org/r/dawn-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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=cwallez@chromium.org Change-Id: If083040df65451a828b022bd4e3c95be0c9c3473 Reviewed-on: https://chromium-review.googlesource.com/c/1365174Reviewed-by:
-
Karolina Soltys authored
Follow-up to patch 1353921. Bug: 891670 Change-Id: Ibae12547f3550debeda11cc400b6cb1b6eabfca9 Reviewed-on: https://chromium-review.googlesource.com/c/1363206Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
-
Mirta Dvornicic authored
webrtc::VideoEncoderFactory::CodecInfo properties are being moved to webrtc::VideoEncoder::EncoderInfo, see: https://crbug.com/webrtc/10065. Bug: webrtc:10065 Change-Id: Ib74503c5390f4f3c148a8039005a28870733a974 Reviewed-on: https://chromium-review.googlesource.com/c/1358061 Commit-Queue: Mirta Dvornicic <mirtad@google.com> Reviewed-by:
Lambros Lambrou <lambroslambrou@chromium.org> Reviewed-by:
Florent Castelli <orphis@chromium.org> Cr-Commit-Position: refs/heads/master@{#614307}
-
Hans Wennborg authored
It fails with tip-of-tree Clang because the diagnositics have changed slightly. Disable for now and possible re-enable with updated expectations after the next Clang roll. TBR=thakis Bug: 912021 Change-Id: Ib0d83178ea5c38fa2d19ada720e8526437c916fd Reviewed-on: https://chromium-review.googlesource.com/c/1365434Reviewed-by:
Hans Wennborg <hans@chromium.org> Commit-Queue: Hans Wennborg <hans@chromium.org> Cr-Commit-Position: refs/heads/master@{#614306}
-
Kunihiko Sakamoto authored
This fixes a bug that viewport width was evaluated to zero when evaluating media queries from Link: rel=preload headers, if "width=device-width" was specified in <meta name="viewport">. viewport_description->max_width has type = kDeviceWidth and value = 0 in such case, so device width must be resolved before overriding viewport dimensions. Note: HTMLPreloadScanner is already doing this, in HandleMetaViewport(). Bug: 590188 Change-Id: Iffb0e031e0316bd41aaf52014609d5e7d0b32284 Reviewed-on: https://chromium-review.googlesource.com/c/1360209 Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/baf62bde47d2..ed9aa80dbd6e Created with: gclient setdep -r src-internal@ed9aa80dbd6e 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: Iff2613e7527e25d3f2621fb252220a890fc26b8e Reviewed-on: https://chromium-review.googlesource.com/c/1365411Reviewed-by:
-
Jérôme Lebel authored
This is a reland of 1d497322 This patch also includes implementation for: RLZTrackerDelegateImpl::ShouldUpdateExistingAccessPointRlz() Original change's description: > cros: Fix DCHECK for rlz_lib::UpdateExistingAccessPointRlz > > In addition to fixing the DCHECK, this CL contains a speculative fix > for flaky test (crbug.com/907379). The guess for the flakiness is > (not able to repro the failure locally though): > > |RecordProductEvent| may fail to acquire the |ScopedRlzValueStoreLock| > when recording the CAI event, because the lock is being taken by > |UpdateExistingAccessPointRlz|. In tests, the delay for > |ScheduleDelayedInit| is set to zero, so it's possible for this race > condition to happen. But in production, the delay is non-zero. > > I don't think we're losing test coverage by disabling > |UpdateExistingAccessPointRlz| here because the call is a no-op in > tests, and another test (RlzLibTest.UpdateExistingAccessPointRlz) is > testing its implementation. > > Bug: 906293, 907379 > Change-Id: I208919f22edc09fc1a739a1efe1909a71e5a2730 > Reviewed-on: https://chromium-review.googlesource.com/c/1353990 > Reviewed-by: Roger Tawa <rogerta@chromium.org> > Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#613172} Bug: 906293, 907379, 911439 Change-Id: Ic1d9f85bc3714f890d2dc7fcc1d070d0711baec0 Reviewed-on: https://chromium-review.googlesource.com/c/1360692 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Wenzhao (Colin) Zang <wzang@chromium.org> Reviewed-by:
Roger Tawa <rogerta@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#614303}
-
Kent Tamura authored
mojo_layouttest_test.mojom is renamed to mojo_web_test_helper_test.mojom. It keeps _test suffix indicating it affects only tests. Bug: 843412 Change-Id: Iddfdfcbfd87228fb9ee8abd2777f4838c4b66801 Reviewed-on: https://chromium-review.googlesource.com/c/1362717Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#614302}
-
Philip Jägenstedt authored
This is like the `./wpt run --no-manifest-update` in upstream wpt: https://github.com/web-platform-tests/wpt/blob/81607985fc9485cbf9f3d0a8ae74359f885eb36b/tools/wptrunner/wptrunner/wptcommandline.py#L52-L53 Because updating the manifest is pretty slow, this reduces the time to run a few tests over and over in development when the manifest is not changing. On a Linux workstation, the time to run a single test (dom/historical.html) went from ~14s to ~7s. Bug: 905406 Change-Id: I50e4b8bdd9d3bedbf1e31bc34a2338317a30aaf0 Reviewed-on: https://chromium-review.googlesource.com/c/1361874 Commit-Queue: Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Doug Turner <dougt@chromium.org> Reviewed-by:
Luke Z <lpz@chromium.org> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#614301}
-
Clemens Arbesser authored
Also in this CL: - refactoring of parts of the backend to remove redundant and potentially error-prone code - modified unit tests for web_controller to test for UTF-8 compliance of SetFieldValue() Change-Id: Ib7dd2429159a527ec81f325622e8bf6f97c5e5ab Reviewed-on: https://chromium-review.googlesource.com/c/1360691 Commit-Queue: Clemens Arbesser <arbesser@google.com> Reviewed-by:
Stephane Zermatten <szermatt@chromium.org> Cr-Commit-Position: refs/heads/master@{#614300}
-
Javier Ernesto Flores Robles authored
Only add the loading indicator if the model hasn't been created Bug: 912085 Change-Id: I0d557013eeb63cf882793460a342e22485fa48d5 Reviewed-on: https://chromium-review.googlesource.com/c/1363179Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Javier Ernesto Flores Robles <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#614299}
-
Yutaka Hirano authored
In order to WebDataConsumerHandle deprecation easy, move content::FtpDirectoryListingResponseDelegate to blink. Bug: 910489 Change-Id: I878df1c90c28de1c3f44d9ed48e518207fa831c5 Reviewed-on: https://chromium-review.googlesource.com/c/1358315 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/8f1b7a66a5e3..d300548fcc28 git log 8f1b7a66a5e3..d300548fcc28 --date=short --no-merges --format='%ad %ae %s' 2018-12-06 enga@chromium.org Add ANGLE_multi_draw entrypoints to proc table Created with: gclient setdep -r src/third_party/angle@d300548fcc28 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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: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:890539 TBR=ynovikov@chromium.org Change-Id: Id92afc679acf0f0e00f46fba3d8806da8987f4d1 Reviewed-on: https://chromium-review.googlesource.com/c/1365175Reviewed-by:
-
Kent Tamura authored
The behavior of ListedElement::checkValidity() with kCheckValidityDispatchNoEvent was very simple. This CL introduces IsNotCandidateOrValid() for that behavior, and removes CheckValidityEventBehavior argument of checkValidity(). Change-Id: Ieff28ada30cc47d8617e73306d874cdfa9dee43c Reviewed-on: https://chromium-review.googlesource.com/c/1364990 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by:
Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#614296}
-
Makoto Shimazu authored
Currently NetS13nServiceWorker is enabled by default, and we don't have tests excercising non-S13nServiceWorker path. This flips the test to enable S13nServiceWorker to disabling S13nServiceWorker forcefully. Bug: 715640 Change-Id: I8ba3401989dcd9da46e24cd7dc0500f66a200741 Reviewed-on: https://chromium-review.googlesource.com/c/1365050 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#614295}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/3dc3d8f696a7..baf62bde47d2 Created with: gclient setdep -r src-internal@baf62bde47d2 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: I405f390d1485db257fc26dca9ad21eede7227b0f Reviewed-on: https://chromium-review.googlesource.com/c/1365173Reviewed-by:
-
Reka Norman authored
This CL adds a fake page handler to the app management page which is used when accessing chrome://apps?fakeBackend=true. Bug: 906508 Change-Id: Ib92a66a45b312efd14b5fb038d7cc506944dd716 Reviewed-on: https://chromium-review.googlesource.com/c/1360034 Commit-Queue: calamity <calamity@chromium.org> Reviewed-by:
calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#614293}
-
Mohamed Amir Yosef authored
This CL introduced the ResumableTaskRunner: https://chromium-review.googlesource.com/c/chromium/src/+/1353946 In the class comment where we describe how to make use of the class, one of the lines was not updated with the rest during the last iterations of the review, leaving the example code incorrect. This CL just ensures that we are using consistent variable naming throughout the example. Bug: 909682 Change-Id: I67666306888909e867f624dddb85d003e24604e6 Reviewed-on: https://chromium-review.googlesource.com/c/1363284 Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#614292}
-
Fumitoshi Ukai authored
Bug: 901708, 901706 Change-Id: I7eaaf7434a465be2020d7e81ebf08e8d5ca93ba6 Reviewed-on: https://chromium-review.googlesource.com/c/1360452Reviewed-by:
Jao-ke Chin-Lee <jchinlee@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Shinya Kawanaka <shinyak@chromium.org> Commit-Queue: Fumitoshi Ukai <ukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#614291}
-
Rakina Zata Amni authored
This reverts commit dcfa9938. Reason for revert: Caused flaky tests: crbug.com/912431 Original change's description: > Add console message in MixedContentChecker::HandleCertificateError > > Print console message when subresources are loaded over broken HTTPS. > Screenshot: https://drive.google.com/file/d/1Sn_2c162NmvCoIXFuFVB68z-MhzphcNT/view?usp=sharing > > Bug: 587168 > Change-Id: I997a8da26f0fe015cc0103b96dcbaa7d40525eb8 > Reviewed-on: https://chromium-review.googlesource.com/c/1306779 > Commit-Queue: Livvie Lin <livvielin@chromium.org> > Reviewed-by: Mustafa Emre Acer <meacer@chromium.org> > Reviewed-by: Yutaka Hirano <yhirano@chromium.org> > Reviewed-by: Mike West <mkwst@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Nate Chapin <japhet@chromium.org> > Cr-Commit-Position: refs/heads/master@{#614212} TBR=sky@chromium.org,meacer@chromium.org,yhirano@chromium.org,japhet@chromium.org,mkwst@chromium.org,livvielin@chromium.org Change-Id: I9142c895af7dd1c6a4c9ea7500ee0eca2ce9d118 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 587168, 912431 Reviewed-on: https://chromium-review.googlesource.com/c/1364974Reviewed-by:
Rakina Zata Amni <rakina@chromium.org> Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Cr-Commit-Position: refs/heads/master@{#614290}
-
Mario Sanchez Prada authored
Replace usage of SigninManager's API with calls to APIs from the IdentityManager and PrimaryAccountMutator. Bug: 890813 Change-Id: I1a70a47e41aa132f0d5e4a1de12bd7c5d5eb31ce Reviewed-on: https://chromium-review.googlesource.com/c/1259009Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
-
David Jean authored
Fixed dynamic types for separating labels. Bug: 912030,912028 Change-Id: I1acbfed7fe6569c8af712c45a04b8f86dbbbf0af Reviewed-on: https://chromium-review.googlesource.com/c/1363180 Commit-Queue: David Jean <djean@chromium.org> Reviewed-by:
Javier Ernesto Flores Robles <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#614288}
-
chromium-autoroll authored
This 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: Ib51caf6f12f66fcb6b8d98ce42033e316e3f87dd Reviewed-on: https://chromium-review.googlesource.com/c/1365390Reviewed-by:
-
dpapad authored
This is in preparation of turning on no-var, and prefer-const ESLint checks for all of chrome/browser/resources/ Bug: 792774 Change-Id: Id1bfd8720970876275499854d4fccce1c911d0b2 Reviewed-on: https://chromium-review.googlesource.com/c/1364053Reviewed-by:
Dan Beam <dbeam@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#614286}
-
Yuichiro Hanada authored
It should use WMHelper::HasInstance() to check there is WMHelper instance, but WMHelper::GetInstance() was used instead. Bug: 911857 Change-Id: I46249edd757eff5364863365983be6bef905505c Reviewed-on: https://chromium-review.googlesource.com/c/1365093Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Commit-Queue: Yuichiro Hanada <yhanada@chromium.org> Cr-Commit-Position: refs/heads/master@{#614285}
-
