- 28 Sep, 2020 40 commits
-
-
Thanh Nguyen authored
This CL ClearIndex function to Index and its subclasses. This function clears all the data stored in the index. Bug: 1132170 Change-Id: Idbec8ed4c721f09577b40c59b9cf24ebe5060252 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2428273Reviewed-by:
Jia Meng <jiameng@chromium.org> Commit-Queue: Thanh Nguyen <thanhdng@chromium.org> Cr-Commit-Position: refs/heads/master@{#811136}
-
My Nguyen authored
Error was "Pref not found for element SETTINGS-TOGGLE-BUTTON source: chrome://os-settings/controls/pref_control_behavior.m.js (41) Clear document.body.innerHTML in setup of the second suite as the document state in the first suite seems to intefere with the second suite. Clearing should be in setup because tests in other suites might intefere with these particular ones. Tested with 1000 repeat and stress cpu to 100%. Bug: 1132577, 1132259 Change-Id: I38cb14a5b892fffdf157a93fda2b7c805c454e92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434584 Commit-Queue: My Nguyen <myy@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#811135}
-
Findit authored
This reverts commit 26e5926b. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 811131 as the culprit for failures in the build cycles as shown on: https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzI2ZTU5MjZiNjY5NzEwODNkOWEwNTBiNDM0OGUxM2VjOTZjZWRiZTAM Sample Failed Build: https://ci.chromium.org/b/8867933801572145104 Sample Failed Step: compile Original change's description: > Video Tutorials : Moved language to a server side concept > > This CL > 1 - Adds language struct to the proto consisting of name, native text, > and locale. > 2 - VideoTutorialService will still store the preferred locale in a > pref which will be used as a key when iterating through the > supporting languages. > 3 - Removes the checked in strings for languages > > TODO : Change server side proto > > Bug: 1132161 > Change-Id: I1483903e0615714aa790b6af1fbfaa4c6637c802 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429979 > Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> > Reviewed-by: Min Qin <qinmin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#811131} Change-Id: I88a5a6858e0ca686b5a28327664dd80c6eeb61a8 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1132161 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434695 Cr-Commit-Position: refs/heads/master@{#811134}
-
Jacobo Aragunde Pérez authored
Bug: 1082217 Change-Id: I97159b1cb3b6ccf8c57f17f54d011447fa6db681 AX-Relnotes: n/a. Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432027Reviewed-by:
Trent Apted <tapted@chromium.org> Commit-Queue: Jacobo Aragunde Pérez <jaragunde@igalia.com> Cr-Commit-Position: refs/heads/master@{#811133}
-
Yuki Shiino authored
Refactors bind_gen/ to support IDL callback functions and IDL callback interfaces in https://crrev.com/c/2434004 Bug: 839389 Change-Id: Ie58f15269efe7c80f0157c4296f0cb37465deab0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432708Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#811132}
-
Shakti Sahu authored
This CL 1 - Adds language struct to the proto consisting of name, native text, and locale. 2 - VideoTutorialService will still store the preferred locale in a pref which will be used as a key when iterating through the supporting languages. 3 - Removes the checked in strings for languages TODO : Change server side proto Bug: 1132161 Change-Id: I1483903e0615714aa790b6af1fbfaa4c6637c802 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429979 Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#811131}
-
Alex Rudenko authored
shadow DOM with unslotted elements. This CL adds additional tests that help to reproduce the issue that was fixed in [1]. [1]: https://chromium-review.googlesource.com/c/chromium/src/+/2426450 Fixed: 1130920 Change-Id: I45a79f5a88d50ccebe543391e47533511b9ae384 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2421820 Commit-Queue: Alex Rudenko <alexrudenko@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Mathias Bynens <mathias@chromium.org> Reviewed-by:
Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#811130}
-
Yuki Shiino authored
Since https://crrev.com/c/2434004 will be going to introduce bindings::CallbackInvokeHelper, which is expected to reduce the APK size by 7KB, then some of IDL callback APIs need to be accessible from CallbackInvokeHelper. So this patch moves such APIs to public. Bug: 839389 Change-Id: I3eafa3992d21d7c011b7fcff07e4c7d57eb8a32f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432149Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
-
Noel Gordon authored
JEITA EXIF color space defines and supports two color spaces only per the JEITA specification: sRGB and AdobeRGB1998. The PIEX C++ returns their values as strings 'sRgb' and 'adobeRgb' so return those values in the PiexLoaderResponse [1]. Note JEITA EXIF color space indicates the color space only. The image does not have an embedded ICC Color Profile. A future option for piex is to slice open the JPEG preview, insert an ICC Color Profile in the AdobeRGB1998 case, before returning the PiexLoaderResponse. [1] In files app, make image_request_task.js convert the value in the PiexLoaderResponse to ColorSpace type. No change in behavior. Bug: 1132695 Change-Id: Ifa076635dc5672a552094bf1b5fa9e69e164d9d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432150Reviewed-by:
-
tby authored
The cooldown between successive calls is set to 15 minutes by default, but is controlled via an experiment parameter if we need to change it later. Bug: 1034842 Change-Id: Ief80a68db10bf67b67c55d679555e6d55b49fab1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434072 Commit-Queue: Tony Yeoman <tby@chromium.org> Reviewed-by:
Thanh Nguyen <thanhdng@chromium.org> Cr-Commit-Position: refs/heads/master@{#811127}
-
Tal Pressman authored
Use GetAgentSchedulingGroup().InitProcessAndMojos() instead. Bug: 1111231 Change-Id: Iee0addc20fa11a7bbeb8fd437455ed26aaf6e2db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434107Reviewed-by:
-
Koji Ishii authored
This patch fixes the logic to recalculate ink overflow, especially for self-painting layers. |PaintLayer| calls |RecalcVisualOverflow| for its owner |LayoutObject|. |LayoutBlockFlow| already redirects this to |NGFragmentItems|, but inline boxes and atomic inlines did not handle this. They have ink overflow stored in corresponding |NGFragmentItem|s, and they need to be recalculated. This patch handles this situation by adding |LayoutBoxModelObject::RecalcVisualOverflow|. Currently, there are multiple workarounds added to pass all tests, but they are causing sometimes redundant, or sometimes missing recalculations. This fix allows to remove one of such workarounds in |NGFragmentItem::RecalcInkOverflowForCursor|, that calls |RecalcInkOverflow| even if the child is |HasSelfPaintingLayer|. Removing this workaround reduces redundant recalculations. Three sets of tests cover this change: 1. This patch adds a unit test that demonstrates when the current code fails to recalculate ink overflow, originally found while investigating <crbug.com/1128199>. 2. When |NGFragmentItem| is enabled, reading dirty self ink overflow hits DCHECK. Not hitting DCHECK can ensure that this change calculate ink overflow for all cases where we read self ink overflow. 3. This patch adds a DCHECK when reading dirty self-and- container ink overflow. 215 unit tests and 653 web tests fail this DCHECK <crrev.com/c/2432707> but pass with this patch (except when NG block fragmentation is enabled, see notes below.) Note: * Both legacy and |NGPaintFragment| have the same issue. This patch addresses only for |NGFragmentItem|. * |LayoutBlockFlow| does not handle |RecalcVisualOverflow| correctly for multicol. This was fixed in r810561 <crrev.com/c/2428516>. Many `virtual/layout_ng_block_frag` tests fail without that fix when this patch is applied. * 3 `virtual/layout_ng_block_frag` tests still fail the new test #3 above, even after the fix above. crbug.com/1132619 tracks this issue. Bug: 1128199, 1130856 Change-Id: I42a75e1dad7c2b35146cca4ddaaf9763df4e7440 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2428530 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#811125}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/0922403dcdc6..ad6aeace6eee 2020-09-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll dawn from eec9edfd5769 to ed2b465f8607 (2 revisions) 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/+doc/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 Cq-Do-Not-Cancel-Tryjobs: true Bug: None Tbr: brianosman@google.com Change-Id: I154ecc6629799cc17cf8f6b571df129a87c55351 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434476Reviewed-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@{#811124}
-
Takashi Sakamoto authored
This CL fixes the following 2 tests' failures: - OutOfMemoryDeathTest.AlignedAlloc - OutOfMemoryDeathTest.AlignedRealloc Bug: 998048 Change-Id: I83f5ac00aded8d750ab610db3c17c17fad8b7074 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2418631Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Takashi Sakamoto <tasak@google.com> Cr-Commit-Position: refs/heads/master@{#811123}
-
tby authored
The steps involved are: 1. Get some results from the cache 2. Run them through LocateFilesByItemIds to convert item IDs into file paths 3. Assign relevance scores and create FileResult objects. Bug: 1034842 Change-Id: If217d5eb5645e9a5eefa8a57609d9ea91014db4b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434108Reviewed-by:Rachel Wong <wrong@chromium.org> Commit-Queue: Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#811122}
-
Matthew Denton authored
Add seccomp ioctl numbers and newer seccomp flags necessary for the USER_NOTIF broker. Also adds linux_prctl.h for prctl constants like PR_SET_PTRACER. Bug: 1117351 Change-Id: Idf56efcb189f1b20f13b23bacc71dd1b38786c4b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2407064 Commit-Queue: Matthew Denton <mpdenton@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#811121}
-
Christopher Lam authored
This CL removes the kSystemWebApps feature which is now always true. Bug: 1067073 Change-Id: Ib2e142fdbd0499e745934d28fac03e2a5695fa08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2395298 Commit-Queue: calamity <calamity@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Reviewed-by:
Jiewei Qian <qjw@chromium.org> Cr-Commit-Position: refs/heads/master@{#811120}
-
Noel Gordon authored
JEITA EXIF image orientation is a number in [1..8]. PIEX C++ defaults to 1 if there is no image orientation or its value is invalid. Return the value in the PiexLoaderResponse and let the client code deal with converting it to other types [1]. [1] In files app, make image_request_task.js convert the value in the PiexLoaderResponse to ImageOrientation type. No change in behavior. Bug: 1132695 Change-Id: I728680d17c31426d72b6b09bd52ee821540cbcd0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432231Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/d6aeaf33afbe..68998788dee4 2020-09-28 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. 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: I18a2a33ded931701f7820ea03eef8bec5c65da85 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434864Reviewed-by:
-
Sergei Datsenko authored
Bug: 1125669 Change-Id: Ia0de3d3be2fb1f91f49a1ec5cf07139625bdf924 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2426148 Commit-Queue: Sergei Datsenko <dats@chromium.org> Reviewed-by:
Austin Tankiang <austinct@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#811117}
-
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-sdk-chromium-autoroll Please CC cr-fuchsia+bot@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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: Iddf2ffa2ea560eb98e9928761cf9ff367914e3d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434826Reviewed-by:
-
Javier Fernández García-Boente authored
During the HandleOverflow we consider the possibility of breaking at previous opportunity. The logic for this case is implemented in the BreakTextAtPreviousBreakOpportunity function. In case that there is such suitable previous opportunity, we modify the ItemResult that includes this opportunity in its offset range. We also create a new ShapeResultView associated to this item_result, using the new offset range. However, it's possible that the trailing_collapsible_space_ structure already has this ItemResult instance, that we are now altering as part of the HandleOverflow logic. The problem described in the issue 1131470 is caused by this situation, where an ItemResult instance hold by the trailing_collapsible_space_ is modified. During the computation of the trailing spaces size, as part of the RemoveTrailingCollapsibleSpace function, we avoid calling to the TruncateLineEndResult, which will reshape again, in case the item_result considered is already present in the trailing_collapsible_space_ structure. This causes that we end up with an ItemREsult of a different size than the previously computed collapsed_shape_result. Bug: 1131470 Change-Id: I15b31303fb8ec52b6d7359880f2785d16e1a0c59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429411 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#811115}
-
danakj authored
The Convert*ToDIP() and Convert*ToPixel() functions are problematic in that they hide what decision is being taken in the underlying code when moving from floating point to integer, and consumers should be thinking about what they want for correctness. They also act in inconsistent ways when converting a Rect vs its components. This removes the functions that perform float->int conversions in order to have callers show what they intend to happen and make it clear that data is being lost in some fashion. R=sky@chromium.org Bug: 1130050 Change-Id: If1297043ae0ac4278b04885d6f908f1178415ede Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419534Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#811114}
-
Shakti Sahu authored
This CL moves all the java code to internal/. VideoIPHCoordinator, and VideoPlayerCoordinator were extracted to interfaces. VideoTutorialServiceFactory is the factory responsible for providing implementation for all public interfaces. Bug: 1131959 Change-Id: I3f5b0383edfef902d12e3a18c50fec2787897fdc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429423 Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
-
Shakti Sahu authored
This is a reland of 278cb45f Original change's description: > Video Tutorials: Added video player UI > > This CL adds the video player UI. > 1 - Added VideoPlayerActivity at chrome layer, which creates > VideoPlayerCoordinator and passes a WebContent supplier in order to > create ThinWebView. > 2 - Added video player MVC, the mediator handles display logic between > loading screen, controls, thinwebview, and language picker. > It also handles callbacks for control buttons, and observes media > session through a PlaybackStateObserver. > > TODO : > Add screenshot. Add UI polish. Define share/watchnext behavior. > Polish media session logic. > > Bug: 1117172 > Binary-Size: Unavoidable method count increase (includes methods from previous CL). > Change-Id: Ide7c0d7bf4fa20a9bfbe0c10717e6983d0da88dc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2415672 > Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> > Reviewed-by: David Trainor <dtrainor@chromium.org> > Cr-Commit-Position: refs/heads/master@{#811094} TBR=dtrainor@chromium.org Bug: 1117172 Change-Id: If6c7fd26ba1643ded722eb274f876dad085dd075 Binary-Size: Unavoidable method count increase (includes methods from previous CL). Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434560Reviewed-by:
Shakti Sahu <shaktisahu@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#811112}
-
Etienne Bergeron authored
The test TtsSsmlBrowserTest.TestStripSSML but it seems to work fine. I suspect the memory leak got fixed by: https://bugs.chromium.org/p/chromium/issues/detail?id=961029#c6 Re-enabling the test. If this issue come back, please revert. Bug: 961029 Change-Id: Ia896498bcab1298f8afaa6fceb090eabac42c890 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432414 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#811111}
-
Johann authored
Tests fail under asan/lsan: ./out/chromeos_asan_lsan/chromeos_unittests \ --gtest_filter="*MultiDeviceSetupWifiSync*" deterministic failures [caused step to fail]: All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.Failure/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.Failure/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.FailureCallback_SyncedHostBecomesUnverified/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.FailureCallback_SyncedHostBecomesUnverified/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.MultiplePendingRequests_EnableDisable/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.MultiplePendingRequests_EnableDisable/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.MultipleRequests_FirstFail_ThenSucceed/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.MultipleRequests_FirstFail_ThenSucceed/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.PendingRequest_NoSyncedHostDevice/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.PendingRequest_NoSyncedHostDevice/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.Retrying_SyncedHostBecomesUnverified/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.Retrying_SyncedHostBecomesUnverified/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.SimultaneousRequests_StartOff_ToggleOnOff/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.SimultaneousRequests_StartOff_ToggleOnOff/1 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.Success/0 All/MultiDeviceSetupWifiSyncFeatureManagerImplTest.Success/1 BUG=chromium:1117619 TBR=cvandermerwe@google.com, khorimoto@chromium.org, jonmann@chromium.org Change-Id: Iad6b03ac9448371e8757a4d823a47bd74421bd18 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434011Reviewed-by:
Johann Koenig <johannkoenig@google.com> Commit-Queue: Johann Koenig <johannkoenig@google.com> Cr-Commit-Position: refs/heads/master@{#811110}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 944f0d0b. With Chromium commits locally applied on WPT: 74e4b2fc "[COOP] access reporting: Use infinite timeout for receive." 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: Idccab5c3db88a1c5cee086ee92dedcff52f3f510 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434626Reviewed-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@{#811109}
-
Ahmed Mehfooz authored
Consolidate pin logic into HoldingSpaceItemView since it will now be common between screenshots and chip views. Bug: 1131262 Change-Id: I600f5e7bb78726fe4b07afd379dd77857cd6bef2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429533 Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Reviewed-by:
David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#811108}
-
Noel Gordon authored
Rename ImagePreviewResponseData => PiexPreviewImageData. Change raw => RAW in comments. Mark private ImageBuffer function @private and rename the this.details member => this.details_ for consistency. Add @param to onAbort error, use !PiexWasmImageResult, cast the Module type !PiexWasmModule. No change in behavior, no new tests. Bug: 1132695 Change-Id: Ia20783dc000b7a3d40537e3811aba6f6f6f6a7c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2433745 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
-
Noel Gordon authored
piex-wasm assumes a fileSystemURL input as the RAW image source. Allow other users to input the RAW image source in an ArrayBuffer. Bug: 1132695 Change-Id: I47bcb837d42a8ca96ccdc79e306e012e0a3a82c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2433744Reviewed-by:
-
Javier Fernández García-Boente authored
In order to create a new ShapeResultView, we use segments of a previous instance, analyzing its runs based on a new index range for the new instance. Since the start/end of the RunInfo structure may be different than the start index of the ShapeResultView, we use an adjusted_start to adapt the logic to the different data structures. We assume that if the passed start_index for the new range is greater than the current run's part start being processed, we should define the new run's start as the difference of these two values. However, we must also ensure that start_index is at least equal to the the part_start value for such run. The problem described in the issue 1130153 comes from the fact that we use the start_index on cases where it's smaller than the run's part_start. This leads to an incorrect (longer than expected) run. Bug: 1130153 Change-Id: Id5dd505baf1f990d2fe7a9406e191979cbd6b9f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2431523Reviewed-by:
-
Ahmed Mehfooz authored
This class will own pinned_files_container and recent_files_container. It will also be responsible for setting the max size for the pinned files container based on available screen space and update it when screen space changes. The reland fixes a bug caused by removing GetBubbleView override from holding_space_tray that broke browser tests. Bug: 1131268 Change-Id: Ia0bf21261bb8f3b96f37366b04e1519dbe85c7a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432367 Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#811104}
-
Anupam Snigdha authored
This patch adds a restriction in |HighestEnclosingNodeOfType| to not cross the enclosing block. This is useful so we can get the "outer" block node without crossing block boundaries as that function only breaks when the loop hits the editable boundary or the parent element has an inline style(as we pass |IsInline| to it). This is useful in cases like <div contenteditable><i><table><tr> <td>Please |put cursor here</td></tr></table></i></div> where | is the selection position. When execCommand("insertUnorderedList") is called the markup ends up as shown below: <div contenteditable><i><table><tr><td><ul><li> |Please put cursor here</li></ul></td></tr></table></i></div>. Test: run_web_tests web_tests/editing/deleting/listify-paragraph-inside-table-cell.html Bug: 1016530 Change-Id: I51f19b35e69fcf57f2c583e6a521e6460e7cb305 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2433025Reviewed-by: -
Kent Tamura authored
- Remove non-existent source files - Add missing dependencies - Move browser_interface_broker_proxy.h and thread_safe_browser_interface_broker_proxy.h from public:blink_headers to public/common:headers These files are included by common:common, however we can't add public:blink_headers to common:common's deps because of a circular dependency. This CL has no behavior changes. Bug: 800764 Change-Id: I61cc0c08978ed0594cb0a0f7b395721978a36a97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434009 Auto-Submit: Kent Tamura <tkent@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#811102}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: I2150bca28954626cc264132084a02bcada980f57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434726Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#811101}
-
Johann Koenig authored
This reverts commit ac56b3e7. Reason for revert: PhoneHubUiControllerTest.NotEligibleForFeature fails on msan builds: $ autoninja -C out/chromeos_msan ash_unittests && ./out/chromeos_msan/ash_unittests --gtest_filter="*PhoneHubUiControllerTest*" [==========] Running 9 tests from 1 test suite. [----------] Global test environment set-up. [----------] 9 tests from PhoneHubUiControllerTest [ RUN ] PhoneHubUiControllerTest.NotEligibleForFeature ==1672876==WARNING: MemorySanitizer: use-of-uninitialized-value https://ci.chromium.org/p/chromium/builders/ci/Linux%20ChromiumOS%20MSan%20Tests/20740 PhoneHubUiControllerTest.BluetoothOff PhoneHubUiControllerTest.NotEligibleForFeature PhoneHubUiControllerTest.OnboardingNotEligible PhoneHubUiControllerTest.PhoneConnected PhoneHubUiControllerTest.PhoneConnecting PhoneHubUiControllerTest.PhoneConnectingForOnboarding PhoneHubUiControllerTest.PhoneDisconnected PhoneHubUiControllerTest.ShowOnboardingUi_WithPhone PhoneHubUiControllerTest.ShowOnboardingUi_WithoutPhone Original change's description: > [CrOS PhoneHub] Add PhoneHubUiController. > > This class translates the PhoneHubManager state into the corresponding > UI state and view that is shown in the tray bubble. Each state has > it's own view class and the PhoneHubTray bubble transitions through > these content views as the UI state changes. > > This CL also refactors PhoneConnectedView into it's own class. > > BUG=1106937,1126208 > > Change-Id: Ie012077a1905f80be91fbfb0b084d89219bd6c26 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2427060 > Commit-Queue: Tim Song <tengs@chromium.org> > Reviewed-by: Kyle Horimoto <khorimoto@chromium.org> > Cr-Commit-Position: refs/heads/master@{#810942} TBR=khorimoto@chromium.org,tengs@chromium.org,leandre@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1106937 Bug: 1126208 Change-Id: Ibd11174488010a7b210896d8f16943cfba90f918 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434010Reviewed-by:
-
Findit authored
This reverts commit 278cb45f. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 811094 as the culprit for failures in the build cycles as shown on: https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzI3OGNiNDVmZDcxNzBjMGYzYWZhMGQ4ZGExYzRjYzRmOWU0YzkzZTgM Sample Failed Build: https://ci.chromium.org/b/8867949354369276768 Sample Failed Step: compile Original change's description: > Video Tutorials: Added video player UI > > This CL adds the video player UI. > 1 - Added VideoPlayerActivity at chrome layer, which creates > VideoPlayerCoordinator and passes a WebContent supplier in order to > create ThinWebView. > 2 - Added video player MVC, the mediator handles display logic between > loading screen, controls, thinwebview, and language picker. > It also handles callbacks for control buttons, and observes media > session through a PlaybackStateObserver. > > TODO : > Add screenshot. Add UI polish. Define share/watchnext behavior. > Polish media session logic. > > Bug: 1117172 > Binary-Size: Unavoidable method count increase (includes methods from previous CL). > Change-Id: Ide7c0d7bf4fa20a9bfbe0c10717e6983d0da88dc > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2415672 > Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> > Reviewed-by: David Trainor <dtrainor@chromium.org> > Cr-Commit-Position: refs/heads/master@{#811094} Change-Id: I8de44b83196ee7ac90090f2c6d765a9aa3429430 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1117172 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434557 Cr-Commit-Position: refs/heads/master@{#811099}
-
tby authored
This replaces the placeholder request with one that correctly identifies ourselves, uses the right scenario, and limits the response details. Two notes: - There's no need for us to control any part of this request via Finch, because we can change what model the scenario maps to on the backend. - I've increased the number of requested results to 10, in case some are unavailable at display time. Bug: 1034842 Change-Id: Ibf37b912e0a2f115ba2bdb13c866b25e24e6e620 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434007Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/90a7c922079a..296dd3b4062e 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 johannkoenig@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:1099917 Tbr: johannkoenig@google.com Change-Id: I8601986a34d1d9da60f57c07e3ff5b80fac45d25 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2434687Reviewed-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@{#811097}
-
