- 13 Mar, 2020 40 commits
-
-
Mustafa Camurcu authored
Extend metrics Arc.PlayAutoInstallRequest.* until 2021-03-11 Bug: b/151314714 Change-Id: Id7734c89b42aa1122762a82f76b49bd80fd3df77 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101543 Commit-Queue: Mustafa Camurcu <camurcu@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#750259}
-
Sebastien Marchand authored
This uses the Clang profiling config during the instrumentation step of a PGO build. This allows getting the profiling data out of the sandboxed processes. This is not an ideal end state, ideally the coverage configuration should be refactored and things common to the two configurations should be moved to a new base config. This requires to use the new version of goma, see go/ma-rbe-dogfood (see crbug.com/990981#c26 for more details). To build a PGInstrumented build (first step to do a PGO build) compile with the following GN args: is_component_build = false is_debug = false chrome_pgo_phase = 1 use_goma = true Bug: 1056189 Change-Id: I717efdcbb6993ad574acce4c0f91eb5fb5bf3972 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2076424 Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#750258}
-
Jon Mann authored
When Wi-Fi sync is first enabled and a list of synced networks has to be merged with the local networks on a device, use the last connected timestamp to decide which configuration should be used. Bug: 966270 Change-Id: Id7688a0fdb9e7922cee8ffec0ef69b804436e3e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093274Reviewed-by:
James Vecore <vecore@google.com> Commit-Queue: Jon Mann <jonmann@chromium.org> Cr-Commit-Position: refs/heads/master@{#750257}
-
Jeff Yoon authored
The iOS tests were being run as gtests, but with the isolated merge script because of iOS' output format (standard JSON output). Moving them to use isolated_scripts instead, as it shortens the definition in test_suites.pyl, and accomplishes the same. Bug: 1055164 Change-Id: Icffb1a1cf72681e2ad08c5a3031d50e842a003f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100714Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Jeff Yoon <jeffyoon@chromium.org> Cr-Commit-Position: refs/heads/master@{#750256}
-
Mustafa Camurcu authored
Extend lifetime of Arc.IconLoadFromFileTime.* metrics until 2021-03-11 Bug: b/151314714 Change-Id: I2252633d23d8747fd6731f2d4eb3ef69709e550e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100636 Commit-Queue: Mustafa Camurcu <camurcu@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#750255}
-
Ahmed Fakhry authored
The first ever default desk created after boot should not trigger any UMA stat reporting. The histograms are deprecated, and replaced by Ash.Desks.NewDesk2, and Ash.Desks.DesksCount2 respectively. BUG=1060709 Change-Id: I09f851009aa05a71e73e914692919d2f8150344e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095973Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#750254}
-
Mustafa Camurcu authored
Extend lifetime of Arc.FirstAppLaunchRequest.TimeDelta metric until 2021-03-11 Bug: b/151314714 Change-Id: I8308e1684468acc8aef37927d40e529ffd05249e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100948 Commit-Queue: Mustafa Camurcu <camurcu@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#750253}
-
Jan Wilken Dörrie authored
This change modifies LeakDetectionRequest to explicitly check for HTTP_TOO_MANY_REQUESTS, indicating the user ran out of quota. It modifies corresponding enums and introduces an optional LeakDetectionError to the LookupSingleLeakCallback, so that the error can be forwarded to consumers like the Password and Safety Check. Bug: 1061108 Change-Id: I86d561b47bbf39e996a425dbd3459eb8bbda52ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100941 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#750252}
-
Lei Zhang authored
Add helper functions to do the conversion and switch to gfx equivalents. Bug: 10611824 Change-Id: Ie4633980d37ba97a92f46b8f465356a4334ac483 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101984 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Daniel Hosseinian <dhoss@chromium.org> Cr-Commit-Position: refs/heads/master@{#750251}
-
Xianzhu Wang authored
Bug: 524134 Change-Id: I99e53d134f2542f06d63519ad2a96b4d24738649 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101494Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#750250}
-
Nate Fischer authored
No change to production logic, only test shell. This adds 'system_webview_shell_package_name' to configure WebView shell's package name. This is handy for Android emulators, because the default package name will cause signature mismatches. I kept the original documentation (for removing the shell from the system image) because it may be relevant for some use cases. But I hope the GN arg is a simpler solution for most developers. Fixed: 1061211 Test: $OUT/bin/system_webview_shell_apk package-info (verify name) Test: $OUT/bin/system_webview_shell_apk install (succeeds on emulator) Test: Upload to gerrit > open file > click "gitiles" > verify markdown Change-Id: I24a35524bc5042da33b8203637ec67a0f610f363 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101859Reviewed-by:
Hazem Ashmawy <hazems@chromium.org> Commit-Queue: Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#750249}
-
Natalie Chouinard authored
This is a follow up to https://crrev.com/i/2756704. Bug: 1047357 Change-Id: I850d2478e978d9dab188f9a2a0913ac30a2083c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101716 Commit-Queue: Gang Wu <gangwu@chromium.org> Auto-Submit: Natalie Chouinard <chouinard@chromium.org> Reviewed-by:
Gang Wu <gangwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#750248}
-
Sean Gilhuly authored
There are variants of viz_unittests that use SkiaRenderer Dawn if it is enabled in the build, and variants of cc_unittests are underway. Add them to the SkiaRenderer Dawn FYI bot, which builds with the gn arg "use_skia_dawn = true". Bug: 1021566 Change-Id: Ia0d96a9389308f45e4a697455f396d69d7d20db4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094758Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Sean Gilhuly <sgilhuly@chromium.org> Cr-Commit-Position: refs/heads/master@{#750247}
-
Charlie Hu authored
This CL migrates UnsizedMedia from feature policy to document policy. Following web tests are being migrated: - unsized-media.tentative.https.sub.html - unsized-media-reporting.html Following web tests are removed, because document policy currently does not have a js API to list all features available. - feature-policy-unsized-media-disabled.html - feature-policy-unsized-media-enabled.html Bug: 993790 Change-Id: Id5e0189c0277c940177460380970eab0015109d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097028 Commit-Queue: Charlie Hu <chenleihu@google.com> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#750246}
-
Robbie McElrath authored
This CL moves the Java-side definition of the kGenericSensorExtraClasses Feature from ChromeFeatureList to a new DeviceFeatureList class that lives in //services/device/public/java. This will allow non-Chrome embedders to query the status of the Feature from Java. Bug: 1058597 Change-Id: Iab3c75fea6c8755de52714be7776547a774842d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095445 Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Reviewed-by:
Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#750245}
-
Oksana Zhuravlova authored
TBR=dmurph@chromium.org Bug: 1061251 No-Try: true Change-Id: I30bbc831be9069b6b246242f3ed4eeb46576e65c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103247 Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#750244}
-
Aaron Leventhal authored
There is a discrepancy between BlinkAXTreeSource::GetChildren() and BlinkAXTreeSource::GetParent(). - GetChildren() does not return direct children when they are ignored and not included in the tree. This is because when AXNodeObject::InsertChild() visits an object that is ignored and not included in tree, it recurses and inserts the next generation. - GetParent() always returns the direct parent. This discrepancy an lead to confusion within AXTreeSerializer, with respect to analyzing reparenting. Bug: 651614 Change-Id: I4e4c8c6a2900cf5516cc98f3b260e619220ea7e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095882 Auto-Submit: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Reviewed-by:
Adam Ettenberger <Adam.Ettenberger@microsoft.com> Reviewed-by:
Jacques Newman <janewman@microsoft.com> Commit-Queue: Mounir Lamouri <mlamouri@chromium.org> Cr-Commit-Position: refs/heads/master@{#750243}
-
Oksana Zhuravlova authored
This reverts commit 823317ab. Reason for revert: unit_tests failing on https://ci.chromium.org/p/chromium/builders/ci/Linux%20Tests%20%28dbg%29%281%29/87477 and https://ci.chromium.org/p/chromium/builders/ci/Win%207%20Tests%20x64%20%281%29/64989 Original change's description: > [Profiles] Record state of profiles on session start > > This CL introduces new metrics about profile representation state that > get recorded on session start: > - Profile.State.Avatar > - Profile.State.LastUsed > - Profile.State.Name > - Profile.State.UnconsentedPrimaryAccountType > all with 6 suffixes that zoom into particular subpopulations of users. > > Bug: 1060113 > Change-Id: Ifdb7dff6638ba29896e33e4f99086c8ca6e21b00 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096697 > Commit-Queue: Jan Krcal <jkrcal@chromium.org> > Reviewed-by: Alex Ilin <alexilin@chromium.org> > Reviewed-by: David Roger <droger@chromium.org> > Auto-Submit: Jan Krcal <jkrcal@chromium.org> > Cr-Commit-Position: refs/heads/master@{#750107} TBR=droger@chromium.org,jkrcal@chromium.org,alexilin@chromium.org Change-Id: I2c6a43c3fd5be4c0eb3d73f66007c6f1b209060a No-Presubmit: true No-Tree-Checks: true No-Try: false Bug: 1060113 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102779 Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#750242}
-
Robbie McElrath authored
This moves ChromeFeatureList.setTestFeatures to a new base.FeatureList class so it can be shared by other *FeatureList implementations. Eventually this class will the the primary method of querying Feature state, but for now it only handles test configuration. See crrev.com/c/2095445 and the bug for more information. Bug: 1060097 Change-Id: Idd0c8ebc447f9f936192f81fc1465600f28ad7d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2099470Reviewed-by:
Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#750241}
-
Matt Menke authored
A number of browser tests use URLLoaderInterceptors solely to look at parameters for requests, and they have similar boilerplate code to set that up. This CL moves URLLoaderMonitor from web_request_apitest to content/public/test/, and switches over a couple browser tests to use it. Bug: 1052148 Change-Id: Id1e9a474ecd77017659f471a54621d0b9c35b0ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2092276Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Shivani Sharma <shivanisha@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#750240}
-
Aaron Leventhal authored
By unifying AddChildren() methods, the code is easier to follow and harder to break. NOTRY=true Bug: None Change-Id: I385a73fa5bb72cb4b733a4f42bd9f35727660ec0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101372 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#750239}
-
Oksana Zhuravlova authored
TBR=jbudorick@chromium.org Bug: 1061249 No-Try: true Change-Id: Ia8392d4bb8c697cea1bd04d7a67cdbbb4f571bc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103254 Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#750238}
-
Garrett Beaty authored
The dummy GPU builders are triggered by non-branched builders that won't have the bucketed_triggers property set. Bug: 1061502 No-Try: true Change-Id: I0054470418bd0736f4e256c5ae49680799362e71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103249 Commit-Queue: John Budorick <jbudorick@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Auto-Submit: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#750237}
-
zhaoyangli authored
Config mb so that this try builder can build with needed gn configs. Bug: 943883 Change-Id: Iad397828705170bf88040922cb4a8d1d74e9dd95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101723Reviewed-by:
Aaron Gable <agable@chromium.org> Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org> Cr-Commit-Position: refs/heads/master@{#750236}
-
Sahir Vellani authored
Put new implementation of ResizeObserver::observe behind ResizeObserverUpdates flag to prevent compat issues until the final version of Resize Observer is shipped. No JavaScript error shold be thrown when unsupported second argument is passed to ResizeObserver::observe(). Issue was introduced in the following CL: http://crrev.com/c/2004051 Bug: 1056535 Change-Id: I7e6d319d14f9d318e0f68ea3e056cfa27eb7f937 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2077280 Commit-Queue: Sahir Vellani <sahir.vellani@microsoft.com> Reviewed-by:
Philip Rogers <pdr@chromium.org> Reviewed-by:
Daniel Libby <dlibby@microsoft.com> Cr-Commit-Position: refs/heads/master@{#750235}
-
yilkal authored
This Cl modifies AppActivityRegistry to correctly handle corner cases where applications are re-installed by doing the following: 1. If the running active time of an app is non-zero restore the application when initializing AppActivityRegistry even if the application is uninstalled. 2. Only remove entries in CleanRegistry if the application is uninstalled, has no unreported activities and its running active time is zero. Bug: 1060340 Change-Id: Idbefe82a686eaa56d31488d305f93c379d7b2bda Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097303 Commit-Queue: Yilkal Abe <yilkal@chromium.org> Reviewed-by:
Aga Wronska <agawronska@chromium.org> Cr-Commit-Position: refs/heads/master@{#750234}
-
Alison Maher authored
Handle fragmentation for fieldset content with a negative margin block start. The following was required to accomplish this: 1. Don't wait to layout fieldset content until there is available space in the current fragment. 2. Break before fieldset content if necessary. Bug: 875235 Change-Id: I0c95fe37694e2a0022ed9222db0a7af13cc958b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2099284 Commit-Queue: Alison Maher <almaher@microsoft.com> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#750233}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/658be97b1940..acd3faab763a git log 658be97b1940..acd3faab763a --date=short --first-parent --format='%ad %ae %s' 2020-03-13 taylori@google.com Merge "ui: Use track name helper for thread tracks" 2020-03-13 hjd@google.com Merge "ui: Fix percentage in thread state label" 2020-03-13 fmayer@google.com Merge "Revert^2 "Assert extra_guardrails fails on user."" Created with: gclient setdep -r src/third_party/perfetto@acd3faab763a If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@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/+/master/autoroll/README.md Bug: None Tbr: perfetto-bugs@google.com Change-Id: Ifca881095f3a2bf1e199d9e30252ffb7bff09106 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103031Reviewed-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@{#750232}
-
Stephen Chenney authored
A previous patch added clamping for sub-pixel borders to ensure they did not round to zero size. The change was done in paint code, leaving the borders as sub-pixel sized in layout. The code prior to that patch clamped the borders during style conversion but did not account for zoom, meaning borders disappeared under zoom. This change removes the clamping in paint and reverts to clamping in the style conversion code, only this time accounting for zoom when considering whether to clamp or not. This will prevent borders growing too large as content is made larger, while preventing sub-pixel borders from disappearing as content is made smaller. Design Doc: https://docs.google.com/document/d/1fAYkOFxp2Luh6OOoXxtwOehmvNRGNss58ibQtVXL0Tw/edit?usp=sharing Bug: 763402, 834489 Change-Id: Iba2ae80b0a981736b93453c1058fd586ba10212c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097514 Commit-Queue: Stephen Chenney <schenney@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#750231}
-
Anastasiia Nikolaienko authored
Screenshot: http://screen/eqiMqP4ssYd Bug: 1043108 Change-Id: I04526e314d7666e1b8d5ba71deed50f7cb0f4ba7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096556Reviewed-by:
Kush Sinha <sinhak@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#750230}
-
Min Qin authored
BUG=1061042 Change-Id: I91c3d60d0f9be69fb8ed82b3ca8f67434b4b84c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100716 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#750229}
-
Fredrik Söderqvist authored
Change-Id: Ie215b673d393b799ed51b669959b19e94b075e95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102728 Auto-Submit: Fredrik Söderquist <fs@opera.com> Commit-Queue: Philip Rogers <pdr@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#750228}
-
Mario Sanchez Prada authored
This CL converts the message that notifies the browser process of changes in the location of accessibility objects in Blink's accessibility tree. Bug: 1010831 Change-Id: I220a005183012b8a509948711b5766f384cced4c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078914 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#750227}
-
Ayu Ishii authored
This change removes the previous iteration of WebOTP, SMS Receiver API, which we ran for Origin Trial. This is now being removed in favor of the credential management version, WebOTP API [1]. WebOTP API Explainer: https://github.com/WICG/WebOTP I2E ending in M80: https://groups.google.com/a/chromium.org/d/topic/blink-dev/-bdqHhCyBwM/discussion Bug: 1045526 Change-Id: I1e73bed5018c814770fa34edcf64daafd234a543 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097041Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Ayu Ishii <ayui@chromium.org> Cr-Commit-Position: refs/heads/master@{#750226}
-
Yilkal Abe authored
This reverts commit 79024af7. Reason for revert: This revert is required because there are scenarios where the profile becomes supervised user after the navigation observer is supposed to be created. i.e. When SupervisedUserNaivgationObserver::MaybeCreateForWebContents is called the user profile is not marked as supervised hence the navigation observer is not created. This resulted in the crashes reported in the bug below. Bug: 1061427 Original change's description: > Only use SupervisedUserNavigationObserver for child accounts. > > This CL makes sure that SupervisedUserNavigationObserver is > created only for supervised user accounts. This saves memory > and processing time. > > Bug: 1043232 > Change-Id: I5d13ce0261d07c29384f5ba5dc83e420ed3138dd > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2008315 > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Dan S <danan@chromium.org> > Commit-Queue: Yilkal Abe <yilkal@chromium.org> > Cr-Commit-Position: refs/heads/master@{#733750} TBR=sky@chromium.org,danan@chromium.org,yilkal@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1043232 Change-Id: I1767afe66b32f34736af2497793d59f598cb61b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102764Reviewed-by:
Yilkal Abe <yilkal@chromium.org> Commit-Queue: Yilkal Abe <yilkal@chromium.org> Cr-Commit-Position: refs/heads/master@{#750225}
-
Danyao Wang authored
The new test is a regression test to make sure the manifest icon download is only subject to the CSP policy of the initiator frame. Bug: 1055360 Change-Id: I0efd8b7532b1d6c6174b95d6dacea0bef8a20767 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097535 Commit-Queue: Danyao Wang <danyao@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#750224}
-
Dave Tapuska authored
In an effort to move compositing layer tree ownership into blink change the way rAF aligned input works. Previously the content layer captured the begin frame dispatched input then let blink know about the main frame. Tell blink now and let blink ask content layer to dispatch rAF aligned input. Eventually all this will go away when blink does everything. Change-Id: I86abe4da3e9005c6973acc8b554c6a87bb9fa485 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101295Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#750223}
-
Xiaocheng Hu authored
FontPreloadManagerTest.OptionalFontFastPreloading is flaky, which might be caused by a race between a timeout firing and a ResourceFinishObserver notification. This patch disables the timeout as a tentative fix. Bug: 1060821 Change-Id: I450b9783c20525a407bc8e3cac19be7b18927b45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101245Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#750222}
-
Pavel Feldman authored
Permission overrides were implemented for the automation purposes (WebDriver, Puppeteer, Playwright, etc). Before this change, protocol required concrete |origin| to be specified in order to override the permissions. This change makes this |origin| input parameter optional and allows global permission override within the given browser context. TBR=mlamouri for updating chrome/ due to changes in content/ semantics. Change-Id: I35e78307643b179a46e296d81f261e5b596c6252 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095668 Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#750221}
-
Sahel Sharify authored
After this cl payment sheets on Desktop and Android will show "Continue" rather than "Pay" when a 3rd party(i.e. non-autofill) payment app is selected. Bug: 1055557 Change-Id: I1717703d50e33df5cdc4a3f0e900b29072df19c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101690 Commit-Queue: Sahel Sharify <sahel@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#750220}
-