- 29 Nov, 2018 40 commits
-
-
Jonathan Freed authored
GetCatalogTask execution. Bug: 908476 Change-Id: I5459d7fdfe8e5eaf2cd9c42e39c1606710cec838 Reviewed-on: https://chromium-review.googlesource.com/c/1355489 Commit-Queue: Jonathan Freed <freedjm@chromium.org> Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Cr-Commit-Position: refs/heads/master@{#612367}
-
Devlin Cronin authored
Extensions.BackgroundContentsServiceStartupTime has expired. Stop recording it and mark it as obsolete. Bug: 878109 Change-Id: I7701d2320b61d57f367b28fe61fc95cd3db738d9 Reviewed-on: https://chromium-review.googlesource.com/c/1354577Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Drew Wilson <atwilson@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#612366}
-
Ben Pastene authored
This is to prevent failures like https://ci.chromium.org/p/chromium/builders/luci.chromium.try/chromeos-amd64-generic-rel/141038 I'm tired of playing catch-up with all the additional gtest args we keep adding to test retries. Change-Id: I86e1180b62a8f47264c47c0bcb8c2d24a589928d Reviewed-on: https://chromium-review.googlesource.com/c/1353996Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#612365}
-
Hector Carmona authored
Once a local profile enters a synthetic trial for on-boarding, it will remain in it forever. This is not a problem because local profiles should not be running the on-boarding process more than once. If a user creates a new profile, it will be assigned into a synthetic trial based on the experiment at the time of onboarding. Bug: 874153 Change-Id: I4f3b9aea39e75fd01ae7dabd7895ea8aa00e633a Reviewed-on: https://chromium-review.googlesource.com/c/1344990 Commit-Queue: Hector Carmona <hcarmona@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#612364}
-
Marco Paniconi authored
https://chromium.googlesource.com/webm/libvpx.git/+log/615922dfb531..932f8fa04dc1 $ git log 615922dfb..932f8fa04 --date=short --no-merges --format='%ad %ae %s' 2018-11-29 marpan vp9-svc: Add num_encoded_top layer counter 2018-11-28 marpan vp9: Fix condition for disabling noise estimation 2018-11-28 jianj vp9 denoiser: force copy block when last not a reference. 2018-11-27 jingning Remove ineffective condition from rc_pick_q_and_bounds Created with: roll-dep src/third_party/libvpx/source/libvpx R=johannkoenig@google.com Change-Id: Ieeb11b4d376312700c558aa9ecc6ecae6ac9db76 Reviewed-on: https://chromium-review.googlesource.com/c/1355432Reviewed-by:
Johann Koenig <johannkoenig@google.com> Commit-Queue: Marco Paniconi <marpan@google.com> Cr-Commit-Position: refs/heads/master@{#612363}
-
Luna Lu authored
1. use ResourceResponse::ExpectedContentLength to get image file size. 2. merge image policy violation check into one place. Note: ValidateImagePolicies will later be called through the pipeline described in here: https://docs.google.com/document/d/1B_GnzEItUXsUUQXtB1zMiiov86_U1BCFo1F8g5F_yOY to unload unoptimized images as early as possible. Bug:910195 Change-Id: Iab2fc2fee4f2963c308ca1cc585259021a86b3f2 Reviewed-on: https://chromium-review.googlesource.com/c/1355330 Commit-Queue: Luna Lu <loonybear@chromium.org> Reviewed-by:Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#612362}
-
Wei Li authored
WelcomeBackStandardNoPolicy and UpdateWithTwoProfiles were disabled because they were falky on Linux. After some minor fix, they are now ready to be re-enabled. BUG=379579 Change-Id: I2e2a79761b4ab6e0f5c9fb18daa3ae80ed9c282a Reviewed-on: https://chromium-review.googlesource.com/c/1354681Reviewed-by:
Tommy Martino <tmartino@chromium.org> Commit-Queue: Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#612361}
-
Oystein Eftevaag authored
The current Perfetto backend splits _COMPLETE trace events into separate _BEGIN and _END pairs, as it's not very feasible to modify existing events after they're written into the Shared Memory Buffers. This is causing some issues with the trace-viewer which has some assumptions about the ordering of begin/end events vs. async events, and is also bloating the sizes of traces and adding extra overhead for the perf infra. Instead, we now keep the _COMPLETE events in an internal stack in TLS and only emit them when we have their duration. R=eseckler@chromium.org,skyostil@chromium.org Bug: 909728,888558 Change-Id: I80e37264de66d8bbcb6c9095d21047957fd6eb9f Reviewed-on: https://chromium-review.googlesource.com/c/1354503 Commit-Queue: oysteine <oysteine@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#612360}
-
Ian Kilpatrick authored
This went through a couple of simplifying revisions, however this now simplifies where we calculate if an inline level box can contain something that is out of flow. There is also a couple of small fixups left over when I was simplifying this code. Bug: 636993 Change-Id: I5b660df11ff26d3a7aaf72fdd087a13be65739cf Reviewed-on: https://chromium-review.googlesource.com/c/1351262 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Aleks Totic <atotic@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#612359}
-
Meilin Wang authored
TODO: Currently a mouse click on any windows outside (e.g. chrome browser window, feedback window) will be recorded as kUnspecified instead of kOutsidePress for Assistant exit point. A further investigation is needed to figure out how to solve this. Test: local compile and manually test. Change-Id: Ic616a9704a89d40e8f07e0ce20e6669b5976ec76 Reviewed-on: https://chromium-review.googlesource.com/c/1324850 Commit-Queue: Meilin Wang <meilinw@chromium.org> Reviewed-by:Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#612358}
-
Xing Liu authored
This CL disables DownloadMediaParserTest.testParseVideoH264 before Android L, since it fails on Kitkat on certain types of devices. Bug: 907577 Change-Id: I9488898d9807a99eb9d871ecd4bc6e093720d790 Reviewed-on: https://chromium-review.googlesource.com/c/1355246 Commit-Queue: Xing Liu <xingliu@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#612357}
-
Xing Liu authored
This CL adds 3 trace event in background download system to debug potential jank reporting through crash report. Bug: 878593 Change-Id: Ibebca6ba331fb00b5b74a327b900c98e8b516151 Reviewed-on: https://chromium-review.googlesource.com/c/1354638Reviewed-by:
ssid <ssid@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#612356}
-
Karan Bhatia authored
This CL moves the isolated world CSP handling from DOMWrapperWorld (at the platform/bindings/layer) to the IsolatedWorldCSP singleton instance (at the bindings/core layer). The platform/bindings layer doesn't know the concept of a CSP, hence move it to the bindings/core/ layer. Furthermore, DOMWrapperWorld is per world, hence it shouldn't necessarily store isolated world specific data. This is required to store ContentSecurityPolicy instances for isolated worlds to add CSP support for the isolated worlds. This CL should have no behavior change. BUG=896041 Change-Id: I0ac995dbae549379934bfadeed148d4ca91c8706 Reviewed-on: https://chromium-review.googlesource.com/c/1354567Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#612355}
-
Ioana Pandele authored
Bug: 903841 Change-Id: I7cfaac674593dc5786afe05aee988d2800c7886d Reviewed-on: https://chromium-review.googlesource.com/c/1355186Reviewed-by:
Friedrich Horschig [CET] <fhorschig@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#612354}
-
Oksana Zhuravlova authored
Change-Id: Iee49a86a9973be6799fb29004ce560fa8d69d068 Reviewed-on: https://chromium-review.googlesource.com/c/1354621Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#612353}
-
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: I8edcf318977ab7e92f8e09869d851385be4473bb Reviewed-on: https://chromium-review.googlesource.com/c/1355367Reviewed-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@{#612352}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/b88aa2d8ba8e..ee85992c5c61 git log b88aa2d8ba8e..ee85992c5c61 --date=short --no-merges --format='%ad %ae %s' 2018-11-29 mlopatkin@yandex-team.ru Fix atrace_agent on pre-M devices Created with: gclient setdep -r src/third_party/catapult@ee85992c5c61 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-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 BUG=chromium:908761 TBR=sullivan@chromium.org Change-Id: Ibdddb4067a1cd003f68b4a2127aebd55180d2011 Reviewed-on: https://chromium-review.googlesource.com/c/1355620Reviewed-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@{#612351}
-
Mike Wasserman authored
Bug: 910224 Change-Id: Ie1d13e6d2c41b4096ce7912ba494db2fe468c678 Reviewed-on: https://chromium-review.googlesource.com/c/1355196Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#612350}
-
Martin Kreichgauer authored
This changes CtapGetAssertionRequest to carry the unhashed AppID rather only its hashed form, the "Alternative Application Parameter". The unhashed value is passed to the Windows WebAuthn API for U2F sign requests. Bug: 898718 Change-Id: I2d9e6b80463859adba9a786e0c478c9d63e4164b Reviewed-on: https://chromium-review.googlesource.com/c/1354689 Commit-Queue: Martin Kreichgauer <martinkr@chromium.org> Reviewed-by:
Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#612349}
-
Krishna Govind authored
R=kariah@chromium.org Change-Id: I410f2c3c78edc2ef684dbd8fce935dbdb50be14b Reviewed-on: https://chromium-review.googlesource.com/c/1355851Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#612348}
-
agrieve authored
This reverts commit b74cdf4b. Reason for revert: Broke compile of remoting_me2me_host_archive (see bug) Original change's description: > Cronet: Simplify repackage rules > > * Use dist_jar() rather than a custom script > * Avoids needing to extract .jars. > * Avoids needing to reference ".javac.jar" files, which > are somewhat of an implementation detail. > > * Added jar_excluded_patterns support to dist_jar() > > * Merges zip.py and dist_jar.py, since they were doing the > same thing. > > Spot checked that these two jars are the same before/after: > * cronet/cronet_impl_common_java.jar > * cronet/cronet_impl_native_java.jar > > Change-Id: I4d5a5f495ce0376ef2e53b6d4e3c9ebbc3d75ae1 > Reviewed-on: https://chromium-review.googlesource.com/c/1349872 > Reviewed-by: Paul Jensen <pauljensen@chromium.org> > Reviewed-by: Eric Stevenson <estevenson@chromium.org> > Commit-Queue: agrieve <agrieve@chromium.org> > Cr-Commit-Position: refs/heads/master@{#612210} TBR=pauljensen@chromium.org,agrieve@chromium.org,estevenson@chromium.org Bug: 910311 Change-Id: I0d06a36a1694e378bdfafda21f63fd2e5f258239 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/1355887Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#612347}
-
sebsg authored
There were failing. * css1/font_properties/font.html * css1/font_properties/font_family.html * css2.1/t1503-c522-font-family-00-b.html * css2.1/t1508-c527-font-03-b.html * fast/block/float/016.html * fast/text/international/complex-character-based-fallback.html * fonts/cursive.html * svg/text/ligature-queries.html Tbr: drott@chromium.org Bug: 910263 Change-Id: If5a9291579bff7d8c3184a8c02813088df6c06f5 Reviewed-on: https://chromium-review.googlesource.com/c/1355523Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#612346}
-
David Tseng authored
Bug: 882261 Change-Id: I8226d70905146da97e9d1e1846a4e6a8e2cd01be Reviewed-on: https://chromium-review.googlesource.com/c/1336656 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Yuki Awano <yawano@chromium.org> Cr-Commit-Position: refs/heads/master@{#612345}
-
Hui(Andy) Wu authored
Avoid blocking important part of the underlying page. Screenshots: https://drive.google.com/file/d/1QHMmX0s2uIE1v82TNmj7KwR-3PJIipGO/view?usp=sharing https://drive.google.com/file/d/13EipCdWx5sJDn8-BTeS9f3vb_v5poJQA/view?usp=sharing Bug: 119282188 Change-Id: I4a5c5144f0dd4ef6803c270129d18cb716438a17 Reviewed-on: https://chromium-review.googlesource.com/c/1340940Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Ganggui Tang <gogerald@chromium.org> Commit-Queue: Hui Wu <wuandy@chromium.org> Cr-Commit-Position: refs/heads/master@{#612344}
-
Paul Jensen authored
Previously my DNS histograms incorrectly thought the internal resolver was not used when it in fact was. Bug: 908535 Change-Id: I1c3a8d41298b91d23236649de72299c47cf93926 Reviewed-on: https://chromium-review.googlesource.com/c/1351742 Commit-Queue: Paul Jensen <pauljensen@chromium.org> Reviewed-by:
Misha Efimov <mef@chromium.org> Cr-Commit-Position: refs/heads/master@{#612343}
-
Eugene But authored
Made Stop and Reload no-op during session restoration. For embedder WebState should behave as idle during session restoration, so ignoring Reload and Stop is fine. Reload will eventually happen after the restoration is complete and stop should be actually no-op for an idle WebState. Bug: 877671 Change-Id: I14c4b1d06122aad03406510c9b3e3d57d80e5f93 Reviewed-on: https://chromium-review.googlesource.com/c/1354035Reviewed-by:
Danyao Wang <danyao@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#612342}
-
Ryan Tseng authored
https://chromium-review.googlesource.com/c/chromium/src/+/1252625 was supposed to do a DCHECK to ensure the semantics of ToIsolate() remained unchanged, but the check was incorrect. This fixes the semantics. The next step is to remove ToIsolate(ExecutionContext). Bug: 889542 Change-Id: Ic0a84fa74cafef47e43858f4dae4880b2f7757ec Reviewed-on: https://chromium-review.googlesource.com/c/1355190Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org> Cr-Commit-Position: refs/heads/master@{#612341}
-
Peter Wen authored
Now that CrashKeys.java and related files are in components/, tiborg@ and I can help isherman@ co-own these files to reduce maintenance burden. Bug: None Change-Id: I790545b5976cdbd2109bfc57f8c615eacdbb6a90 Reviewed-on: https://chromium-review.googlesource.com/c/1355483 Commit-Queue: Ilya Sherman <isherman@chromium.org> Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#612340}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/3ee605d7ccb9..8ce3dbabb8b5 git log 3ee605d7ccb9..8ce3dbabb8b5 --date=short --no-merges --format='%ad %ae %s' 2018-11-29 zoddicus@users.noreply.github.com Merge two implementations of ValidateExecutionScope (#2131) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@8ce3dbabb8b5 The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-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=dsinclair@chromium.org Change-Id: I630443223412245a2f9250cbca06de98376327de Reviewed-on: https://chromium-review.googlesource.com/c/1355621Reviewed-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@{#612339}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/9af233a50848..dd530811aba9 git log 9af233a50848..dd530811aba9 --date=short --no-merges --format='%ad %ae %s' 2018-11-29 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@dd530811aba9 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-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=agable@chromium.org Change-Id: I45f77bd1ed1f30a6c5a038907d4c12dc7deaea28 Reviewed-on: https://chromium-review.googlesource.com/c/1355622Reviewed-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@{#612338}
-
Fabrice de Gans-Riberi authored
This changes the optimization level for debug builds on Fuchsia to optimize for size. This fixes an issue where the KVM would run out of space on the Fuchsia debug component bot. Bug: 910243 Change-Id: I7b1257c26b820af97b7c29151cde8f1a52692f5e Reviewed-on: https://chromium-review.googlesource.com/c/1352621 Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#612337}
-
Matthew Wang authored
FT pref used to be a dictionary with a single boolean field. Make it a boolean instead. Bug: 909753 TEST: shill unittests, browser_tests Change-Id: I80304f6be20ebc9e7eec56f0dd2f36e2e8cf5af5 Reviewed-on: https://chromium-review.googlesource.com/c/1354343 Commit-Queue: Matthew Wang <matthewmwang@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#612336}
-
manuk authored
This CL intends to style the output table to approximately resemble the end goal. There will be more styling changes in future as 1) a single CL for all the styling changes would be too large, and 2) some of the styling changes depend on changes to how we generate and manage the HTML elements from the JS code. In addition to 1) trivial CSS styling, this CL also 2) scales the table and column widths with the page width (down to a minimum width) and therefore gives all tables the equal width and consistent column widths; and 3) merges the combined results and individual provider results tables into a single table delimited with inner header rows. Bug: 891303 Change-Id: Ifc823456a8404ade90f34a2c07545c1f643cb96d Reviewed-on: https://chromium-review.googlesource.com/c/1354132 Commit-Queue: manuk hovanesian <manukh@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#612335}
-
Peng Huang authored
Bug: 905337 Change-Id: I14e4188def4719f0b04151bf9d76731a46889612 Reviewed-on: https://chromium-review.googlesource.com/c/1351266Reviewed-by:
Michael Spang <spang@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#612334}
-
Alex Leung authored
This reverts commit f1854090. Reason for revert: Causes internal b/120143260. Change-Id: I83286ee7cb9d5e96e63be66bc47efda802421232 Reviewed-on: https://chromium-review.googlesource.com/c/1354326 Commit-Queue: Alex Leung <alexleung@google.com> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Cr-Commit-Position: refs/heads/master@{#612333}
-
Jonathan Backer authored
When we access RasterDecoderContextState::gr_context, the RasterDecoderImpl::context_state may become inconsistent with actual GL context state and we should set need_context_state_reset Bug: 910274 Change-Id: I834ce6e4cf9b5fe549de432b3bc6c06fa2a6ccbd Reviewed-on: https://chromium-review.googlesource.com/c/1355438 Commit-Queue: Jonathan Backer <backer@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#612332}
-
Troy Hildebrandt authored
SharedProtoDatabaseClient inherited from UniqueProtoDatabase which was potentially risky since we can accidentally use some of UniqueProtoDB's functionality resulting in incorrect keys/entries from the database because we're not stripping/adding prefixes. We now inherit from ProtoDatabase to ensure that any functions added in the future must also be implemented in SharedProtoDatabase. Bug: 870813 Change-Id: I270afdb9dcffa0ba045756272a2f36af9cc93048 Reviewed-on: https://chromium-review.googlesource.com/c/1355423Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Cr-Commit-Position: refs/heads/master@{#612331}
-
Dale Curtis authored
This is part 1/4 CLs to move the <audio>/<video> elements off of a synchronous API that can lead to renderer hangs and premature audio renderer errors. BUG=905506 TEST=updated tests, compiles. Change-Id: I9b26ed5ed8faea2f691b3a415dbdc79e18f5125a Reviewed-on: https://chromium-review.googlesource.com/c/1347588 Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Olga Sharonova <olka@chromium.org> Cr-Commit-Position: refs/heads/master@{#612330}
-
Eric Roman authored
Bug: 910212,729849 Change-Id: Iff0a02015b4b6dabfeff5b3865468df70748ddd6 Reviewed-on: https://chromium-review.googlesource.com/c/1355492Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#612329}
-
Xi Han authored
Bug: 902311 Change-Id: I6f39faecc38ad002a70bfda7fd355ab83afc0889 Reviewed-on: https://chromium-review.googlesource.com/c/1337487Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Commit-Queue: Xi Han <hanxi@chromium.org> Cr-Commit-Position: refs/heads/master@{#612328}
-