- 19 Dec, 2019 40 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/5acad7aa14e2..330a8816909a Created with: gclient setdep -r src-internal@330a8816909a 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 jbudorick@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 Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:1029520,chromium:984162 Tbr: jbudorick@google.com Change-Id: I12e810a1d1e6fb72bf726936dc6ee9b9112b7e15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977080Reviewed-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@{#726600}
-
Alexey Baskakov authored
Observe OnWebAppWillBeUninstalled instead of OnWebAppUninstalled. It's too late to obtain the app pointer from the registrar after a web app uninstall (deletion) happened. Bug: 1029221 Change-Id: I7794ed4fa7ed52ac396949eed88bd4514ed2154a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975473Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Commit-Queue: Alexey Baskakov <loyso@chromium.org> Cr-Commit-Position: refs/heads/master@{#726599}
-
David Bokan authored
ScrollBy currently has some logic to check the |current_native_scrolling_element| and use that rather than the CurrentlyScrollingNode. This allows the currently scrolling node to be changed mid-gesture, even though this should never happen. The goal we're aiming for is that every scroll gesture should follow this sequence: ScrollBegin - select the scroller and "latch" it ScrollUpdate - * * * ScrollEnd - release the scroller i.e. "unlatch" it Between begin and end, there should be no modification of the latched scroller. This CL removes the mid-gesture uses of |current_native_scrolling_element|. After this CL, this member of the scroll state is only valid when passed into ScrollBegin where it can be used to specify the scroller that should be scrolled. A future CL will rename this member for clarity. Bug: 1016229 Change-Id: I7a8b24795990623dd27c471f05d06481ff05e147 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1953331 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Majid Valipour <majidvp@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#726598}
-
behdad authored
The experimental percentage of representative perf tests increased from 25% to 100% Bug: chromium:1000837 Change-Id: I9d86c3358bcfe8a05d29896453dd0e9154c2f45c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976191 Commit-Queue: Behdad Bakhshinategh <behdadb@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#726597}
-
Yuke Liao authored
Due to crbug.com/1034002, linux-code-coverage fails frequently with an infra failure at the step of collecting results for weblayer_browsertests, so this CL removes the offending target for now. Bug: 1016929, 1034002 Change-Id: I41fd82097ff55bf3a84ec9d139e1a3f22894fc3a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977021Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#726596}
-
Clark DuVall authored
Switches all test to ContentJUnit4ClassRunner since it already has the @CommandLine.Flags parsing bits set up. This also removes a sleep that was added because we couldn't pass command line args before. Change-Id: Ic7e63c31e24d540af0bbf1dda58e800319a1d844 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975075 Commit-Queue: Clark DuVall <cduvall@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#726595}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/e6d31bd0894c..6e6b4dd7d3e2 git log e6d31bd0894c..6e6b4dd7d3e2 --date=short --first-parent --format='%ad %ae %s' 2019-12-19 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-12-19 jvanverth@google.com Revert "Generate small path SDFs directly from path." 2019-12-19 jvanverth@google.com Generate small path SDFs directly from path. 2019-12-19 bsalomon@google.com Revert "Revert "Blend functions as SkSL built ins."" 2019-12-19 reed@google.com Revert "temporarily expose postIDiv" 2019-12-19 jvanverth@google.com Add wireframe support to Metal. Created with: gclient setdep -r src/third_party/skia@6e6b4dd7d3e2 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 borenet@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 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 Bug: None Tbr: borenet@google.com Change-Id: I1ab721b51199c34fb8fd762c1ea77ec936ef729d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976881Reviewed-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@{#726594}
-
Aga Wronska authored
Detailed app activity information are needed by Per-App Time Limits feature. Bug: 1015656 Change-Id: Ibb0a59067a9d9d01adc5b02a1e6fc59ca8fec681 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1970855Reviewed-by:
Henrique Grandinetti <hgrandinetti@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Aga Wronska <agawronska@chromium.org> Cr-Commit-Position: refs/heads/master@{#726593}
-
Lu Huang authored
The pwa install icon should not be visible when the user is adding input to the omnibox. http://crrev.com/c/1933852 made this the default behavior for all page action icons. This change adds a specific test for the PwaInstallView icon view. Bug: 969902 Change-Id: I6cd4fa495f70fefaf228771a2d8fd706e88f88e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975155Reviewed-by:
Alan Cutter <alancutter@chromium.org> Reviewed-by:
Lorne Mitchell <lomitch@microsoft.com> Commit-Queue: Lorne Mitchell <lomitch@microsoft.com> Cr-Commit-Position: refs/heads/master@{#726592}
-
Mason Freed authored
In the (corner) case that the <iframe> src attribute is relative to a data URL, the previous behavior was to silently fail. With this CL, there is now a console message indicating what happened. Example: <iframe id="i"></iframe> <script> let html = `<iframe src="resource.html"></iframe>`; i.src = `data:text/html,${encodeURIComponent(html)}` </script> Bug: 1033313 Change-Id: Ie164edaacd2d92ea2bebc89419782e9204cdf410 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972493 Commit-Queue: Mason Freed <masonfreed@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by:Kent Tamura <tkent@chromium.org> Auto-Submit: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#726591}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/5217ad51c521..d28b578b6b09 git log 5217ad51c521..d28b578b6b09 --date=short --first-parent --format='%ad %ae %s' 2019-12-19 yunchao.he@intel.com Disallow storage buffer binding in vertex shader 2019-12-19 enga@chromium.org fuzzing: Add Vulkan backend fuzzer Created with: gclient setdep -r src/third_party/dawn@d28b578b6b09 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC cwallez@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 Cq-Include-Trybots: luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: None Tbr: cwallez@google.com Change-Id: I2d4e14e934b49523773b2d9ac8dd85c27eb6e365 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977119Reviewed-by:
-
Clark DuVall authored
This will allow testing WebView/WebLayer behavior when using a bundle, and will help investigate any issues that can happen. Bug: 1033098 Change-Id: Id88c128d5dd555ccb188a172d52dcb49ee373af9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1962899Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#726589}
-
Brian Norris authored
<meme>Why do we even have that lever?</meme> (Maybe just a holdover of the Flimflam API?) Shill never has (and most likely never will do so) managed Bluetooth devices. I plan to drop this (unimplemented) definition from system_api and stop documenting it in the Shill docs (https://crrev.com/c/1976962), so Chrome should stop using it. Bug: none Change-Id: I6c868fb05add97da800e39e7c32add5b8e9a1c14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977140Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Brian Norris <briannorris@chromium.org> Cr-Commit-Position: refs/heads/master@{#726588}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=ericorth@chromium.org Bug: 1023677 Change-Id: Ic527ff1a7cb4654e17df53ba36a3da314187de17 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957495 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Eric Orth <ericorth@chromium.org> Commit-Queue: Eric Orth <ericorth@chromium.org> Cr-Commit-Position: refs/heads/master@{#726587}
-
Kramer Ge authored
OS API creates vulkan swapchain images as render targets. Estimate the allocated GPU resource with vkGetImageMemoryRequirements. Bug: 899905 Change-Id: I162ab641c3e13a10fa29e8e8860dee06b2598a79 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1967552Reviewed-by:
Peng Huang <penghuang@chromium.org> Commit-Queue: Kramer Ge <fangzhoug@chromium.org> Cr-Commit-Position: refs/heads/master@{#726586}
-
Aran Gilman authored
ExecuteJsScript() is used only once in this CL but will be used in new tests added by https://crrev.com/c/1922788. This CL also runs clang-format on pinch_tester.js and modifies it to return a bool instead of a dict. The latter simplifies verification of script execution in the C++ portion of the test. Bug: 1016615 Change-Id: I2c847eb53556a30e991d988fb676c6a3fa5aee8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1971702 Commit-Queue: Aran Gilman <gilmanmh@google.com> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#726585}
-
Michael Thiessen authored
Split off of https://chromium-review.googlesource.com/c/chromium/src/+/1901591 The motivation here is to make get rid of the memcpy and leaked ptr in DoAddSchemeWithType to make the code more readable. Also DCHECKs that schemes aren't locked when calling EnableNonStandardSchemesForAndroidWebView. Bug: 783819 Change-Id: I959d364fd0572a3c09c68ab75e48f46809f8dbc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974759 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#726584}
-
Ahmed Mehfooz authored
Bug: 1021133 Change-Id: Ic7cbf25c094006efd970c75dff2b733b31620bc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974599 Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Reviewed-by:
Tim Song <tengs@chromium.org> Cr-Commit-Position: refs/heads/master@{#726583}
-
Ella Ge authored
To support replay input events (crrev.com/c/1944333). This CL adds input event's detail information (coordinates, modifiers, button to trace data, when "devtools.timeline.inputs" category is enabled. Change-Id: I73cad86a9922e0a7b494ac0a01f8817e53cf7e26 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1943179Reviewed-by:
Mathias Bynens <mathias@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Ella Ge <eirage@chromium.org> Cr-Commit-Position: refs/heads/master@{#726582}
-
Aaron Leventhal authored
Make sure even if painting returns early, that accessibility focus is still updated. Bug: 820351 Change-Id: I4d712742b9677095cdcf19d486945c892b73a11c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976071 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Reviewed-by:
-
Oksana Zhuravlova authored
This change addresses the crash in TextSuggestionHostMojoImplAndroid::StartSuggestionMenuTimer() if text_suggestion_host_ is null. TextSuggestionHostMojoImplAndroid is now not created if a TextSuggestionHost has not been set for the corresponding RenderWidgetHostViewAndroid. Bug: 1034125 Change-Id: Ia21fba5e6d486637715743d33e65d2fcccc86c43 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976829Reviewed-by:
Tom Sepez <tsepez@chromium.org> Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#726580}
-
Ben Pastene authored
All uses of Simple Chrome currently already do this: https://cs.chromium.org/chromium/src/third_party/chromite/cli/cros/cros_chrome_sdk.py?rcl=8b42807c6f1e5b2a046695f363227138f56db6cb&l=999 While the ebuild explicitly overwrites the host toolchain with the host toolchain in the chroot: https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/aeeb6096850b8fe779d87b83a5498160e25a3ffa/chromeos-base/chromeos-chrome/chromeos-chrome-9999.ebuild#907 So this should have no effect on ebuilds of chrome, while letting us delete parts of simeplchrome's chromite code. Bug: 937821 Change-Id: Ic589c948407c6dd34bd67e95520cc05ed84740b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976284Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
George Burgess <gbiv@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#726579}
-
Nate Fischer authored
This adds an OWNERS file for the aw/nonembedded folder. I'm nominating myself and torne@ for initial owners since we've written/reviewed a number of CLs in the folder recently, but this still inherits the top-level AW owners. Test: N/A Change-Id: Ic4591f59e1a4e81088ea0de69136e1dcc39d4b10 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976882Reviewed-by:
-
Sonny Sasaka authored
Bug: b:145163508 Change-Id: I93f832f958a13fd88c13fe2a74bdea66a69e584d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976497Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Sonny Sasaka <sonnysasaka@chromium.org> Cr-Commit-Position: refs/heads/master@{#726577}
-
nancy authored
BUG=1011235 Change-Id: Icb262a58b4464d7963acb0b22a9d30ce7b070060 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972738 Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#726576}
-
Devlin Cronin authored
There was a class, ConsoleWebContentsDelegate, in chrome/browser/ssl/security_state_tab_helper_browsertest.cc. However, it's never used, and is obsolete with content::WebContentsConsoleObserver. Remove it. Bug: None Change-Id: I1178fe62927b4d6bb5e8d8072a65814f3aaca525 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974321Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#726575}
-
Peter Kotwicz authored
This CL enables preconnecting and DNS prefetching for webapps BUG=1035752 Change-Id: If9856ecd6febc630d84da870a95ce2810458f3ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976560 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#726574}
-
John Budorick authored
instead of pool:Chrome-CrOS-VM. Bug: 1011474 Change-Id: I27b1be5840cc4f017eeb96cfc7560a87aab14d74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976553Reviewed-by:
Ben Pastene <bpastene@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#726573}
-
Kelvin Jiang authored
This CL leverages crrev.com/c/1923611 by using the navigation ID within WebRequestInfo to correlate a main-frame navigation request originating from the previously active WebContents to the new WebContents loaded from that request. This is accomplished by adding the actions matched with a request to a pending set of actions for a given navigation ID if the request is associated main-frame navigation. Once the navigation completes, the actions matched for the navigation are added to the badge text and cleared from the pending set. Bug: 992251 Change-Id: Iac7b3502a3598c9d06ab3261ea33f30d2cb20ee7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1956110 Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#726572}
-
Ian Kilpatrick authored
IsLogicalHeightTreatAsAuto -> IsLogicalHeightTreatedAsAuto There should be no behaviour change. Change-Id: I074dba0f468981f901869fa1bb96c098157c7ba3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1977059Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Aleks Totic <atotic@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#726571}
-
Ian Kilpatrick authored
There should be no behaviour change. Change-Id: Ic9b22f18358779f9808c7e2aa2f37422e9696936 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976939Reviewed-by:
-
Manuel Rego Casasnovas authored
There was a Mozilla test failing in Chromium because of "will-change: contain" is not creating a stacking context. This has been clarified in the following CSSWG issue: https://github.com/w3c/csswg-drafts/issues/4612 The fix is just adding the "contain" property to HasPropertyThatCreatesStackingContext() in computed_style.cc. BUG=881057 TEST=css/vendor-imports/mozilla/mozilla-central-reftests/contain/contain-paint-stacking-context-001b.html Change-Id: I52a50cfac7bc46a74557760006dc46abba0632ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975913Reviewed-by:
Oriol Brufau <obrufau@igalia.com> Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#726569}
-
Natalie Chouinard authored
Replace usages of java_files with sources in all //chrome/android BUILD.gn files, and automatically format. Also include "sources" in the lists of forwarded variables alongside "java_files" where necessary. Bug: 1035074 Change-Id: I5b570a3a0115993b2ba517362b47a31f64edcc1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1973243Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Commit-Queue: Natalie Chouinard <chouinard@chromium.org> Cr-Commit-Position: refs/heads/master@{#726568}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=jsbell@chromium.org Bug: 1023677 Change-Id: Id1edd2e303e6227cfd80aa983a05bf15134e457e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1957526 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Joshua Bell <jsbell@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#726567}
-
Eric Karl authored
Update the SharedImageBackingFactoryAHB to add lock in the destructor as well as in ProduceLegacyMailbox(). TBR: vikassoni@chromium.org Bug: 1034007, 1003686 Change-Id: I4f6109cdc05f4287ddd45e3e85ec2c03ee15edeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974855 Commit-Queue: Eric Karl <ericrk@chromium.org> Commit-Queue: vikas soni <vikassoni@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#726566}
-
Jorge Lucangeli Obes authored
Metrics were added in https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1958504. Enums were added in https://chromium-review.googlesource.com/c/chromium/src/+/1976310. BUG=985492 TEST=validate_format.py, pretty_print.py. Change-Id: I9ed97d931fbe7877b7d1deb33a5426e9ef8c4011 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976436 Commit-Queue: Jorge Lucangeli Obes <jorgelo@chromium.org> Reviewed-by:
Christopher Thompson <cthomp@chromium.org> Cr-Commit-Position: refs/heads/master@{#726565}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 4bce4c14. With Chromium commits locally applied on WPT: c15c1083 "Add test that datachannels close when connection does." 9899425d "[css-layout-api] IntrinsicSizes Implementation" d75fede5 "[EventTiming] Fix crossiframe test" 93b03639 "Roll WebGPU CTS" 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=robertma No-Export: true Change-Id: Ia745a7edc3344d0492ba8901bbe7a8924bbcafcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976844Reviewed-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@{#726564}
-
Torne (Richard Coles) authored
When we're on O+ and loading from the WebView implementation, we can use the WebView's declared sandboxed service processes instead of the client's, which will cause them to be forked from the WebView zygote, saving some startup time and memory. Fixed: 1025602 Change-Id: I902cc7bd5685ebddfa4681d7d87bb45fc8330dbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974840 Commit-Queue: Richard Coles <torne@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Auto-Submit: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#726563}
-
Kent Tamura authored
... if attribute's owner element is an HTML element and its owner document is an HTML document and it has no namespaceURI. The new behavior matches to Firefox. Bug: 179453 Change-Id: I499f809c8ad2620da2ef2c78fe6d301feec6f67b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975494Reviewed-by:
-
Bruce Dawson authored
The feature flag that I added to control whether the Windows timer interval on DC power should be 4 ms or 8 ms had a race condition in that we might query the feature flag before the feature-flag system was configured. This change fixes that by adding the ReadMinTimerIntervalLowResMs function which is called after the feature system is initialized. While working on this I realized that ActivateHighResolutionTimer and EnableHighResolutionTimer were extremely subtle so I simplified them. When calling timeBeginPeriod you *must* eventually call timeEndPeriod to undo your request, and you must call it with exactly the same value. With the raised timer frequency now having three possible values (1, 4, or 8 ms) the logic was getting far too messy so I now do the simpler task of just recording the value passed to timeBeginPeriod and using it in timeEndPeriod. This is more obviously correct. The new rule is that timeBeginPeriod and timeEndPeriod should always be called with g_last_interval_requested as the parameter. I also tidied up and expanded on some comments - I always get confused by the purpose and semantics of ActivateHighResolutionTimer and EnableHighResolutionTimer. I tested with --disable-highres-timer, with and without --enable-features=SlowDCTimerInterruptsWin, using trace_timer_intervals.bat to confirm that all Chrome processes were lower the timer interrupt setting to 4 ms or 8 ms as requested. Bug: 1033108, 927165 Change-Id: I69b9253b456d7e3273ea1823a61c20926b0a4510 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1965846Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Cr-Commit-Position: refs/heads/master@{#726561}
-
