- 08 Oct, 2020 40 commits
-
-
Olga Sharonova authored
This reverts commit a5354ff6. Reason for revert: multiple crostini tests failing with "Failed to take screenshot by Chrome API: cdp.Runtime: Evaluate: context deadline exceeded.". Examples of first failing builds : https://ci.chromium.org/p/chrome/builders/ci/chromeos-eve-chrome/10585, https://ci.chromium.org/p/chromium/builders/ci/chromeos-kevin-rel/29400, https://ci.chromium.org/p/chrome/builders/ci/chromeos-eve-chrome/10585. Original change's description: > Move the entire modeset operation to Plane Manager Commit > > 1. Set all props for CRTC, Connector and Plane to Plane Manager to Commit. > 2. Maintain a consolidated path to modeset both Atomic and Legacy. > 3. Simplify CRTC Controller by removing all modeset operations and maintain its state from PlaneManager::CrtcState. > 4. This makes all CRTCs on the same hardware display controller to be modeset at the same time. > > BUG=1082882,1015104 > TEST=hardware_display_plane_manager_unittest > > Change-Id: I2401a1b40ce74158416484e6418239d5fb8c1c25 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429586 > Commit-Queue: Mark Yacoub <markyacoub@google.com> > Reviewed-by: Daniel Nicoara <dnicoara@chromium.org> > Cr-Commit-Position: refs/heads/master@{#814933} TBR=dnicoara@chromium.org,markyacoub@google.com Change-Id: If2d243c39d9d4940113b4c46382c1ad6635f9c83 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1082882 Bug: 1015104 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460807Reviewed-by:
Olga Sharonova <olka@chromium.org> Commit-Queue: Olga Sharonova <olka@chromium.org> Cr-Commit-Position: refs/heads/master@{#815054}
-
Harvey Yang authored
This commit adds fake classes: FakeSensorService and FakeSensorDevice in chromeos/components/sensors to add unit tests of sensor clients in Chromium, such as ash/accelerometer and Generic Sensor. BUG=chromium:1006141 TEST=unit tests of ash/accelerometer Change-Id: If4cab4907a3db0e84214ec274ce9dab0a7f2cc44 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2449311 Commit-Queue: Cheng-Hao Yang <chenghaoyang@chromium.org> Reviewed-by:
Ryo Hashimoto <hashimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#815053}
-
Bartek Nowierski authored
There are two functions named GetAllocatedSize, but return different value. Improve comments while at it. Change-Id: I5aef0fbf8f63bf94f184b85d40187363e05b08db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2455947 Commit-Queue: Bartek Nowierski <bartekn@chromium.org> Auto-Submit: Bartek Nowierski <bartekn@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#815052}
-
Eric Willigers authored
The tests in app_service_app_model_builder_unittest.cc no longer need to support bookmark apps, as DesktopPWAsWithoutExtensions has shipped. Bug: 1065748 Change-Id: I0da59a448fa5fab619c10fdb4bd6f761ef8bca15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454554 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#815051}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/36ccb695bd6b..1a87f1f025ef 2020-10-08 syoussefi@chromium.org Add a test for deferred clears and 0-sized scissor clears If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC timvp@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md 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-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: timvp@google.com Change-Id: Ibadd22634eb171c00f99d3896320c602d62eaa17 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460546Reviewed-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@{#815050}
-
Colin Blundell authored
There is currently a check for the subresource filter feature being enabled that gates whether //chrome attaches a ContentSubresourceFilterThrottleManager instance as a tab helper. This CL moves the check to inside ContentSubresourceFilterThrottleManager::CreateForWebContents() itself to (1) ensure that all //chrome-level invocations of this method respect the state of the feature, and (2) ensure that //weblayer also transparently reflects the state of the feature in this flow once the ThrottleManager is brought up there. This CL also adds a relevant //components-level unittest. Bug: 1116095 Change-Id: Iadfe6e40979307746e9573da85b05ea1306dfd7b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2456526Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#815049}
-
Ewann authored
This CL adds ExtendOpenInFilesSupport to the fieldtrial testing config Bug: 1117398 Change-Id: I9d52da7c57e101b1e2607b0c9c6e700e9415468c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454075Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Ewann Pellé <ewannpv@chromium.org> Cr-Commit-Position: refs/heads/master@{#815048}
-
Hayato Ito authored
Add path restriction so that this matches the recent change [1] of the explainer. Subresource path restriction is: > its [path](https://url.spec.whatwg.org/#concept-url-path) contains > the bundle's path as a prefix, Note that the explainer says "the bundle's path as a prefix", however, this CL uses "the bundle's path *without a filename* as a prefix" as subresource path restriction. For example, if a bundle's URL is "https://foo.com/dir/foo.wbn", a subresource's path should contain "dir/", instead of "dir/foo.wbn", as a prefix. We'll update the explainer to match the implementation. - [1] https://github.com/WICG/webpackage/pull/601. Bug: 1082020 Change-Id: I9c2d25108c4169310c395372ac8f7ae74c6a7325 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460366 Commit-Queue: Hayato Ito <hayato@chromium.org> Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#815047}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/306f83589eb0..c267a5583028 2020-10-08 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com Change-Id: I2b23ba5a807f4e259b1ea3e910d28b94f0e2d744 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460585Reviewed-by:
-
Prakhar authored
Bug: 1024915 Change-Id: Ie78693d6b88d6d852144aaac755009954e6ab941 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2423672 Commit-Queue: Prakhar Asthana <pasthana@google.com> Commit-Queue: Erik Staab <estaab@chromium.org> Reviewed-by:
Erik Staab <estaab@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Auto-Submit: Prakhar Asthana <pasthana@google.com> Cr-Commit-Position: refs/heads/master@{#815045}
-
Noel Gordon authored
◑ Install upstream emscripten at version 1.38.33 (git hash 7e841b5) ◑ Pull upstream android.googlesource.com PIEX (git hash 6672dab) ◑ % make clean && make, run tests PASS ◑ Binary size: reduced by 1.3 kByte ◑ Assess performance: unchanged Test: browser_tests --gtest_filter="QuickView*FilesAppBrowserTest*" Bug: 1132695 Change-Id: I52787b1b9ac31a927899672261895037e8cc3bbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460570Reviewed-by:
Alex Danilo <adanilo@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#815044}
-
Gavin Williams authored
Update the tests to expect Save to Drive being a default destination on Chrome OS. Bug: 1112416 Change-Id: I548cb97b1fa07a43d5fd4553786f05a56fb08a45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440986 Commit-Queue: Gavin Williams <gavinwill@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#815043}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/6f4898e52832..cdd72c9222b3 2020-10-08 szuend@chromium.org [ts] Type-check network/NetworkDataGridNode.js with TypeScript If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: chromium:1011811 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I6cc3725f971411fb214deff9a5dfed22587ffe2c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460545Reviewed-by:
-
Christopher Cameron authored
Pass the color space from CoreMedia or CoreVideo to the FrameReceiver, and to the rest of the pipeline. Prefer to use the ColorSpace specified by the CVPixelBuffer over the CMSampleBuffer. Of note is that on macOS, the default capture transfer function is BT709_APPLE (gamma of 1.961) which is different from regular BT709 (sRGB transfer function, in our interpretation). This fixes a bug whereby the colors of a captured IOSurface are different based on if it is being composited by CoreAnimation (in which case it getting the BT709_APPLE via IOSurface metadata) versus viz::GLRenderer (in which case it used to be defaulting to BT709). Bug: 959962 Change-Id: Idb2fdab39e4b4b292932c9bdb2f62dd3f662d4f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459690Reviewed-by:
Markus Handell <handellm@google.com> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#815041}
-
My Nguyen authored
We need to show users "Learn more about suggestion" option in VK as per b/166670314 so adding the api to open the suggestion page in Chrome OS settings app. Drive-by: - Rename IsLanguageSettingsEnabled to IsSettingsEnabled because it only checks that. - Add VirtualKeyboard.OpenLanguageSettings to separate metrics between opening settings from VK and from system tray. Bug: b/166670314 Test: Manual test Change-Id: I25d138fb04e96812db80e5e918b51c69b98aa9c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2455077Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Commit-Queue: My Nguyen <myy@chromium.org> Cr-Commit-Position: refs/heads/master@{#815040}
-
Takuto Ikuta authored
Change-Id: Ieced88baa380ba8952385b67035f50547b364cb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460530Reviewed-by:
Vadim Shtayura <vadimsh@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#815039}
-
Jacobo Aragunde Pérez authored
Bubbles that extend LocationBarBubbleDelegateView have the member function ShowForReason, which may show them in an inactive way when they are not displayed as a result of direct user action. Bubbles displayed in an active way would get focus and ATs would immediately read them, so their role would be a dialog. Likewise, bubbles that show inactive would not get focus and we need them to behave like alert dialogs so ATs can announce them properly. This fixes several bugs: * crbug.com/1119367: the save credit card bubble was not announced because it didn't have the alert dialog role. * crbug.com/1119734: the translate page bubble was not announced because it didn't have the alert dialog role. * crbug.com/1132318: the password bubble was read twice because it was getting focus in addition to sending an alert event. properly announce save credit card bubble, properly announce translate page bubble. Bug: 1119367, 1119734, 1132318 Change-Id: I1c46d65ec2a47a25947f6a75a8935595d974bfa1 AX-relnotes: Do not read password bubble twice when explictly opened, Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2431526Reviewed-by:Allen Bauer <kylixrd@chromium.org> Commit-Queue: Jacobo Aragunde Pérez <jaragunde@igalia.com> Cr-Commit-Position: refs/heads/master@{#815038}
-
Brandon Wylie authored
The dependency was removed in crrev.com/c/2438639, but the exception wasn't removed. Bug: 1123209 Change-Id: I65b1557d8458ffc150d7f36dfb6aa323ada999f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2453505 Commit-Queue: David Trainor <dtrainor@chromium.org> Auto-Submit: Brandon Wylie <wylieb@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#815037}
-
Eric Willigers authored
DesktopPWAsWithoutExtensions has shipped. SystemWebAppManagerTest tests have replaced the corresponding bookmark app tests. Bug: 1065748 Change-Id: Iacdab45883210bd5b74b21eba72e07a8fdfb12e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2455466 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#815036}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1602039546-1494d436e25ae807c20aabdd04cfd168e713ca95.profdata to chrome-win32-master-1602104345-f59e702ee5ca3195d69eeaf0b87216343ca1f336.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC pgo-profile-sheriffs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:win-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: Icc4a2ed3dbac3336cb37abf9ea9782981441b271 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459887Reviewed-by:
-
Eric Willigers authored
DesktopPWAsWithoutExtensions has shipped. The test no longer needs to be parameterized Bug: 1065748 Change-Id: I961a892e59aa651bd81a48cb3467746fc5c60e6f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454921 Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
-
Wei Lee authored
This reverts commit 79e008a0. Reason for revert: The CL is reverted but the failure (SystemWebAppManagerUpgradeBrowserTest.Upgrade/WebApps) does not fix: https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome/9028 So it is very likely that the CL is not the culprit. Original change's description: > Revert "[CCA WebUI] Don't capture navigation other than main page URL for CCA" > > This reverts commit ab12c6e3. > > Reason for revert: Breaks browser_tests: SystemWebAppManagerUpgradeBrowserTest.Upgrade/WebApps > https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome/9011 > > Original change's description: > > [CCA WebUI] Don't capture navigation other than main page URL for CCA > > > > We have the following requirements: > > 1. We prefer loading CCA UI in the app window rather than browser tab > > => Therefore we added "capture_navigation=true" for CCA > > 2. We don't want to load resource URLs or invalid URL in the app window > > => Short term: Landing this CL to filter out unwanted URLs > > > > We decide not to support URL filtering feature for capture navigation > > since only camera app has such requirement at this moment. > > (crbug.com/1132151) > > > > Bug: 980846 > > Test: Type CCA url other than chrome://camera-app/views/main.html and > > the page is loaded in browser tab > > > > Change-Id: I467aa1498aa9fe975d34bdf82adee7f6dff6a14c > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440076 > > Commit-Queue: Wei Lee <wtlee@chromium.org> > > Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org> > > Reviewed-by: Jiewei Qian <qjw@chromium.org> > > Auto-Submit: Wei Lee <wtlee@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#814557} > > TBR=ortuno@chromium.org,wtlee@chromium.org,qjw@chromium.org > > Change-Id: I12e4bbd7ee461e7ed0a3d0664e90ca3e4ea435f9 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 980846 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2457586 > Reviewed-by: Ella Ge <eirage@chromium.org> > Commit-Queue: Ella Ge <eirage@chromium.org> > Cr-Commit-Position: refs/heads/master@{#814741} TBR=ortuno@chromium.org,eirage@chromium.org,wtlee@chromium.org,qjw@chromium.org # Not skipping CQ checks because this is a reland. Bug: 980846 Change-Id: Ia0244f12f8d12c03184b73c27e374072b8c76b09 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460250Reviewed-by:
Wei Lee <wtlee@chromium.org> Commit-Queue: Wei Lee <wtlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#815033}
-
Makoto Shimazu authored
This reverts commit 3c5a2dff. Reason for revert: All/SystemWebAppManagerUpgradeBrowserTest.Upgrade/WebApps consistently fails and this CL touches web_applications and makes substantial changes. Let me speculatively revert this CL. https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome/9011 is the first build that we see the failure. Original change's description: > PWA on Desktop Windows: Implement uninstallation via OS settings. > > When user uninstall a PWA from Windows Settings or Control, it also > uninstall PWA from browser registrar. > > The detailed design spec: > https://docs.google.com/document/d/1XjFYnHXCJt08UsbWrQKw5vLW3_vQumOPrB7A9bpHq_s/edit?usp=sharing > > Bug: 957043 > Change-Id: I927213c847033ddf5767679f3d7081d2c6760343 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316848 > Commit-Queue: Sunggook Chue <sunggch@microsoft.com> > Reviewed-by: Phillis Tang <phillis@chromium.org> > Reviewed-by: Ben Wells <benwells@chromium.org> > Reviewed-by: Alexey Baskakov <loyso@chromium.org> > Reviewed-by: Greg Thompson <grt@chromium.org> > Reviewed-by: Daniel Murphy <dmurph@chromium.org> > Reviewed-by: Tommy Martino <tmartino@chromium.org> > Cr-Commit-Position: refs/heads/master@{#814546} TBR=gab@chromium.org,dmurph@chromium.org,benwells@chromium.org,loyso@chromium.org,tmartino@chromium.org,grt@chromium.org,sunggch@microsoft.com,phillis@chromium.org,johnjan@microsoft.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 957043 Change-Id: I14559730f89d0334bb417be9e4ec128950313fbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459309Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#815032}
-
Hiroki Sato authored
This change exposes the docked magnifier preferences via extension API. Bug: 1125420 Test: browser_test ChromeOSInfoPrivateTest Change-Id: Id38a8a486e8c082e8f5d7973332f1fb8f26673d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2409072Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Hiroki Sato <hirokisato@chromium.org> Cr-Commit-Position: refs/heads/master@{#815031}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/b9eab31a99bf..24d353df49fd If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC shimazu@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:1091475 Tbr: shimazu@google.com Change-Id: I0185555832b10ef296d6cf21e52e205579a5e83f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460346Reviewed-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@{#815030}
-
Eric Willigers authored
DesktopPWAsWithoutExtensions has shipped. The flag no longer needs to be enabled by tests. Bug: 1065748 Change-Id: Ia9889f875fd5680ac56da38cec3e1ebcb46db715 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459448 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#815029}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/041fd0ad7d93..38e6d226f24e 2020-10-08 senorblanco@chromium.org Dawn: refactor texture uploading from GrDawnTexture to GrDawnGpu. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC jvanverth@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md 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 Cq-Do-Not-Cancel-Tryjobs: true Bug: None Tbr: jvanverth@google.com Change-Id: Ie9793972e056ab9b25a2065a5fc2e1cb35981a42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460292Reviewed-by:
-
Eric Willigers authored
DesktopPWAsWithoutExtensions has shipped. The test no longer needs to be parameterized. Bug: 1065748 Change-Id: Id0f546f5add36b3f43e1599088d63063e9619d68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459258 Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
-
Josh Simmons authored
Uses a combination of share-specific values (user hash, URL, source, authentication type, workgroup & username) as input to a SHA256 hash that is then used to create a mount path that does not change across mount / unmount cycles. Test: unit_tests --gtest_filter="*SmbFs*" Test: Deploy to DUT, ensure SMB share mounts successfully Test: Mount share, unmount, remount, ensure mount path remains constant Bug: 1130884 Change-Id: I24603cc9d09609e65c3b869c3c42142c30425788 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2430539 Commit-Queue: Josh Simmons <simmonsjosh@google.com> Reviewed-by:
Anand K Mistry <amistry@chromium.org> Cr-Commit-Position: refs/heads/master@{#815026}
-
David Manouchehri authored
Bug: 1136327 Change-Id: Id47e06bfd088e5c72d5f41c091882e99768fb93d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459447 Commit-Queue: Reilly Grant <reillyg@chromium.org> Auto-Submit: David Manouchehri <david@davidmanouchehri.com> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#815025}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/goldctl-win-chromium-autoroll Please CC bsheedy@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: bsheedy@chromium.org Change-Id: Ia79528eca23aad751f5b69aa1a063d7fe1c37d94 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459903Reviewed-by:
-
Timothy Loh authored
Bug: b:169973809 Change-Id: I2cd308d0805a7f7009f539572a8e0437069fe9d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459547 Commit-Queue: Timothy Loh <timloh@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Auto-Submit: Timothy Loh <timloh@chromium.org> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#815023}
-
Tomasz Tylenda authored
This reverts commit 1864e6e3. Reason for revert: fixing b/169804713 - we use RepeatingCallback in arc_file_system_watcher_service.cc. Original change's description: > Revert "Migrate base::Callback to base::OnceCallback." > > This reverts commit a0282608. > > Reason for revert: b/169804713 > > Original change's description: > > Migrate base::Callback to base::OnceCallback. > > > > BUG=764795 > > TEST=build > > > > Change-Id: I8afa2fd87eb5cf5d9a8f7ecde9fad95d420c5399 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434405 > > Reviewed-by: Hidehiko Abe <hidehiko@chromium.org> > > Commit-Queue: Tomasz Tylenda <ttylenda@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#811980} > > TBR=hidehiko@chromium.org,ttylenda@chromium.org > > Change-Id: I7de1e3dac102f4c760aef90da690016b020baa3f > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 764795 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440088 > Reviewed-by: Hidehiko Abe <hidehiko@chromium.org> > Commit-Queue: Tomasz Tylenda <ttylenda@chromium.org> > Cr-Commit-Position: refs/heads/master@{#812516} TBR=hidehiko@chromium.org,ttylenda@chromium.org,vecore@google.com Bug: 764795 Change-Id: I5cc6f59381be3ced7d07d9629414bdfe0c2edfde Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440233Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: Tomasz Tylenda <ttylenda@chromium.org> Cr-Commit-Position: refs/heads/master@{#815022}
-
Yi Gu authored
This patch adds SmsFetcher::Subscriber::OnFailure(FailureType) to capture SMS parsing failures (this patch) and prompt timeout/cancelled failures (follow up). We also record the reason why an SMS is not parsed. The metrics can help us to quantify the level of unexpected SMSes. UKM collection review: https://docs.google.com/document/d/15orQiLx06QSuZr4ShBaozY0dvPzXiTWwJZ9Xk5cgzAA/edit?usp=sharing Bug: 1126266 Change-Id: I59214e2be64677dae1c9f0b47c2154eefeb1207a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424542 Commit-Queue: Yi Gu <yigu@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Sam Goto <goto@chromium.org> Reviewed-by:
Majid Valipour <majidvp@chromium.org> Cr-Commit-Position: refs/heads/master@{#815021}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/f09b0b11305f..6f4898e52832 2020-10-08 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I303e56281f2fa77829184f34e2d1b5f1517f637e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460288Reviewed-by:
-
Andrew Grieve authored
Change-Id: I6ee9728be16e04f62982258323f00cf91bb35589 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460227 Auto-Submit: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Samuel Huang <huangs@chromium.org> Commit-Queue: Samuel Huang <huangs@chromium.org> Cr-Commit-Position: refs/heads/master@{#815019}
-
Daniel Ng authored
Will followup this CL with actual usage (launching the VM when installed) when CL:2435028 and CL:2454952 land. Tests: Wrote and ran tests. Tested on DUT. Bug: b:168763038 Change-Id: Ifdc10dc269515ba78c54950d455e34133624aa90 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454914Reviewed-by:
Nic Hollingum <hollingum@google.com> Commit-Queue: Daniel Ng <danielng@google.com> Cr-Commit-Position: refs/heads/master@{#815018}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/7b603fae..c942e1b7 Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I0efacbabd250ce3b52c017fff64069d79c9d31ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459088Reviewed-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@{#815017}
-
Takashi Sakamoto authored
For 64bit chrome, we are able to reserve a large virtual address space pools and to allocate/deallocate a part of address space from the pool. However for 32bit chrome, we are not able to reserve such a large virtual address space. It causes out of virtual address space (OOM). So we wrap AllocPages and FreePages inside PartitionAlloc by AddressPoolManager, and update bitmaps by using returned address space (by AllocPages) and freed address space (provided by FreePages' arguments). The bitmaps' sizes depend on the alignment and the number of pages when PartitionAlloc allocates. Regarding the alignment, kSuperPageSize. Regarding NomralBucket, for DirectMapped, multiples of PageAllocationGranularity(), and for NormalBuckets, kSuperPageSize. So bitmap sizes are 4GiB/kSuperPageSize bits and 4GiB/PageAllocatoinGranularity() bits (kSuperPageSize is multiples of PageAllocationGranularity()). Change-Id: I84778fb113df1af2b4ce2a181fc3ae08d9041d65 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2418496 Commit-Queue: Takashi Sakamoto <tasak@google.com> Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Cr-Commit-Position: refs/heads/master@{#815016}
-
Vladislav Kaznacheev authored
Set expiry for Accessibility.CrosShelfNavigationButtonsInTabletModeChanged.* at M91. Fixed: 1088526 Change-Id: Ide9b07ec83596a77d316538aa630e9188cb16d4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2449936Reviewed-by:
Weilun Shi <sweilun@chromium.org> Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org> Cr-Commit-Position: refs/heads/master@{#815015}
-
