- 23 May, 2019 40 commits
-
-
Guido Urdaneta authored
Unified Plan has been default for a while. Fixed the message to reflect that. drive-by: remove redundant expectation file for passing test. Bug: 965989 Change-Id: I1544e6b8016ca459f0ffa6470ac762240f8127a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624909 Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#662559}
-
David Benjamin authored
The last comment on the bug suggests this is no longer applicable. If so, we should remove the suppression. Bug: 347534 Change-Id: I9da64043c95c167a5cab72f09645f6db9a3a50f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1625099 Commit-Queue: Adam Klein <adamk@chromium.org> Auto-Submit: David Benjamin <davidben@chromium.org> Reviewed-by:
Adam Klein <adamk@chromium.org> Cr-Commit-Position: refs/heads/master@{#662558}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d49c2a06..24e3eea4 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Id539e9f2b9ac756f15e189b6de3cb52f718b2039 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626642Reviewed-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@{#662557}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/cae8cde7fa65..8458639991d4 git log cae8cde7fa65..8458639991d4 --date=short --no-merges --format='%ad %ae %s' 2019-05-23 swiftshader.regress@gmail.com Regres: Update test lists @ cae8cde7 Created with: gclient setdep -r src/third_party/swiftshader@8458639991d4 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-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 TBR=swiftshader-team+autoroll@google.com Change-Id: I56877e4f927aecda7ed5226cfad4c7b1df626e5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627292Reviewed-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@{#662556}
-
Arthur Hemery authored
We currently discard any NavigationRequest that matched a DidCommit* IPC but that had a different URL than returning parameters. Instead, we just modify the internal state of NavigationRequest to have the returning URL instead. See the design doc in the bug for discussion about why this is an okay change to make. Bug: 963396 Change-Id: If7f2a158ff1901e5403a3bc88b0bcd7e83f48aba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1554760Reviewed-by:
Camille Lamy <clamy@chromium.org> Commit-Queue: Arthur Hemery <ahemery@chromium.org> Cr-Commit-Position: refs/heads/master@{#662555}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/e85b6a54b641..550868a2b2b4 git log e85b6a54b641..550868a2b2b4 --date=short --no-merges --format='%ad %ae %s' 2019-05-23 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 fb8e1b25ad71..8224a58b4749 (5 commits) 2019-05-23 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ../src aacbdfe8..a410b698 (444 commits) 2019-05-23 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader f4d2a446cc12..cae8cde7fa65 (12 commits) Created with: gclient setdep -r src/third_party/skia@550868a2b2b4 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=csmartdalton@google.com Change-Id: Ic332d69e101382793d80cd79fc9712f259630823 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627289Reviewed-by:
-
Peter E Conn authored
When on the verified origin in a Trusted Web Activity, the page's theme color will be used for the status bar. Previously this was ignored and the toolbar color provided by the client was used. The page's theme color takes precedence over the toolbar color if provided. This results in the following logic for the status bar color in TWAs: - If on the verified origin: - If the page has a theme color, use that. - Otherwise if the client provided a toolbar color, use that. - Otherwise use Chrome's default. - Otherwise, if not on the verified origin: - If the client provided a toolbar color, use that. - Otherwise use Chrome's default. Bug: 964914 Change-Id: Ic41f4cfd1b834fdc9c3444f33dc163de52c9792f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624332 Commit-Queue: Peter Conn <peconn@chromium.org> Reviewed-by:
Pavel Shmakov <pshmakov@chromium.org> Cr-Commit-Position: refs/heads/master@{#662553}
-
Henrik Boström authored
This temporarily disabled the test to unblock webrtc from rolling in to chromium. After the roll, this CL[1] will fix the failure and the test can be re-enabled. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1621912 TBR=guidou@chromium.org NOTRY=True Bug: webrtc:10453 Change-Id: I732b6680e318d55c0700540982a01dbc8ee086a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627030Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Henrik Boström <hbos@chromium.org> Auto-Submit: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#662552}
-
Timothy Loh authored
This reverts commit 9f25eee5. Reason for revert: UnifiedAutoplaySettingBrowserTest.Block_Wildcard failing The UnifiedAutoplaySettingBrowserTest.Block_Wildcard test is failing consistently on linux-chromeos-dbg, e.g. https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/12528. #0 0x7f7f4f902c7f base::debug::CollectStackTrace() #1 0x7f7f4f66af5d base::debug::StackTrace::StackTrace() #2 0x7f7f4f66af18 base::debug::StackTrace::StackTrace() #3 0x55f1ab98d34d content::(anonymous namespace)::DumpStackTraceSignalHandler() #4 0x7f7f0f0a1cb0 <unknown> #5 0x7f7f4921a366 std::__Cr::default_delete<>::operator()() #6 0x7f7f4921a2ef std::__Cr::unique_ptr<>::reset() #7 0x7f7f49219c5d std::__Cr::unique_ptr<>::operator=() #8 0x7f7f49219726 gfx::AnimationContainer::SetAnimationRunner() #9 0x7f7f4a35f1fa views::AnimationDelegateViews::UpdateAnimationRunner() #10 0x7f7f4a35f2f9 views::AnimationDelegateViews::OnViewRemovedFromWidget() #11 0x7f7f4a4d55f0 views::View::PropagateRemoveNotifications() #12 0x7f7f4a4d54d3 views::View::PropagateRemoveNotifications() #13 0x7f7f4a4d54d3 views::View::PropagateRemoveNotifications() #14 0x7f7f4a4d54d3 views::View::PropagateRemoveNotifications() #15 0x7f7f4a4d54d3 views::View::PropagateRemoveNotifications() #16 0x7f7f4a4d54d3 views::View::PropagateRemoveNotifications() #17 0x7f7f4a4d54d3 views::View::PropagateRemoveNotifications() #18 0x7f7f4a4c983b views::View::DoRemoveChildView() #19 0x7f7f4a4c9ad0 views::View::RemoveAllChildViews() #20 0x7f7f4a4edfb3 views::internal::RootView::~RootView() #21 0x55f1af7f9e26 BrowserRootView::~BrowserRootView() #22 0x55f1af7f9f7c BrowserRootView::~BrowserRootView() #23 0x7f7f4a4fb5df std::__Cr::default_delete<>::operator()() #24 0x7f7f4a4f93df std::__Cr::unique_ptr<>::reset() #25 0x7f7f4a4f3045 views::Widget::DestroyRootView() #26 0x7f7f4a4f2db9 views::Widget::~Widget() #27 0x55f1af7f3ba7 BrowserFrame::~BrowserFrame() #28 0x55f1af7f3c6c BrowserFrame::~BrowserFrame() #29 0x7f7f4a534ab5 views::NativeWidgetAura::~NativeWidgetAura() #30 0x55f1af852032 BrowserFrameAsh::~BrowserFrameAsh() #31 0x55f1af85213c BrowserFrameAsh::~BrowserFrameAsh() #32 0x7f7f4a5338b8 views::NativeWidgetAura::OnWindowDestroyed() #33 0x7f7f3c8dd499 aura::Window::~Window() #34 0x7f7f3c8ddbbc aura::Window::~Window() #35 0x7f7f4a531fc2 views::NativeWidgetAura::CloseNow() ... Original change's description: > Use views::AnimationDelegateViews wherever possible > > Replace gfx::AnimationDelegate with views::AnimationDelegateViews. > This will help gfx::AnimationContainer be driven by ui::Compositor. > > TODO: > * files under ash/ > * other files which can't use AnimationDelegateViews. > > Bug: 953585 > Change-Id: I39279bcc779c72bedb26f8c1e43661f0a0a9691d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1616921 > Reviewed-by: Peter Kasting <pkasting@chromium.org> > Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org> > Cr-Commit-Position: refs/heads/master@{#662481} TBR=pkasting@chromium.org,sangwoo108@chromium.org Change-Id: Id63677148acee966377a6ec7a4c4b0ab2becde76 Bug: 953585 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626064 Commit-Queue: Timothy Loh <timloh@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#662551}
-
Jordan Demeulenaere authored
Change-Id: Idc736ea18951da0f52fcc4635c31218a7f1e2212 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624806Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Cr-Commit-Position: refs/heads/master@{#662550}
-
Stephane Zermatten authored
Before this change, a PromptProto.Choice with only element_exists would be rendered as a chip with no names. After this change, we can add PromptProto.Choice again with only element_exists set and no chip definitions set, neither name, nor icon, nor chip type. This is useful to script automatic end of the prompt action. Bug: b/132950869 Change-Id: Ie552934311d8345c4a5496b10d7a63e2bc97c705 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1617520Reviewed-by:
Jordan Demeulenaere <jdemeulenaere@chromium.org> Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Auto-Submit: Stephane Zermatten <szermatt@chromium.org> Cr-Commit-Position: refs/heads/master@{#662549}
-
Yuki Shiino authored
Checks whether script execution is allowed or not in IDL callbacks. See the bug for details. Bug: 964928 Change-Id: Icfe11d7cd029a52794cdd4525d04288735dc3b29 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624985 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#662548}
-
Adam Langley authored
These values get translated into a generic "other" error, so it's useful to see what the underlying code was. Change-Id: I320b41f82efe6a2b2a168d4b1ee87f5a01c2acf1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626731 Auto-Submit: Adam Langley <agl@chromium.org> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#662547}
-
Matt Falkenhagen authored
There can legitimately be invalid WebFeatures on disk when features are removed from the WebFeature enum. Just ignore them when reading. Bug: 965944 Change-Id: I45dbcf0b4e92adf5f648c6cf7ba1e09f6b7cc63a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626917 Commit-Queue: Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#662546}
-
Friedrich Horschig authored
The ViewPager has timing issues that surface if any bottom sheet is invoked very, very quickly after its creation. I witnessed this approximately 1 out of 30 times when testing manually and approximately 1 in 5 times in automated tests. As far as I know, this test flakiness wasn't reported yet and the most affected test has yet to land. This CL posts a SetCurrentItem call right after it's regularly called. That way, if there are no async issues, it will work as it should. But if another SetCurrentItem marks the wrong item active, it will be overridden very soon after. Visually, the difference can (afaict) only be perceived during the very first opening _if_ the wrong sheet would trigger. The flickering is not perceivable unless expected (since the sheet is still opening at the moment when the switch happens). Bug: None Change-Id: I2dcd7ab329746efed6aff294f2dbdb5657357a75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624816Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#662545}
-
Chen Xing authored
Bug: 966001 Change-Id: I7546061c0b7e245720c3706aa7d5e48a510cf059 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1625109 Auto-Submit: Chen Xing <chxg@google.com> Reviewed-by:
-
Jordan Demeulenaere authored
This CL disables the clipping of the bottom sheet content if it is not scrollable. This allows to correctly display the details shadow if it is the only visible element in the scrollable area. We still need to enable the clipping of the content if it is scrollable, otherwise scrolled content will overlap the carousel and header (given how the ScrollView works internally). This is fine as the details are usually shown at the top of the scrollable area, so clipping it when it happens shouldn't have an impact on the details shadow. Given that the actions carousel was relying on the clipChildren flag to be disabled, the spacing before and above the carousel has been moved into the carousel item decorations. * Before: https://screenshot.googleplex.com/p6DUiOBnyDp.png * After: https://screenshot.googleplex.com/AAKU42uvhgv.png Screenshots: Change-Id: I900f47ab41a36cb3f2140d1387671e392d694d42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624194 Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Reviewed-by:
Clemens Arbesser <arbesser@google.com> Reviewed-by:
-
Yi Su authored
This CL moves all certificate related files from //ios/web/net into //ios/web/security. This is a prerequisite work for navigation refactor. Bug: 956511,942244 Change-Id: Iddf802f75b8ca7038e00a062fcb26773d7948a2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624572Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Yi Su <mrsuyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#662542}
-
Timothy Loh authored
This reverts commit b128ac8e. Reason for revert: Some cookie tests failing on linux-chromeos-dbg The following tests are failing consistently when the network service is disabled (aka the non_network_service_browser_tests task)" CookieSettingsTest.AllowCookiesUsingExceptions/1 CookieSettingsTest.AllowCookiesUsingExceptions/3 CookieSettingsTest.BlockCookiesUsingExceptions/1 CookieSettingsTest.BlockCookiesUsingExceptions/3 Failure output: ../../chrome/browser/content_settings/content_settings_browsertest.cc:390: Failure Expected equality of these values: 1u Which is: 1 blocked->GetCookieCount() Which is: 0 Failing build: https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/12530 Original change's description: > RestrictedCookieManager: report activity to ContentBrowserClient > > ...So that things done via async cookie-store API show up in used/blocked > cookies GUI. > > This is done via NetworkContextClient, with some existing reporting via > NetworkServiceClient converted to it, as in case of service worker it's > not associated with process + routing ID pair. > > This now invokes ContentBrowserClient::OnCookie{sRead,Change} on IO, > not UI, thread. > > Bug:958923 > > Change-Id: I7d1832ad30f57c67ea292481444a5e6ca07657df > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1577493 > Reviewed-by: Tom Sepez <tsepez@chromium.org> > Reviewed-by: Matt Falkenhagen <falken@chromium.org> > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Reviewed-by: Martin Šrámek <msramek@chromium.org> > Commit-Queue: Maks Orlovich <morlovich@chromium.org> > Cr-Commit-Position: refs/heads/master@{#662253} TBR=falken@chromium.org,jam@chromium.org,tsepez@chromium.org,msramek@chromium.org,pwnall@chromium.org,mkwst@chromium.org,morlovich@chromium.org Change-Id: I4e587a04252b135a48f5e0d09414f62df65f9278 Bug: 958923 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626207 Commit-Queue: Timothy Loh <timloh@chromium.org> Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium db450183. With Chromium commits locally applied on WPT: c4dc3441 "[WPT] Reland Upload a test for sending mouse events with key pressed" fec827fb "Added wpt tests for overscroll customization." c9077e39 "Restore original tail-processing for ScriptProcessor and AudioWorklet" d814fafc "Check for detached ArrayBuffers in Web Bluetooth writeValue functions" e90c6ef0 "NG/DL: Implement size containment (and display lock) for NG fieldset algo" 8e9cad0d "Cookies WPTs: Add test coverage for SameSite=None" 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 Directory owners for changes in this CL: jochen@chromium.org, estark@chromium.org: external/wpt/referrer-policy NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Ic4b17b54f8d7400bf8d82f65bfd3bb2dee3ae7b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626644Reviewed-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@{#662540}
-
David Benjamin authored
This is probably a false positive. The test currently assumes that, in the period when it's calling SetClearSiteDataHeader, there are no requests in flight. This appears to be true, however, it depends on signals across processes, which I suspect TSan is not taking into account. The race doesn't trigger with --single-process, which supports this theory. Still, adding a lock is easy and seems prudent when state is accessed in this way across threads. Bug: 965719 Change-Id: I7c5c34a06df4684845f76c0e8730c4bfea7170c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626297 Auto-Submit: David Benjamin <davidben@chromium.org> Commit-Queue: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#662539}
-
Rob Schonberger authored
Will increment a histogram on arc install/uninstall, with reason for install or uninstall included. Adds unittest for behavior. Change-Id: I174ed9a2dc5d2c3c74904a1685810ea3c9129c91 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1616825 Commit-Queue: Rob Schonberger <robsc@chromium.org> Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Shao-Chuan Lee <shaochuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#662538}
-
Jan Krcal authored
These histograms were motivated by an investigation for WALLET_DATA and are not needed any more. Bug: 960074 Change-Id: Ic0024664ef58c7ecfbf717102c965821d8ba9219 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1621905Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Nik Bhagat <nikunjb@chromium.org> Auto-Submit: Jan Krcal <jkrcal@chromium.org> Commit-Queue: Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#662537}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/cbb4f6b5b7c6..9d034a4e2c20 Created with: gclient setdep -r src-internal@9d034a4e2c20 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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=lowell@google.com,kmilka@google.com,yutak@google.com Change-Id: Ic8f441f492b28321ebd0cbb5a5c3e4a1a42a1041 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627016Reviewed-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@{#662536}
-
Maksym Onufriienko authored
Navigation test is updated in smoke_egtest to exercise all of them. Bug: 922813 Change-Id: I4d9fe1dec5133310ebc771d7b94240761b0e1003 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1622920 Commit-Queue: Maksym Onufriienko <monufriienko@chromium.org> Reviewed-by:
-
Hitoshi Yoshida authored
This CL does followings - Renames pickle files in bindings code generators from CamelCase to snake_case. - Moves bindings/modules/InterfacesInfoOverall.pickle to bindings/interfaces_info.pickle - Renames InterfacesInfo{Core,Modules}Individual.pickle as interfaces_info_{core,modules}.pickle - Cleans up dependencies in 'idl_impl' template. Bug: 839389 Change-Id: Ib840c323b536a7c3f857c653f73506fa1a4f6793 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626200 Commit-Queue: Kentaro Hara <haraken@chromium.org> Auto-Submit: Hitoshi Yoshida <peria@chromium.org> Reviewed-by:Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#662534}
-
chromium-autoroll authored
The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-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:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I84b8115dd650d21dbc6887d66f17ebe9da7d0b69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627014Reviewed-by:
-
Maciej Pawlowski authored
This one works with strings and other complex types in addition to integrals. Bug: 954080 Change-Id: Ic9961e0ccffdce89d160a5b4d8cab9b7d94a100b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1617481Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Commit-Queue: Maciej Pawlowski <mpawlowski@opera.com> Cr-Commit-Position: refs/heads/master@{#662532}
-
Takashi Toyoshima authored
- Rename CorsOriginAccessMatchMode to CorsDomainMatchMode in order to introduce network::mojom::CorsPortMatchMode. - Replace int32_t port with uint16_t port and CorsPortMatchMode. Bug: 936900 Change-Id: Ic5cbf91ee74366f3380fedc35349edf6c5f09bb6 Tbr: rdevlin.cronin@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626061 Auto-Submit: Takashi Toyoshima <toyoshim@chromium.org> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#662531}
-
Hiroshige Hayashizaki authored
This CL introduces - WorkerGlobalScope::WorkerScriptFetchFinished() as the single point that is always called when worker script fetch is successfully finished. - WorkerGlobalScope::RunWorkerScript() as the unified interface to synchronously evaluate the worker top-level script. For this function, this CL introduces Script::RunScriptOnWorker(), merging code from - WorkerGlobalScope::EvaluateClassicScriptInternal() - WorkerModuleTreeClient::NotifyModuleTreeLoadFinished() - ServiceWorkerModuleTreeClient::NotifyModuleTreeLoadFinished() - Simplifies pausing mechanism by making RunWorkerScript() called when both WorkerScriptFetchFinished() and ReadyToRunClassicScript() are called. (Previously EvaluateClassicScriptInternal() was re-triggering itself when not ready) After this CL, code paths after successful worker script fetch are unified into the single path of WorkerScriptFetchFinished() + ReadyToRunClassicScript() -> RunWorkerScript(). Behavior changes: After this CL, module service worker scripts wait for ServiceWorkerGlobalScope::ReadyToEvaluateScript() before evaluation. Bug: 945673 Change-Id: I20c57d28cf908c4e6492ea56ac4962f3b4674acf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1618628 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by: -
Antonio Gomes authored
... in AnimationWorkletMutatorDispatcher code. Since the code in case is within Blink, there is no need to convert the WTF::CrossThreadOnceFunction to base::OnceCallback (with ConvertToBaseOnceCallback). CL adopts the use of WTF::CrossThreadOnceFunction to simplify the code. BUG=963574 R=haraken@chromium.org, hiroshige@chromium.org Change-Id: I8586dc1397f0861bb42c59dd88bfb7defc534b66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1625969 Auto-Submit: Antonio Gomes <tonikitoo@igalia.com> Commit-Queue: Kentaro Hara <haraken@chromium.org> Reviewed-by:
-
Maksim Sisov authored
This is a prerequisite to unite WaylandShmBufferManager and the WaylandBufferManager. The last requires buffer_id to be passed when managing buffers. Thus, make canvas assign new ids when creating new SkSurfaces and buffers for them. Bug: 963853 Change-Id: I0d980d468a2cc000fd23f0c4abffab2322dcacd3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1617503 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#662528}
-
Yutaka Hirano authored
It is no longer used. Bug: 914739 Change-Id: Ic5d4df6954ea1bee276b0ac41735c4c1bae2fba4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1619523Reviewed-by:
Alexei Filippov <alph@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#662527}
-
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: I2e2ffb7f98d3911547f7f0ffef7d8b0e52db34f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1627291Reviewed-by:
-
Jia authored
[On-device adaptive brightness] Fixes a bug that does not set metrics reporter when model is disabled Bug: 881215 Change-Id: Icb120b47a37c078b3b98dd947369f4240da81d68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626059 Commit-Queue: Jia Meng <jiameng@chromium.org> Reviewed-by:
Michael Martis <martis@chromium.org> Cr-Commit-Position: refs/heads/master@{#662525}
-
tzik authored
ServiceWorkerHost used to use the default task queue to notify its API completion to the user. However, an upcoming change breaks down the global MicrotaskQueue into per-global scope, and disables the microtask checkpoint on the default task queue. That implies that Promises settled in the default task queue will not trigger the promise handler immediately. This CL updates the ServiceWorkerHost setup to use an GlobalScope-bound TaskRunner. Change-Id: Iaea71470917d37d1444b83698315036b1ce2f92a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1617045Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
-
Jan Wilken Dörrie authored
Following r661908 Autofill Mojo types are now available in the browser and on all platforms. However, the provided StructTraits assume the existence of renderer ids in NewPasswordFormGenerationData, which currently don't exist on iOS. This change fixes this. Bug: 962853 Change-Id: I08d664d88d99c4bc76df8b6e0330a3c719e1063c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1624919 Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#662523}
-
Ramin Halavati authored
Accessible window titles in guest mode had an (Incognito) addition. This is changed in this CL to (Guest mode). Bug: 947933 Change-Id: Ica441beb893140218a8448e94313c0d06fe38855 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1611933Reviewed-by:
David Tseng <dtseng@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#662522}
-
Adam Langley authored
Prior to this change, it was possible to request an assertion with an empty allowList and UV=discouraged. If an authenticator had internal UV then this would be passed down as-is. But we don't want resident credentials without UV to be a thing on the Web. Bug: 941120 Change-Id: I8e246cba0f2d687ee14ddcc822430dbe1eff46af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626730 Auto-Submit: Adam Langley <agl@chromium.org> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Reviewed-by:
-
Tsuyoshi Horo authored
This reverts commit 6d9fb7a5. Reason for revert: Fixed test Initial CL: https://crrev.com/c/1616937 Revert CL: https://crrev.com/c/1624588 The original CL was reverted due to the data race bug in test. Change-Id: I7280e271f4739da54b9fcf5e022b3a57256b05ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1625026 Commit-Queue: Tsuyoshi Horo <horo@chromium.org> Reviewed-by:
Lowell Manners <lowell@google.com> Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Reviewed-by:
-
