- 31 Jul, 2020 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/37710e42bb90..828c1805a64b 2020-07-31 t-olfly@microsoft.com Updating CDP and dependencies If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I78f9866c47c15d30b22d619073d71a96d4695194 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332851Reviewed-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@{#793796}
-
Ilia Samsonov authored
Baselines were generated in the following build: https://ci.chromium.org/p/chromium/builders/try/linux-blink-optional-highdpi-rel/5 Bug: 1068400 Change-Id: I8dc2315c348c3c47561947e6aaf30f1f44fd533f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2292872Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#793795}
-
Ilia Samsonov authored
Baselines were generated in the following build: https://ci.chromium.org/p/chromium/builders/try/linux-blink-optional-highdpi-rel/5 Bug: 1068400 Change-Id: I598970475e76cb2d56825254008ad98ba5c532e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2293158Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#793794}
-
Toni Barzic authored
Update shelf navigation widget metrics reporter to use the hotseat state from the time of reporter callback creation to select which hotseat state dependant metric to report. Note that shelf component layout happens after the hotseat state is set, so the hotseat state should have the desired value by the time navigation widget transitions start. The report callback is run asynchronously from the time the animation completes, so the hotseat state can change between the hotseat animation changing and the reporter callback running (e.g. if an app window is closed just after the animation to hidden hotseat ends, which may happen in perf tests). With the existing logic, where the reporter uses the current hotseat state to select the histogram to report, the animation may get registered for an incorrect transition. Also, when animating home button bounds, do not request animation, and create metrics reporter if the view is already animating to the same target bounds. BUG=1108641 TEST=ran ui.HotseatAnimation tast tests a couple of times Change-Id: Idde82ce98545d7ee7e54170abea588c9793e6d3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333141Reviewed-by:
Andrew Xu <andrewxu@chromium.org> Commit-Queue: Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#793793}
-
Ilia Samsonov authored
Baselines were generated in the following build: https://ci.chromium.org/p/chromium/builders/try/linux-blink-optional-highdpi-rel/5 Bug: 1068400 Change-Id: I681631cee37468ab20e89d6bbefcdd82dea149b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2293005Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Ilia Samsonov <isamsonov@google.com> Cr-Commit-Position: refs/heads/master@{#793792}
-
Jeffrey Cohen authored
Bug: 1024586 Change-Id: I586d8dc0a05c0aab3776b42c9145d0f6a586c1f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2032505 Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#793791}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/56387075825a..9aa97c0bd39e 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 pkasting@google.com,nicolaso@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:1111845 Tbr: pkasting@google.com,nicolaso@google.com Change-Id: Ia1c7ae443e5f430ef1b0d4173e58209ac4b23b12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333229Reviewed-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@{#793790}
-
Vasiliy Telezhnikov authored
In case of ImageReader the UpdateTexImage can fail even if RenderToTextureOwnerFrontBuffer succeeded. This can lead to situation when we don't have current frame to get size from. This CL handles this situation to fallback to visible size until we will be able to get right size. Bug: 1109016 Change-Id: I2fc6381cccc4560e97355ac465f62a72734facef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2328013Reviewed-by:
Frank Liberato <liberato@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#793789}
-
Weilun Shi authored
For "exact" linear histograms, the bucket index for Sample |value| can be easily computed and thus we don't need to do a costly binary search for the bucket index. This could have a notable performance improvement given that some histograms (like TaskCount metrics, done after every queued task) can see huge numbers of lookups. the code is already covered by the HistogramFunctionsTest.ExactLinear test. Without this change, running 1 million times the logging part of the above test takes 640ms. With this change, it only takes 530ms which improves ~20% of the bucket lookup performance. Bug: 950536 Change-Id: I1fa9a1cfd0ccbf1967a678ba82b19f15f90c9848 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2328305Reviewed-by:
Brian White <bcwhite@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Auto-Submit: Weilun Shi <sweilun@chromium.org> Cr-Commit-Position: refs/heads/master@{#793788}
-
Julian Pastarmov authored
BUG=1104999 Change-Id: I4e97a51c6858506adc3a2f857fdae96d58af504e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332716 Auto-Submit: Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#793787}
-
Peter Kasting authored
* With this turned on, gtest unconditionally asks absl to initialize the symbol handler. This causes base's attempt to do the same thing to fail. Make base handle this by trying a force-reinit if this failure occurs. * Allow gtest to see the full absl sources, since it unconditionally #includes banned headers. * Fix include dirs not being set correctly, leading to compile failures. * Narrow gtest absl dependency from public_deps to deps, since we don't want "all targets depending on gtest" to silently be able to use absl. * Leave this flag disabled by default for NaCl, since the toolchain there can't compile some of the absl code. Bug: none Change-Id: I24930d1c2086bec1522922665708abee5ce8fc08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332038 Commit-Queue: Peter Kasting <pkasting@chromium.org> Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Bruce Dawson <brucedawson@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@google.com> Cr-Commit-Position: refs/heads/master@{#793786}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1596200193-009c7ebe0bb7420a825c0bad7e0cdc3d24f6a3ae.profdata to chrome-win32-master-1596214766-ef32b2ce97242365d96c5d969f5553eac2e23ec9.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:win-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: Ia16290fb18cfbabcd853f18ca14b5ae9b8179679 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333102Reviewed-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@{#793785}
-
Boris Sazonov authored
1. Moves the code that reconciles ChromeSigninController state with SigninManager state from GoogleServicesManager to SigninManager itself. 2. Removes now empty GoogleServicesManager. 3. Removes GoogleServicesManagerIntegrationTest (it contained no tests, so there's no behavior change). Bug: 1107942 Change-Id: I492f2e12ec1a8f6378d424ba54e0c022284323bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2312697Reviewed-by:
Alice Wang <aliceywang@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#793784}
-
Chris Harrelson authored
This is in preparation for separating them into different Blink lifecycle steps. Bug: 1100711 Change-Id: I8c76a57aa127795910a0db5f36fad2fc1ac60162 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2324205 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#793783}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/39c302bccf3a..fce06b20005c 2020-07-31 fmayer@google.com Merge "Add field for object size of java type." 2020-07-31 dproy@google.com Merge "ui: Add gn rule to copy protobufjs light" 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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: Iad2fb56471ee34af15eb3910aa293a719a7b1339 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333104Reviewed-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@{#793782}
-
Yi Gu authored
The metric helps with identifying cases that OTPs were being collected without WebOTP API. Note that we only report when a frame has forms parsed. Bug: 1107590 Change-Id: Ib0b81ef2208de1d86fab069a596cfb98f75c2e99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2323586 Commit-Queue: Yi Gu <yigu@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#793781}
-
Jonathan Freed authored
Change-Id: Idd57fc3867bf88ee242d3bf09c68d05647600146 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332848 Auto-Submit: Jonathan Freed <freedjm@chromium.org> Reviewed-by:
Dan H <harringtond@chromium.org> Commit-Queue: Dan H <harringtond@chromium.org> Commit-Queue: Jonathan Freed <freedjm@chromium.org> Cr-Commit-Position: refs/heads/master@{#793780}
-
John Delaney authored
What: No longer reset conversion sql::Database in the error callback Why: This is called synchronously within the sql::Database callstack, so we are actively freeing memory that is still in use. How: Use a bool to track db status for errors instead of the unique_ptr value. Bug: 1111047 Change-Id: I889a3c4e627062ed6472b1cd96301a69632af097 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2330335Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Commit-Queue: John Delaney <johnidel@chromium.org> Cr-Commit-Position: refs/heads/master@{#793779}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 9d3e5b57. With Chromium commits locally applied on WPT: 6d63b58a "Split iceGatheringState tests into separate files & update expectations" 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: rego@igalia.com: external/wpt/css/css-grid NOAUTOREVERT=true TBR=lpz@google.com No-Export: true Change-Id: I1a8f5537acd7e901c5498bd8f953d1629f1b86bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2331912Reviewed-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@{#793778}
-
William Lin authored
To add unbundled consent for GetAuthToken in the extensions API, the token scopes returned from the server needs to be accessible for the extension to know which scopes were approved or rejected. This CL adds the functionality and API changes to allow extensions to obtain the list of accepted scopes. Until unbundled consent is implemented, these returned scopes do not add any additional useful information to the GetAuthToken function. Bug: 1100535 Change-Id: Iba3233ae8a33c49118e7093fd2e9020b2951575f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303596 Commit-Queue: William Lin <williamlin@google.com> Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Alex Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#793777}
-
Internal Frameworks Autoroller authored
TBR=bling-team@google.com Change-Id: I626d94ff6d403a12c655255c9e69668732ce8d39 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332847Reviewed-by:
Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#793776}
-
Ilia Samsonov authored
Ignore-CQ-Freeze: Removes non cq builder Bug: 1104846 Change-Id: Ibacfe41adaa9f5d90d74b3d493dd50cf44b4ae25 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317705 Commit-Queue: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Erik Staab <estaab@chromium.org> Cr-Commit-Position: refs/heads/master@{#793775}
-
Jan Wilken Dörrie authored
This change performs the following mechanical changes in chrome/chrome_cleaner: - Replace base::char16 with wchar_t - Replace base::StringPiece16 with base::WStringPiece - Replace base::UTF16ToUTF8 with base::WideToUTF8 - Replace base::UTF8ToUTF16 with base::UTF8ToWide - Replace base::UTF16ToASCII with base::WideToASCII - Replace base::ASCIIToUTF16 with base::ASCIIToWide - Replace base::ASCIIToUTF16 with base::ASCIIToWide - Inline calls to base::WideToUTF16 - Replace base::NumberToString16 with base::NumberToWString - Replace base::FilePath::AsUTF16Unsafe with base::FilePath::value - Replace STRING16_LITERAL("...") with L"..." Actually replacing base::string16 with std::wstring will be done in a future CL due the large number of occurrences. This change does not change any functionality, it is purely refactoring. Bug: 911896 Change-Id: If1221d5f2f03e559e0e5e5dda1bc6824bb984b83 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332697Reviewed-by:
Varun Khaneja <vakh@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#793774}
-
Sven Zheng authored
The current zip file missing a lib file. TEST: Manually tested adding this file, ash chrome can start and signed in. Bug: 1109287 Ignore-CQ-Freeze: True Change-Id: Icf1cfdc59d4ad3c3f902fba08c3a1878b98ac33f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332689Reviewed-by:
Yuke Liao <liaoyuke@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Commit-Queue: Sven Zheng <svenzheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#793773}
-
Dominic Mazzoni authored
Borrowed heavily from the work Bruce Long did last year in http://crrev.com/c/1666567 (see patchset 18), this introduces UI to toggle caret browsing, now that we've gone through UI review (see go/caret-browsing-ux for approved mocks/strings). 1. Toggles caret browsing with F7 on Mac/Win/Linux, and Ctrl+Search+7 on Chrome OS. 2. When pressing the accelerator, shows a confirmation dialog with a checkbox that allows you to skip the dialog next time. 3. No dialog to disable caret browsing, it just turns off. 4. Caret browsing always resets to false when you start the browser (unless you use the command-line flag, which is still supported it and now turns it on by default). That's so users don't get confused and forget they had it on. 5. Also allows you to toggle in settings. Bug: 977390 AX-Relnotes: Implements UI to toggle Caret Browsing in settings or with F7 Change-Id: Ib8debb6f4154ae5f4a1083db4d8af6ade56d2b3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2320127 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#793772}
-
Guillaume Jenkins authored
Adds support for the CloudReportingEnabled policy, which controls whether enterprise reports are sent to the device management server. This policy is a privacy requirement of launching CBCM. For now, it is declared as a Future policy for iOS until iOS CBCM is launched. Bug: 1066495 Change-Id: Ic4d175df8a52dabb841d905eaa7fa76e1a9c13bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332254 Commit-Queue: Guillaume Jenkins <gujen@google.com> Reviewed-by:
Owen Min <zmin@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#793771}
-
Rohit Rao authored
This reverts commit 216b0ee0. Reason for revert: Breaks symupload on iOS official bots Original change's description: > Roll breakpad to a740aa2625df2c0ed21ed7118bd8ab5faec375b8 > > This is primarily needed to bring in symupload v2 protocol support on Mac. > > a740aa2 Fix Mac symupload non-XCode builds > 28d7cbd Check for tombstone as very first entry in the line table > 7d65240 Properly handle new tombstone values that now appear > 1143368 Port new symbol API to symupload on Mac > 428a01e There are source files with APSL-2.0 as well > 2e6f8de Add 3-clause BSD license text as well > 678c4c5 Include libunwind license in LICENSE file > 721108f fix NDEBUG breakage with new dwarf5 code > 09b0569 fix pointer style to match the style guide > a741027 Add support for dwarf5 line tables > b6f36dd Move GetSectionByName out of CompilationUnit > 8efe0f1 Read dwarf5 compilation unit headers > 72c118f Add and handle various dwarf5 addrx forms > feb2dca Add and handle new dwarf5 string-related forms > > BUG=1021742 > > Change-Id: I7ed42d6696452c58bdbbdee324248c656b180605 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2321537 > Commit-Queue: Michael Moss <mmoss@chromium.org> > Auto-Submit: Michael Moss <mmoss@chromium.org> > Reviewed-by: Mark Mentovai <mark@chromium.org> > Cr-Commit-Position: refs/heads/master@{#793068} TBR=mmoss@chromium.org,mark@chromium.org,nbilling@google.com,saugustine@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1021742 Change-Id: I02d2815c83c4014ee3a9ef49491fc64464bb93a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332843Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Mark Mentovai <mark@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Commit-Queue: Michael Moss <mmoss@chromium.org> Cr-Commit-Position: refs/heads/master@{#793770}
-
Wenhan (Han) Zhang authored
Duplicate "expires_after" attribute in test histograms prevented one test from passing. Change-Id: I9a824eea9cd10d8e4483335ea2ea8a5f1ef1ce30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333173Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#793769}
-
Sergey Ulanov authored
When paused WebMediaPlayerMS copies current video frame to a new buffer. Previously it was doing it ever for frames that were already copied in GpuMemoryBufferVideoFramePool. That's not really necessary. Updated WebMediaPlayerMSCompositor to keep track of the frames that have been copied by GpuMemoryBufferVideoFramePool This allows to avoid copying frames in WebMediaPlayerMS when it isn't necessary. Also update the class to use WTF::Vector instead of std::map. Bug: 1110218 Change-Id: I21d4269c74fec8b5e67329331611f4ae9993d1e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2331003 Auto-Submit: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#793768}
-
Eric Roman authored
Change-Id: If23d2017ad8d15b38ffe22e0f51e5ed070def161 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332584 Commit-Queue: Eric Roman <eroman@chromium.org> Commit-Queue: Matt Mueller <mattm@chromium.org> Auto-Submit: Eric Roman <eroman@chromium.org> Reviewed-by:
Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#793767}
-
Avi Drissman authored
Bug: 1105907 Change-Id: I5c7e37923ed46c1e45139efd318ed595789b9e16 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2330084 Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@google.com> Reviewed-by:
Mark Mentovai <mark@chromium.org> Cr-Commit-Position: refs/heads/master@{#793766}
-
Eric Roman authored
It was previously in a different subdirectory from the file it tests. Change-Id: Ia5fe22ab35ce774456ce8e19b733d73edbee8e3b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333060 Commit-Queue: Eric Roman <eroman@chromium.org> Commit-Queue: Matt Mueller <mattm@chromium.org> Auto-Submit: Eric Roman <eroman@chromium.org> Reviewed-by:
Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#793765}
-
David Maunder authored
This supports the effort of moving Tab attributes to UserData objects Bug: 1109036 Change-Id: Iafb008decac9e8f214d8c86f2d88e76a5b415cbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316803 Commit-Queue: David Maunder <davidjm@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#793764}
-
My Nguyen authored
Since there are no more comments. There should be no need for clang-format off in this file. No-Bug: Drive-by Change-Id: I0ee518af6bcccaed67c7e184bb48d5beb6f0950e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332103Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: My Nguyen <myy@chromium.org> Cr-Commit-Position: refs/heads/master@{#793763}
-
David Black authored
To reduce initial complexity (and possible user confusion) we will allow duplicates in clipboard history. Bug: 1111895 Change-Id: I16995c0b48e98b68c514017c23c509803a068bbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333246Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#793762}
-
Xianzhu Wang authored
The view background is positioned in root element: https://drafts.csswg.org/css-backgrounds-3/#root-background. Invalidate the view background when the root element changes size or content box when needed. Bug: 1083511 Change-Id: I7053fead0bca2655f1c6e13518897235f1fac5a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2330748Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#793761}
-
David Schinazi authored
This test broke when we switched the default version from Q046 to Q050 because Q050 does not accept the legacy Google Alt-Svc format. This CL fixes the test to use the new IETF format that is supported by all QUIC versions. R=renjietang@chromium.org Bug: 1111900 Change-Id: Ice6f2541f3f3f43ce1987fc027c9ec2a51454ec1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333288 Commit-Queue: David Schinazi <dschinazi@chromium.org> Commit-Queue: Renjie Tang <renjietang@chromium.org> Auto-Submit: David Schinazi <dschinazi@chromium.org> Reviewed-by:
Renjie Tang <renjietang@chromium.org> Cr-Commit-Position: refs/heads/master@{#793760}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/f6c424bb5923..22b8657a5558 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-win-chromium 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/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I1370adee2e0bba57833b0aa8b84923cf6b753622 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332325Reviewed-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@{#793759}
-
Yuke Liao authored
An attempt to understand whether ash-chrome is really taking more time to start or getting stuck due to bugs. Bug: 1104318 Change-Id: I691c4681b26506b7f4b17389f7f1f5c867428d36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2333168Reviewed-by:
Sven Zheng <svenzheng@chromium.org> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#793758}
-
John Williams authored
This reverts commit 9b5bc632. Reason for revert: Causes a crash. Original change's description: > [Cast MRP] Stop mirroring when Chrome shuts down. > > Bug: b/161367720 > Change-Id: I619b88eec584b2124dcab6434e59f45cfe0e9c2b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2300661 > Commit-Queue: John Williams <jrw@chromium.org> > Reviewed-by: Takumi Fujimoto <takumif@chromium.org> > Cr-Commit-Position: refs/heads/master@{#793279} TBR=takumif@chromium.org,jrw@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1111760 Change-Id: Ia4236495b58a25cba7b4cb165f36f0a3c63bce8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332328Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Commit-Queue: John Williams <jrw@chromium.org> Cr-Commit-Position: refs/heads/master@{#793757}
-