- 27 Jan, 2020 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/5f86e050c2e8..8fdda11ff772 git log 5f86e050c2e8..8fdda11ff772 --date=short --first-parent --format='%ad %ae %s' 2020-01-27 tvanderlippe@chromium.org Add presubmit check to prevent checking in large files Created with: gclient setdep -r src/third_party/devtools-frontend/src@8fdda11ff772 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/+/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I84c1d276864c3cec2680c78b9b37978f7e2d543d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2021966Reviewed-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@{#735373}
-
David Jean authored
Keep them as already sorted (by locale use). Bug: 1043943 Change-Id: Iac9f5d75ed5dae5d917ab9c97f7442965a37a9b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022706Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: David Jean <djean@chromium.org> Cr-Commit-Position: refs/heads/master@{#735372}
-
Wez authored
The //topaz/packages/sdk:topaz target appears to now be empty, so remove it from the build specification in local-sdk.py Change-Id: I71d02f7f7e2baf61db33bbfb26068deda3115669 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019367Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#735371}
-
David Roger authored
This CL removes a reference to diceEnabled in shouldShowProfile_. Additionally it removes dead code: on desktop this branch is only used when signin is disallowed and the user is signed out. The tests related to the signout button are converted to use the Dice flow. Bug: 891781 Change-Id: I227bdbce52c38c24920433d6b2c8d21a4c6f3bd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015247 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#735370}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/5a0e94681cf8..995992bed045 Created with: gclient setdep -r src-internal@995992bed045 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: None Tbr: jbudorick@google.com Change-Id: Ib1210dd5ab16182614fdd71d19872f9416c4d4bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022613Reviewed-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@{#735369}
-
Ross McIlroy authored
Removes the old 2016 system health stories that have been updated to 2018/9 recordings. Perf Sheriffs: This change will cause movement in metrics which track across the whole story set instead of individual stories. BUG=chromium:878390 Change-Id: I3504263c8f432fe6a26336f45d56775e08cd579a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012954 Commit-Queue: Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#735368}
-
David Roger authored
Bug: 891781 Change-Id: I25eed43ab6bd1b0ead335b9246962932b5b2f4b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014912Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#735367}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/ded3dd5b..f78f1881 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: I059ca6e74659fafe673e0b6d46a9fa2974f713a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2021344Reviewed-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@{#735366}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: I7488cb81c2e655dec6aaf630009030cb82bc3bbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017507Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#735365}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: I1e39b9c33f4b1a99c713f560e10c5d759dd25b5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017329Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#735364}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: Ib956dacb9e2c63120faec9de2b485130903c87bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017522Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#735363}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: I2379c1b78e2088aa448ab50b2156c99f545d8139 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017363Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#735362}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: Ia66d2e3144a30257a5056a5e027948e756bd4da7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017348Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#735361}
-
Colin Blundell authored
As one of the final steps in componentizing SSLErrorHandler for sharing with WebLayer, this CL introduces a //components-level SecurityBlockingPageFactory interface and has ChromeSecurityBlockingPageFactory derive from it. By doing so, SSLErrorHandler no longer needs to know about the embedder-level ChromeSecurityBlockingPageFactory. Bug: 1030692 Change-Id: If95f7d1871ff337f7aa252aa74291dc4fa89d3be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019264 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Carlos IL <carlosil@chromium.org> Cr-Commit-Position: refs/heads/master@{#735360}
-
Eric Seckler authored
Replaces the KeyedServiceFactory::GetServiceForContext trace event with a typed trace event. Corresponding proto buffer definitions for these events were added in aosp/1180211. Typed trace events will allow inclusion of the event arguments in slow reports data in the future. The trace format is protocol-buffer based under the hood already (see docs.perfetto.dev for details), this patch adds strong typing for the arguments of the respective event(s) only. TBR: eseckler@google.com Bug: 644264 Change-Id: I67c0f8f78caca2db76e201f8f649f84eda53e3fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1950966 Commit-Queue: Eric Seckler <eseckler@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#735359}
-
Sylvain Defresne authored
The ChromeBrowserState has been moved from the "ios" namespace to the global namespace by http://crrev.com/c/2014913. Fix the uses from ios::ChromeBrowserState to ChromeBrowserState. Also remove includes of chrome_browser_state_forward.h and use a simple forward declaration instead (since the old name is no longer necessary). This CL was uploaded by git cl split. R=olivierrobin@chromium.org Bug: 1042208 Change-Id: I8681d8f472f14a1dbc22e680b236581e02093c45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017503Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#735358}
-
sczs authored
Bug: 1040844 Change-Id: Id780551c330924f783bad502cad8bfc6195d63b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2018328 Auto-Submit: Sergio Collazos <sczs@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Cr-Commit-Position: refs/heads/master@{#735357}
-
Omer Katz authored
Tracing a Member or a WeakMember first checks whether the member is deleted. If the check passes, the member is traced. The check and the trace both use atomic loads, but not the same load. This can result in time-of-check vs time-of-use data races. This CL merges both loads to a single load to avoid such issues. Furthermore, members in hash tables now might be deleted while the relevant entry is deleted. Since the is deleted check is performed on a copy, member types need to bail out on deleted values (used to be a DCHECK before). Bug: 986235 Change-Id: I4307361af94081b62d159af67cdbde5666378550 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012285 Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#735356}
-
Koji Ishii authored
Following bot results are included. 1140 1146 1151 1158 4 lines were removed and 4 lines were deflaked by consecutive results since 1128. TBR=yosin@chromium.org NOTRY=true Bug: 982194 Change-Id: Ibb13d19075f0edeceed016a972af5e7e46eebf6b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022043Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#735355}
-
Koji Ishii authored
This patch fixes |LayoutInline::InvalidateDisplayItemClients| to support changes to culled inline boxes in rr729656 <crrev.com/c/1774425>. Also moves |NGInlineCursor::CurrentDisplayItemClient| to |NGInlineCursorPosition::GetDisplayItemClient|. Bug: 982194 Change-Id: Icc4ac6c3c27132473d235b9384c46a4f010b6932 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016822Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#735354}
-
Dominik Röttsches authored
The out-of-process font loader, now that it has moved to using CTFontManagerCreateFontDescriptorFromData, does not need to create a full CTFontRef anymore, but can instead wrap the out-of-process font data into a CTFontDescriptorRef built from data. Doing that avoids creating an extra CTFont copy in font_platform_data_mac, as we can merge the data descriptor with the cascade list attributes, and only then create the CTFontRef from it. Bug: 1033478 Change-Id: I025c265caf472578fd77a11acbc4b23a1d8fffb2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2013289 Commit-Queue: Dominik Röttsches <drott@chromium.org> Auto-Submit: Dominik Röttsches <drott@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#735353}
-
Yuki Shiino authored
Bug: 839389 Change-Id: I81e8fe019528dc7c2009c83fed797ef23ad1a15d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022565 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#735352}
-
Rushan Suleymanov authored
Do not send sharing message to sync server after first failure. Bug: 1034930 Change-Id: Ic88c12d646ef8028a3280f017d851ba1759575ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2017410 Commit-Queue: Rushan Suleymanov <rushans@google.com> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
vitaliii <vitaliii@chromium.org> Cr-Commit-Position: refs/heads/master@{#735351}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/fcef6dcb6ab2..589b1875f6b6 git log fcef6dcb6ab2..589b1875f6b6 --date=short --first-parent --format='%ad %ae %s' 2020-01-27 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/vulkan-tools/src 93ab50fc27de..b181c1a7ddb5 (4 commits) Created with: gclient setdep -r src/third_party/angle@589b1875f6b6 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/+/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 Bug: None Tbr: timvp@google.com Change-Id: Idfab494e303fcc1b5a0ace737ffec8df2e488aff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022610Reviewed-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@{#735350}
-
Koji Ishii authored
This function was added to be the same interface as |NGPaintFragment::InlineFragmentsFor|, but as |NGInlineCursor| evoles, it has the same functionality though |MoveTo(const LayoutObject&)| and |MoveToNextForSameLayoutObject()|. Now we prefer |NGInlineCurosr| for all traversals that this patch removes |ItemsFor()| function and switch all usages to |NGInlineCusror|. Bug: 982194 Change-Id: Ia9dfd4f608d8cfc2c5f6ccc5b40b1e1dda0876f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022445Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#735349}
-
Mathias Carlen authored
This patch fixes a NPE by guarding for null tabs. There are situations where a tab is not ready or available and direct action availability is queried. In that case a null tab would crash instead of reporting that this direct action is simply not yet available. Bug: 1041669 Change-Id: Idda1c166405c726cca416898fcd842606e74d3cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016625Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Commit-Queue: Mathias Carlen <mcarlen@chromium.org> Cr-Commit-Position: refs/heads/master@{#735348}
-
Colin Blundell authored
As one of the last stages of componentizing SSLErrorHandler for sharing with WebLayer, this CL starts the process of eliminating SSLErrorHandler's knowledge of ChromeSecurityBlockingPageFactory. To do so, it turns ChromeSecurityBlockingPageFactory's static methods into instance methods and has SSLErrorHandler(DelegateImpl) own an instance of ChromeSecurityBlockingPageFactory. The next CL will introduce a ChromeSecurityBlockingPageFactory interface and have SSLErrorHandler talk in terms of that rather than in terms of the //chrome-level implementation. Bug: 1030692 Change-Id: I42f337c9f3141cd23d46d370e0dfba065e95a6bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019242 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Carlos IL <carlosil@chromium.org> Cr-Commit-Position: refs/heads/master@{#735347}
-
Julian Pastarmov authored
If the data parameter is undefined the API will clear the current stored data and delete the file used to store as well as its immediate folder if it has become empty. BUG=1044673 TEST=unit_tests Change-Id: I2f4dd796c9b0e4eea2474def32a47924d0195ccc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015168Reviewed-by:
Owen Min <zmin@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#735346}
-
Martin Šrámek authored
This reverts commit b283bd25. Reason for revert: Probable culprit for the intermittent compile failures on android_arm64-builder-perf. See crbug.com/1045866. Original change's description: > Android: Use R8 for dex splitting. > > R8 has now added support (b/122902374) for this, fixing a few > outstanding bugs related to class merging. This also makes build rules > a bit simpler. > > Still need to switch Trichrome synchronized proguard over to using > R8 for dexsplitting, then we can remove dexsplitter support entirely. > > This CL updates R8 with another patch, but does not change the R8 > base version used. > > Binary-Size: R8 patch now keeping extra methods. > Bug: 1032609 > Change-Id: I79f88736c20d96b718eb129cdbd093fe1cd297b9 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014309 > Commit-Queue: Eric Stevenson <estevenson@chromium.org> > Reviewed-by: Sam Maier <smaier@chromium.org> > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Cr-Commit-Position: refs/heads/master@{#735315} TBR=agrieve@chromium.org,estevenson@chromium.org,smaier@chromium.org Change-Id: I917a73a43e38a4583d9fcedf43d8a29fffbdf914 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1032609, 1045866 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2020964Reviewed-by:
Martin Šrámek <msramek@chromium.org> Commit-Queue: Martin Šrámek <msramek@chromium.org> Cr-Commit-Position: refs/heads/master@{#735345}
-
Miyoung Shin authored
Move FrameMsg_ScrollRectToVisible to a RemoteFrame and FrameHostMsg_ScrollRectToVisibleInParentFrame to a LocalFrameHost. Introduce web_scroll_into_view_params.mojom for WebScrollIntoViewParams struct. Bug: 1039285 Change-Id: I80e385fa332aa1749da5a842585d9fd28ec08e2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1993076 Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#735344}
-
Mihai Sardarescu authored
Bug: 1028509 Change-Id: I2e0c01f00093e889963ffe96d2972bad10e61bde Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022663Reviewed-by:
Colin Blundell <blundell@chromium.org> Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#735343}
-
Simon Zünd authored
This CL relands one of the tests in https://crrev.com/c/2000023. This test is relanded as-is. Original change's description: > Remove event races in two breakpoint decoration tests > > This CL changes two layout tests to follow the pattern of: > 1. Install an instrumentation promise > 2. Run and await an async action > 3. Await that the instrumentation is called > 4. Dump breakpoint information > > This CL is also in preparation for asyncification of source mapping, > hence making the "action" async. > > Drive-by: Re-baseline one of the layout tests that seems to be fixed > now. > > Bug: chromium:1032016 > Change-Id: Ibad4160fe9ab32aa3f818cdd4937112c3f4e6f08 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2000023 > Commit-Queue: Simon Zünd <szuend@chromium.org> > Reviewed-by: Sigurd Schneider <sigurds@chromium.org> > Cr-Commit-Position: refs/heads/master@{#731171} Bug: chromium:1030216 Change-Id: I71873e000d0b67dfde630f3175968a87c4a49dcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022662Reviewed-by:
Sigurd Schneider <sigurds@chromium.org> Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Auto-Submit: Simon Zünd <szuend@chromium.org> Cr-Commit-Position: refs/heads/master@{#735342}
-
Yuki Shiino authored
Auto-generate bindings/{core,modules}/v8/generated_bindings.gni which defines the list of generated source files. Bug: 839389 Change-Id: Ic4342e392509a613bfc7739c2e0002e627ec528e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022042Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#735341}
-
Hitoshi Yoshida authored
If we want to instantiate a Union classes object from a member type value, we need to use a factory method |FromXxx|. This CL updates make_deafult_value_expr to use it. Bug: 839389 Change-Id: Ie2abbef18a6211c8fe59e6abe6f795966131b4fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2018563Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#735340}
-
Hirokazu Honda authored
Originally GpuVideoAcceleratorFactoriesImpl get HW encoder capabilities by referencing values stored GpuInfo instance. However, the values are not updated after GPU process boots up. Even if a new hw encoder is available after GPU process is initialized, the hw encoder is not detected. This fixes the problem by querying GPU process the hw encoder capabilities when a render process is created. e.g. open the page. Bug: b:147404923 Test: HW encoder is used at appr.tc Test: tast run webrtc.* on krane Change-Id: Ibca82b6e54e9b5d585037c8e63f4c663d085db0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1994849Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#735339}
-
Monica Basta authored
The CL that was causing the failure has been reverted in 8d4d0ff0. Re-enabling the tests. This reverts commit 1ea2a91b. Bug: 1043665 Change-Id: Idd8d15db36bb35cf76b9172baff381ac2f78c8d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019368Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Monica Basta <msalama@chromium.org> Cr-Commit-Position: refs/heads/master@{#735338}
-
Koji Ishii authored
This fixes paint invalidation problem, because this function is used in |LayoutObject::AdjustStyleDifference| to optimize invalidations. Fixes ~45 failures. Bug: 982194 Change-Id: I2c749aed856bdbf8e06b2fb5c1f50b266a68efb9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022444 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#735337}
-
Hirokazu Honda authored
The original code of VaapiVEA expects the dimension of the GMB-based VideoFrame on Encode() to be the same as the one aligned by minigbm. This is wrong; GMB-based VideoFrame has a dimension specified by an application (e.g. webrtc app). In other words, the size is not aligned at all. This fixes the wrongness. See https://tinyurl.com/frame-dimension-in-VEA for the detail. Bug: 982201 Test: Hangout Meet on atlas Test: video.EncodeAccel.* Change-Id: I7b42c9e1b2486cb7b95715185e071f0cefcf7597 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1997426Reviewed-by:
David Staessens <dstaessens@chromium.org> Reviewed-by:
Alexandre Courbot <acourbot@chromium.org> Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#735336}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/05d914745a97..5a0e94681cf8 Created with: gclient setdep -r src-internal@5a0e94681cf8 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: None Tbr: jbudorick@google.com Change-Id: Ia90648bc57068a221102c379ac005efd0f5464dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022608Reviewed-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@{#735335}
-
Alexandre Courbot authored
crrev.com/c/1982499 started passing NativePixmapHandles to the GL import methods instead of raw DMAbuf FDs without layout information. But creating a NativePixmapHandles involves duplicating the FDs of a frame ; something that would fail on Tegra which passed invalid FDs that were to be ignored during import. Fix this by returning a dummy FD created from pipe() instead of an invalid one. The dummy FD is not any more useful, but it can be duplicated safely. BUG=b:148105252 TEST=video_decode_accelerator_tests passing on nyan_big. Change-Id: Ifda5f98d8383f4883494d13708a45783149810f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2016817Reviewed-by:
Miguel Casas <mcasas@chromium.org> Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Commit-Queue: Alexandre Courbot <acourbot@chromium.org> Cr-Commit-Position: refs/heads/master@{#735334}
-