- 18 Sep, 2019 40 commits
-
-
Tina Wang authored
- Rename the DeviceIconType attributes. - Remove default case of CreateDeviceIcon because of using desktop icon by default. Bug: 1003812 Change-Id: Ie1e1a9cc9ef397769c8b37d25b867dfd1018cf37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1802500 Commit-Queue: Tina Wang <tinazwang@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#697735}
-
James Cook authored
Follow up for comments in: https://chromium-review.googlesource.com/c/chromium/src/+/1807777 If every settings-subpage is inside a settings-section, then we can clean up the while() loop a bit. Also added a test that an associated control gets a search bubble highlight, which is one of the side effects of this function. Bug: 1003970 Test: browser_tests Change-Id: Ib2509dc5e3ffff075e46c1e61823fdf67229ddea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808423 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Cr-Commit-Position: refs/heads/master@{#697734}
-
Mandy Chen authored
Wrap a user facing string in ls so it's localizable. Bug: 941561 Change-Id: If4285af49d7fd3ca8b2b7c2a447c35900e9efae5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1793748 Commit-Queue: Mandy Chen <mandy.chen@microsoft.com> Reviewed-by:
Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#697733}
-
Aidan Beggs authored
This CL updates the URL only for Android to no longer use an answer ID. P-links allow easier tracking and prevent us from deprecating a link that's still used in-product. Bug: 703445 Change-Id: I2bb405e99ef230229a38aab8fa01ae3e5995486d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810840Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Joe DeBlasio <jdeblasio@chromium.org> Commit-Queue: Aidan Beggs <beggs@google.com> Cr-Commit-Position: refs/heads/master@{#697732}
-
Tomasz Śniatowski authored
Don't add a string to a list, as that creates a broken depfile with "deps" of single-character paths. Also while there make the depfile preserve the relative path passed in -- use abspath when it's needed instead of ahead of time. Change-Id: Ib61efc68dfc9ecfaac7f3dcb6007bafb9fb8af67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1811252Reviewed-by:
-
chrome://flagsPeng Huang authored
Change-Id: Ieff9d96ceb1dc1af2e93b4e9cacd90a02c0b9ad0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1802389 Commit-Queue: Peng Huang <penghuang@chromium.org> Reviewed-by:
Jonathan Backer <backer@chromium.org> Cr-Commit-Position: refs/heads/master@{#697730}
-
David Tseng authored
Bug: 990425 Change-Id: I70006053906de020ae8e0ca7f5fda70285e5aeb9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810254 Commit-Queue: David Tseng <dtseng@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#697729}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/122f4535b1be..4599045c795c git log 122f4535b1be..4599045c795c --date=short --no-merges --format='%ad %ae %s' 2019-09-18 yshaul@google.com afdo: Add pupr chrome-profiles handler 2019-09-18 yshaul@google.com Preserve ebuild position and quotes during uprev 2019-09-18 yshaul@google.com Add multiversion support to packages api 2019-09-18 dburger@chromium.org Remove test config override for betty-nyc-arc-release. Created with: gclient setdep -r src/third_party/chromite@4599045c795c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I6272f711bb83f5812b7190cc4911ba0b04074331 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810246Reviewed-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@{#697728}
-
Abigail Klein authored
Convert the arguments passed to accessibility_ui.cc functions into a dictionary rather than a long list. Since the number of arguments passed to accessibility_ui.cc is expected to grow, this will keep the argument list easier to understand. Bug: 785493 Change-Id: I089e0e6621f57b9b453f650da6c2c9b14386a687 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1802198Reviewed-by:
-
Ryan Hamilton authored
Change-Id: Ia5780557422a98e990a188e6b538309c31d8dbf2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810771 Commit-Queue: Ryan Hamilton <rch@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#697726}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/6f9a0238ce40..3f79763629b4 git log 6f9a0238ce40..3f79763629b4 --date=short --no-merges --format='%ad %ae %s' 2019-09-18 thakis@chromium.org Set core.trustctime=false during git checkout operations. Created with: gclient setdep -r src/third_party/depot_tools@3f79763629b4 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@chromium.org 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 TBR=agable@chromium.org Bug: chromium:330461,chromium:642711,chromium:746763,chromium:740301. Change-Id: Ibfd372e677ff40706e35e6f3a7590a7a7bd7d9ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1811343Reviewed-by:
-
Aran Gilman authored
The main consumer of DistillabilityResult on desktop, the Reader Mode icon, updates whenever the user switches tabs. However, the distillability service usually will not run again when the tabs are switched, so the icon will not update if it sees the web contents' distillability only when OnResult() is called. It needs to be retrieve the distillability of a given web contents in between calls to OnResult(), as well as keep track of which web contents the result belonged to. The simplest solution is to store the result in DistillabilityDriver, since it is already correctly scoped to a single web contents. Bug: 952042 Change-Id: I8a8633a52583f04b49298ee5186b24a5eb092fdf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788498Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Aran Gilman <gilmanmh@google.com> Cr-Commit-Position: refs/heads/master@{#697724}
-
Patrick Monette authored
Bug: 1004324 Change-Id: I85a80a8d590c44f2f23659bbafb79c6c3b83e059 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804513 Commit-Queue: Patrick Monette <pmonette@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#697723}
-
Nikunj Bhagat authored
Change-Id: Ib38e214d278c0b4629734e81cb1a473e5b18f5d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1589363 Commit-Queue: Nik Bhagat <nikunjb@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#697722}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/7259f61fb600..56f9ee6b49fc git log 7259f61fb600..56f9ee6b49fc --date=short --no-merges --format='%ad %ae %s' 2019-09-18 nicomazz@google.com Merge "Implement android traced_consumer socket comunication" Created with: gclient setdep -r src/third_party/perfetto@56f9ee6b49fc 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/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: None Change-Id: Ie7c745a0482844fac5bbb6f496035e6577428967 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810247Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/795cab2777e2..377dbef2b5ab Created with: gclient setdep -r src-internal@377dbef2b5ab 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 TBR=jbudorick@google.com Bug: chromium:1005170 Change-Id: I130ff5b06c6483422671e88198fe8380f25af89e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1811339Reviewed-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@{#697720}
-
Mandy Chen authored
This CL adds the following functions: * Runtime.getResourceURL: returns the request URL given resource name and request base URL (e.g. devtools://devtools/bundle/ or devtools://devtools/remote/serve_file/@010ddcfda246975d194964ccf20038ebbdec6084/) * (Runtime) module: return Runtime.Module object given module name * (Runtime.Module) fetchResource: make a request to load a script of the given name in the current module. This is in the Runtime.Module class so that the request can be adjusted based on if the module is remote or not. We can leverage these functions to request specific files. I'm planning to use them to request the appropriate locale file for lighthouse in the audits worker, e.g. `await self.runtime.module('audits_worker').fetchResource('lighthouse/locales/zh.json')`. Bug: 941561 Change-Id: Ib5c7bfdc46bef7d98533c89b12c7d365d4a26dfd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784981Reviewed-by:
Paul Irish <paulirish@chromium.org> Commit-Queue: Mandy Chen <mandy.chen@microsoft.com> Cr-Commit-Position: refs/heads/master@{#697719}
-
Matt Menke authored
SocketPosix::Close() was freeing the write buffer before destroying a socket. Unclear if this is the cause of the linked bug, but it seems not a great thing to be doing. Bug: 1005042 Change-Id: I2cc35154b6f9de8d272244456eeed5fb9465da4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810321Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#697718}
-
Wei-Yin Chen (陳威尹) authored
This should not increase the crash rate. Bug: 718153 Change-Id: I86915c51945db21cd5e528d7270150e648fbfc1b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810035 Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Auto-Submit: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#697717}
-
Adrienne Walker authored
This definitely looks extremely sketchy when reading code. Make the BlobReader owned by a callback itself instead of needing to document everywhere that a raw new is ok because the class is self-deleting. Ultimately, this ends up being the same, but is a readability improvement. Change-Id: I56f19679a97e3bf220a09fc7f84d3772972b52c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804821 Commit-Queue: enne <enne@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Auto-Submit: enne <enne@chromium.org> Cr-Commit-Position: refs/heads/master@{#697716}
-
Antonio Gomes authored
BUG=919392, 787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ie40d53355cbcfeecd65e30f0bd25334e86ebf5cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807716 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#697715}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/8d5e8816..46e424ec 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: I447a153f9a99df61093fa152fd0aba2c01317638 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810723Reviewed-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@{#697714}
-
John Lee authored
The interactions and visual polish of drag and drop is still in flux but this CL adds basic functionality such that TabElements can be dragged to be re-ordered within the tab strip. This CL also only adds support for reordering tabs within the same pinned status. Bug: 989131 Change-Id: I93cd72ea527f53e2973c98ad1739abb11561e285 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800914Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: John Lee <johntlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#697713}
-
danakj authored
Emulation only changes a few VisualProperties, and then calls SynchronizeVisualProperties(), which is a subset of OnSynchronizeVisualProperties(). This moves code from inside SynchronizeVisualProperties() to OnSynchronizeVisualProperties() when the emulator does not change those values. This helps to clarify which code needs to run from the emulator and which does not. R=avi@chromium.org Change-Id: Ia8253ab6a32d4fd2c5f0b69c1310536d75faa6b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809604Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#697712}
-
siashah authored
Bug: 980716 Change-Id: Ib4222f1ab31ed135705a4d3cb3be0531a8ed81a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809408Reviewed-by:
Jared Saul <jsaul@google.com> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Siddharth Shah <siashah@chromium.org> Cr-Commit-Position: refs/heads/master@{#697711}
-
Chrome Metrics Logs authored
Updates the expires_after attribute for 6 histograms that show frequent access in the past 90 days. These are the 80% most frequently used histograms over that time that do not already have a date later than or within 60 days of 2020-03-15. Change-Id: I865023cb3ea0ddfb0b5e2d88551cc6343cef9aeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809418 Commit-Queue: Brian White <bcwhite@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#697710}
-
Andrew Xu authored
Two issues are exposed during tests when enabling the scrollable shelf flag: (1) TabletModeController is destructed before ScrollableShelfView. But the pointer to TabletModeController is not checked in the current code. (2) StatusAreaWidget is destructed before ScrollableShelfView. However, ScrollableShelfView relies on StatusAreaWidget during its destruction. The CL fixes those problems. Bug: 973481 Change-Id: I89ff7acab60960939e586a22c09c0b2d65340c39 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809904Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Andrew Xu <andrewxu@chromium.org> Cr-Commit-Position: refs/heads/master@{#697709}
-
Matt Jones authored
Change-Id: I60385d89b85ef354e4bf86aa6c1580a6a343d988 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1797242 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#697708}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/e3671013e5a1..8391bac5c6f0 git log e3671013e5a1..8391bac5c6f0 --date=short --no-merges --format='%ad %ae %s' 2019-09-18 bsalomon@google.com Add F16Norm mode to viewer 2019-09-18 bsalomon@google.com blurrect_compare GM: draw "masks" in white for RGBA fallback. 2019-09-18 csmartdalton@google.com Weed out duplicates from the proxy version of addDependency 2019-09-18 mtklein@google.com faster PNG encoding in skottie_tool 2019-09-18 csmartdalton@google.com Revert "Use a "decal" domain for filterMaskGPU" 2019-09-18 csmartdalton@google.com Revert "Abolish absClear" 2019-09-18 bsalomon@google.com Use mipmapping/protected to lookup scratch textures. Created with: gclient setdep -r src/third_party/skia@8391bac5c6f0 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC brianosman@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-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=brianosman@google.com Bug: None Change-Id: Idf5b5f85bbe4afcdd14d81268b8c2ae219107ad1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1811337Reviewed-by:
-
Matt Reynolds authored
GamepadService maintains a set of GamepadConsumers that are registered to receive gamepad events. In normal operation, ConsumerBecameActive is called when the consumer is ready to receive events and ConsumerBecameInactive is called when the consumer wants to temporarily pause events without unregistering. The |num_active_consumers_| member tracks how many consumers are currently active and should always equal the count of consumers in |consumers_| that have the is_active flag set. If a consumer calls these methods in an invalid order, this invariant may be broken. For instance, calling ConsumerBecameInactive will decrement |num_active_consumers_| even if the consumer was already inactive. This CL enforces the invariant by changing DCHECKs into CHECKs. This also fixes a bug where consumers_.end() may be dereferenced when ConsumerBecameInactive is called for a consumer that was never added to |consumers_|. BUG=998431 Change-Id: I86c53a481601cf05a66b3b0a8b011412660c805e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1804379Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Matt Reynolds <mattreynolds@chromium.org> Cr-Commit-Position: refs/heads/master@{#697706}
-
Fredrik Söderquist authored
All calls (one) to SetTargetElement(...) that can set a non-null target are surrounded by calls to Disconnect... and Connect..., so we need not disconnect again inside SetTargetElement(...). Similarly we can't get to NeedsToProgress() without having a valid target (DCHECKed in SMILAnimationSandwich::UpdateTiming). Bug: 998526 Change-Id: I73028646021ce6be2169194d6d371f43e19f0c12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810978Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#697705}
-
Ted Meyer authored
This will help us debug _why_ certain D3D11 calls in DXVA are failing. Change-Id: I0e05f87c9411d0ff973a7074908b1ddd58f82416 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809556Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#697704}
-
Robert Ogden authored
Plenty more clean up to do after this, but this is a start. Bug: 1005295 Change-Id: I35e89e0a25aa2eee3f6129158f0b1a5f5c47b6f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810769Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#697703}
-
liberato@chromium.org authored
It turns out to be very convient to have the LearningTask on the client side of the mojo connection. This CL allows us to specify a LearningTask during construction of the client, that is returned by GetLearningTask. Change-Id: I41e852f338dcbeed26a2a20022e8ceaa5226a649 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809891Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: Frank Liberato <liberato@chromium.org> Cr-Commit-Position: refs/heads/master@{#697702}
-
Khushal authored
We currently ignore overlay video enabled when SurfaceControl is enabled, because we never expected it to be used when this feature is enabled since video overlaying is done using the SurfaceControl API in the GPU process. But AR will be using a similar approach for overlaying WebGL based content composited in the browser process using a seperate Dialog. So fix the above by always respecting that bit, even if SurfaceControl is enabled. Also respect the |mAlwaysTranslucent| bit used on low memory devices. The expectation for using a RGB_565 format in the compositor which affects this decision also apply to the SurfaceControl case. Also ensure that if the compositor is using the alpha channel when drawing the primary plane, then it is not marked opaque. R=dtrainor@chromium.org Change-Id: I95bb5603e9e21f0b0e4cb94bbfb56e5443952d7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801753 Commit-Queue: Khushal <khushalsagar@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
-
Antonio Gomes authored
Now that their solely client [1] has been Onion soup'ed, it is possible to bypass (and remove) these helpers in favor of accessing blink::RtcVideoDecoderFactory's and RtcVideoEncoderFactory's ctors directly. [1] //third_party/blink/renderer/platform/peerconnection/video_codec_factory.cc BUG=787254, 919392 Change-Id: Ieb090709080de71fb98d8b2be68afa57ad88fee5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1807714 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
-
Sebastien Marchand authored
The bug causing TabLifecycleUnitTest.CannotFreezeOrDiscardIfConnectedToBluetooth was due to the test running right before it, which was modifying a global state without resetting it to the proper value. Bug: 1004578 Change-Id: I0c8fa306e0d2f40911c18de52c062352d380fb5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1808335 Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Auto-Submit: Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#697699}
-
Tarun Bansal authored
Add experiment to preconnect to predicted redirect target. The redirect target is returned as part of loading prediction by the loading predictor. Change-Id: I9e00363db609470d92dd5f2196aad71868dc68ed Bug: 983234 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1699738 Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Alex Ilin <alexilin@chromium.org> Reviewed-by:
-
Eric Karl authored
Change-Id: I2249d8380a3c46764476d5ec59aec65705c4344d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809114 Commit-Queue: Eric Karl <ericrk@chromium.org> Commit-Queue: vikas soni <vikassoni@chromium.org> Auto-Submit: Eric Karl <ericrk@chromium.org> Reviewed-by:
vikas soni <vikassoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#697697}
-
Maksim Ivanov authored
Refactor the SystemTokenCertDBInitializer class to live in its own .h+.cc files, rather than being defined inside the already heavy chrome_browser_main_chromeos.cc. Other minor changes made there: 1. Removed the Initialize() method, and added a ShutDown() method (the latter will become non-empty in a follow-up CL:1796426). 2. Replaced base::Bind/base::Callback with their modern Once/Repeating analogs. Bug: 725500 Change-Id: Ia172f3cf1e049171914f5fd251a8cc88f6643e55 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1806859 Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Cr-Commit-Position: refs/heads/master@{#697696}
-
