- 20 Aug, 2020 40 commits
-
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-aemu-chromium-autoroll Please CC chonggu@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: chonggu@google.com Change-Id: Ie36b7fb24ee930592df6dfe1b94ff2428d4e3f0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366476Reviewed-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@{#800316}
-
Andres Calderon Jaramillo authored
This CL renews the "enable-vaapi-jpeg-image-decode-acceleration" and "enable-vaapi-webp-image-decode-acceleration" flags which are still useful to experiment with the JPEG and WebP hardware decode path in Chrome OS (not launched yet). Bug: None. Test: None. Change-Id: Id314de274805667c05c5d340a58c0e0ad254be25 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367733Reviewed-by:
Andres Calderon Jaramillo <andrescj@chromium.org> Reviewed-by:
Gil Dekel <gildekel@chromium.org> Commit-Queue: Andres Calderon Jaramillo <andrescj@chromium.org> Cr-Commit-Position: refs/heads/master@{#800315}
-
Lan Wei authored
We have replaced ContinuousMouseScrollBy with gpubenchmarking.smoothscrollbyXY to simulate scroll actions, now we can remove it from eventsender. Bug: 1047176 Change-Id: I770c2ba9a8a79f875d46be379ceec533b65e837d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2361184Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Lan Wei <lanwei@chromium.org> Cr-Commit-Position: refs/heads/master@{#800314}
-
Russ Hamilton authored
Instrument RTCIceCandidate.{address,candidate,port,relatedAddress,relatedPort} for web Identifiability study. Bug: 973801 Change-Id: If0783bc203c996cf7603b47395e1f628905ac9f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2359735 Commit-Queue: Russ Hamilton <behamilton@google.com> Reviewed-by:Florent Castelli <orphis@chromium.org> Cr-Commit-Position: refs/heads/master@{#800313}
-
Avi Drissman authored
The "Manage People" window is not correctly labeled in the Task Manager. Do so. Bug: 1115850 Change-Id: Ib749091a49c34e5666b2b82d14c7852f40a2e829 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364954Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#800312}
-
Clark DuVall authored
Incognito mode should not automatically grant DSE permissions. Bug: 1119910 Change-Id: I387ade23bdc654b7b4ab0337522a09a62e6268e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367634Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#800311}
-
Caitlin Fischer authored
The feature is disabled by default. Also, rename two existing IDCollectionKeys. This change paves the way for limiting the set of experiment IDs included in X-Client-Data headers sent in third-party contexts. Bug: 1094303 Change-Id: Ib535b0b8657aeb82570c552b31d722bd30168019 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363467 Commit-Queue: Caitlin Fischer <caitlinfischer@google.com> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#800310}
-
James Cook authored
Bug: none Change-Id: Id2b6ccc6985b92d8c22ecf20fb8ae3edc3bac9d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367031 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#800309}
-
adamta authored
Adds URL for 'Learn More' item in the NTP' Discover feed header menu. Bug: 1111837, 1085419 Change-Id: If0bd7519f213cfb0667fe67c962d6d9ceef1d62b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367732Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Adam Trudeau-Arcaro <adamta@google.com> Cr-Commit-Position: refs/heads/master@{#800308}
-
David Bienvenu authored
Also remove base/compiler_specific.h includes and other cleanup. No functional changes. Bug: 1010217 Change-Id: I29bcb79a46b7ee4dbb1bf92e59f55d13599a98cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365752 Commit-Queue: David Bienvenu <davidbienvenu@chromium.org> Reviewed-by:
David Bertoni <dbertoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#800307}
-
Mei Liang authored
This CL also avoids MobileTabReturnedToCurrentTab being logged twice when using back button to leave GTS. Bug: 1025111, 1119414 Change-Id: Ie0839f27481b9b6033e4f8547ec85f7790207a2d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366032Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Mei Liang <meiliang@chromium.org> Cr-Commit-Position: refs/heads/master@{#800306}
-
Alex Ilin authored
New colored avatars use person_filled material design icon instead of person_outline. It also has two variants of the icon with different padding. The larger icon has bigger padding. See mocks in the attached bug for the reference. Bug: 1100835 Change-Id: I563ed52b9176a0bbed2abcf3e32cd8209e8da315 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362898Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Alex Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#800305}
-
Mei Liang authored
IncognitoTabSnapshotController sets FLAG_SECURE when viewing incognito tabs. The AssistStatusHandler is no longer needed. This CL deletes the AssistStatusHandler and TabbedAssistStatusHandler. In addition, this CL deletes the #addOverviewModeObserverPreNative in ChromeTabbedActivity, and renames #addOverviewModeObserverPostNative() to #initJourneyManager(). Change-Id: I0a729e78089f48a810710b944af92cebeeb79ceb Bug: 1108496 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2337496 Commit-Queue: Mei Liang <meiliang@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#800304}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1597838234-9c863d7c9dc15ae374a8358c943ba78053f4a35c.profdata to chrome-mac-master-1597946202-cb1005c445d83fc8c9db9d03ba0a749f78675139.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-mac-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:mac-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: I979c9f55d04ca2f0d7fd5218afbaae963a892e01 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367752Reviewed-by:
-
Chris Harrelson authored
There are already hit tests and animations entirely in the impl thread for such scrollbars. Bug: 1116800 Change-Id: I2714258717c89e0ea41232eb86abe12570526d28 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364972 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
-
Elaine Chien authored
Bug: 1057764 Change-Id: I0a585d6b98a0a6323a50220b16a90d004f2af24b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363306 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#800301}
-
Olesia Marukhno authored
Fixed a11y for remove suggestion button and buttons in dedicated row (set role to be list box option and set correct accessible name). Fixed selection event not being triggered for dedicated row buttons (GetSecondaryButton wasn't returning buttons from dedicated row). Fixed value changed event triggering extra time when switching selection from button to button. Bug: 1104264 Change-Id: If5fc4692ac4e86f50e1a1d1b754feec795b76c10 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366758 Commit-Queue: Olesia Marukhno <olesiamarukhno@google.com> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#800300}
-
Justin Donnelly authored
Follow up to https://crrev.com/c/2363852, which added Omnibox.SuggestionUsed.Pedal. Tested manually with chrome://histograms. Bug: 893183, 1117634 Change-Id: I9033ea200d820f74cb2d5fd0162383496128bd75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367472Reviewed-by:
Orin Jaworski <orinj@chromium.org> Reviewed-by:
-
Sven Zheng authored
First adds the target with one test file in it. Now it can compile and run on device, and fails. I will get the target pass in the following work. Bug: 1117345 Change-Id: If3acc03e002a5ec23f1bcac5bff0ff7b1e3403a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360598 Commit-Queue: Sven Zheng <svenzheng@chromium.org> Reviewed-by:
Ian Struiksma <ianstruiksma@google.com> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#800298}
-
Joanmarie Diggs authored
AXSelection::Select has logic meant to handle selection within an native text field. But when the selection is being set, the base and extent may be the static text descendant of the AXObject associated with the field. In this instance, the logic for text fields is not being used. Fix this by retrieving the native text control ancestor AXObject. Then, to ensure there is no behavior change, set focus on the element after the selection range was successfully set. Bug: 1113650 AX-Relnotes: N/A Change-Id: I5b1db3dcf7d01baf971d5858c936a4e5d168543e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2352898 Commit-Queue: Joanmarie Diggs <jdiggs@igalia.com> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#800297}
-
Thomas Guilbert authored
This CL changes the behavior of VideoEncoder to make an internal copy of frames passed in through encode(), before destroying the passed frame. This means that users don't need to call destroy() after passing frames to encode(), which could result in a frame being invalidated before we send it to the internal encoder. It also means we can destroy the frame internally when we no longer need it without risking that it might still be in use by the user. Users that which to keep their frames alive after giving them to the encoder can send a cloned copy instead. Bug: 1108023 Change-Id: Ie5cd98d688a53314ac16a6a3ac7afe2a65d742f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363863 Commit-Queue: Thomas Guilbert <tguilbert@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Cr-Commit-Position: refs/heads/master@{#800296}
-
Yao Xiao authored
When the feature is enabled: the 1st floc computation condition will not only be first-sync-enabled-seen, but would also require first-blocklist-loaded-seen, as both conditions are likely to occur after a while after browser start. When floc is computed from history, invalidate it if it's in the blocklist; invalidate any floc when the blocklist is not yet loaded / has error in loading. Bug: 1062736 Change-Id: I81dcfc27b0d2a7023ba1559d30131de41d5842ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365396Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Commit-Queue: Yao Xiao <yaoxia@chromium.org> Cr-Commit-Position: refs/heads/master@{#800295}
-
Ryan Harrison authored
https://dawn.googlesource.com/tint.git/+log/5e7ef27ca7b1..b08e25388629 $ git log 5e7ef27ca..b08e25388 --date=short --no-merges --format='%ad %ae %s' 2020-08-17 sarahmashay [validation] Validates if return statement type matches function return type 2020-08-17 sarahmashay [validation] Validates function name uniqueness 2020-08-13 sarahmashay [validation]: Add disabled test for validating return statement type matches function types 2020-08-13 sarahmashay [validation] Validates return statement 2020-08-13 rharrison Fix handling of .spvasm inputs Created with: roll-dep src/third_party/tint/src Change-Id: I3b34649e3994f4d894f298d96f9d7a0bcfafe471 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2359748 Auto-Submit: Ryan Harrison <rharrison@chromium.org> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Commit-Queue: Ryan Harrison <rharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#800294}
-
Xianzhu Wang authored
Use the new compositing reason instead of kWillChangeTransform to distinguish from will-change:transform specified in web pages which implies that the transform may change to any arbitrary value. Bug: 1114504 Change-Id: I8a1b4e5fa909dd50f3ae6465cb40d9336c9cd640 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367446Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#800293}
-
Erik Chen authored
The transition to Lacros is intended to be transparent to websites. As such, Lacros needs to set the ChromeOS user-agent. However, in some contexts (e.g. chrome://settings) Lacros must behave like the Linux browser. The problem is that webUI makes the assumption that user-agent uniquely identifies platform, and that there is only one browser per platform. Thus, setting the ChromeOS user-agent triggers ChromeOS settings behavior, which uses ChromeOS-only private extension APIs, which crashes the renderer. The appropriate long-term fix to this is to correctly plumb additional information into WebUI so that it can distinguish between Linux, ChromeOS and Lacros browsers. This is currently not possible without refactoring a substantial number of tests. This CL is a stopgap measure to allow chrome://settings to be usable in Lacros. Bug: 1096590, 1111985 Change-Id: Iabb8691e37ba6db4baba680e3d624248e8647321 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367473Reviewed-by:
Darin Fisher <darin@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Auto-Submit: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#800292}
-
Chris Cunningham authored
Change-Id: I82a28a93cdec01e587c2f281561af041010b9446 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364075 Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Cr-Commit-Position: refs/heads/master@{#800291}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: I33fcd43f4524d498142c5a49e3a508b4c3797b08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366671Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#800290}
-
Dan Sanders authored
Previously AudioDecoder would read the whole buffer, rather than the provided view of it. Bug: 1094084 Change-Id: Ibdf601e1d01114456093230c5daa137807566f6b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367641Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Commit-Queue: Dan Sanders <sandersd@chromium.org> Auto-Submit: Dan Sanders <sandersd@chromium.org> Cr-Commit-Position: refs/heads/master@{#800289}
-
Chromium WPT Sync authored
Using wpt-import in Chromium c4db0322. With Chromium commits locally applied on WPT: 3b705a74 "WPT: Service Worker: Add unexpected state check to wait_for_state()" 9f52c7e4 "COOP: add reporting to redirects" f3b221b0 "[COOP] access reporting: Extend report.body.type values." f3c8d8d0 "Move WebRTC insertable streams web tests to WPT" 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=foolip@google.com No-Export: true Change-Id: I4f6e723f1a210780235b822d7e4e6ce21b5ce1f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367453Reviewed-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@{#800288}
-
Sammie Quon authored
We are getting some pushback from system services team. Disable for M86 :(. Test: manual Bug: 1111479 Change-Id: Ifce10b92081a7c162758273221fedfe8de76563b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367118 Commit-Queue: Sammie Quon <sammiequon@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#800287}
-
Prashant Nevase authored
After crrev.com/c/2352251, the code added for storing cache result after doing initial classification, is not required. This patch removes it. Bug: 1094005 Change-Id: I010bfeba0d97e9360482a57ba83bae43e178c88e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364383Reviewed-by:
Fredrik Söderquist <fs@opera.com> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Prashant Nevase <prashant.n@samsung.com> Cr-Commit-Position: refs/heads/master@{#800286}
-
Shakti Sahu authored
Bug: 1114883 Change-Id: I4b067fd6fa31feb0673186170f116134619f6e69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347915 Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Hesen Zhang <hesen@chromium.org> Cr-Commit-Position: refs/heads/master@{#800285}
-
Sadrul Habib Chowdhury authored
The dropped-frame counter needs to be reset after navigation to avoid showing (or reporting) dropped-frame count from the previous page. BUG=1115376 Change-Id: Id9fd54df5fcaf31b6d69e7e7d1190ac29581b6df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366613 Auto-Submit: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#800284}
-
Sky Malice authored
Bug: 1103476 Change-Id: Iabfeafad5641f6735dfc243e5000369bc416338e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366653Reviewed-by:
Wenyu Fu <wenyufu@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#800283}
-
Dan Sanders authored
This is implemented in terms of an |is_closed_| state, which prevents reentrancy during shutdown. Bug: 1045247 Change-Id: I69e8bd095979f6571fbe95b298f91166b61dc9f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363998 Commit-Queue: Dan Sanders <sandersd@chromium.org> Reviewed-by:
-
Garrett Beaty authored
Some CI builders are mirrors for CQ try builders despite not being a part of the main waterfall. These builders are handled by separate sheriffing rotations for trunk, but the branches only have a single rotation, so make sure that when config is generated for the branch, they appear on the main console. Change-Id: I6e7370606968d9a255eb08b5a7c4d2c0fef90220 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367635Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> Auto-Submit: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#800281}
-
Nick Burris authored
Add the body view to complete the secure payment confirmation UI. This patch adds the title text and the merchant, instrument, and total rows. Bug: 1110322 Change-Id: Ied9d2c36cf3581108f811d82aab73c207c37dbd3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365364 Commit-Queue: Nick Burris <nburris@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#800280}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/8c58b2beba89..e592b36b32d8 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 xialinyan@google.com,mohsen@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:1113326 Tbr: xialinyan@google.com,mohsen@google.com Change-Id: I3247007ef560f1dce015779b1f60c46751d0888b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367280Reviewed-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@{#800279}
-
Tommy Martino authored
Change-Id: Ie7416541017fcb2a8dcbfaa61477319b48c1e50f Bug: 1061949 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364223 Commit-Queue: Tommy Martino <tmartino@chromium.org> Reviewed-by:
sebsg <sebsg@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Cr-Commit-Position: refs/heads/master@{#800278}
-
Thomas Guilbert authored
This CL front-loads the configuration parsing and codec support verification into the Configure() method. This means that and invalid or unsupported config should immediately throw an exception, rather than being reported in the error callback. This CL also allows encoders to be re-configured. Bug: 1116783, 1094166 Change-Id: I88df452cfa2a1b0fcfeb4da6c279665661d7380f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363711 Commit-Queue: Thomas Guilbert <tguilbert@chromium.org> Reviewed-by:
-
