- 10 Oct, 2020 40 commits
-
-
Rakina Zata Amni authored
If we've dispatched the 'pagehide' event with the 'persisted' property set to true, we should not dispatch the unload event after that. This is because the 'persisted' property is set to the 'salvageable' status of the page, which also determines whether the unload event is fired. If 'salvageable' is true, 'persisted' will be true and 'unload' won't be dispatched. If 'salvageable' is false, 'persisted' will be false and 'unload' should be dispatched. Relevant spec PR: https://github.com/whatwg/html/pull/5889 More context: https://groups.google.com/a/google.com/g/chrome-bfcache/c/L-ZreZDY4n0/m/jna_jQJkCQAJ Bug: 1110744 Change-Id: I54b44cfdcb6c2922ca57071d695df6c4c2d77d77 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2455510 Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#815898}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/294729f33cbf..55169d89c60e 2020-10-09 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 15eae87d..9186bb4a (815630:815743) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@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 Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: Ice1848139b435057aa8df0863279530f9b73fe0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463798Reviewed-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@{#815897}
-
chrome://flagsZhuoyu Qian authored
Has been hidden since M78, so time to remove. Bug: 1110589 Change-Id: Ic195337a541eadfd859022ed7cd49768303b552b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2418595Reviewed-by:
Naoki Fukino <fukino@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Zhuoyu Qian <zhuoyu.qian@samsung.com> Cr-Commit-Position: refs/heads/master@{#815896}
-
Dana Fried authored
Previously, the target tab preview size for the tablet (Mohnstrudel) tabstrip was determined by the size of the web viewport. However, the size of the viewport can be affected by the visibility of the bookmark bar, which is dependent on a number of factors including whether the user has opted to enable the bar, whether the current page is the NTP, whether the current page is crashed, etc. That resulted in a situation where the size of the thumbnails in the tabstrip could change as the user switched between tabs. This CL standardizes the thumbnail size to be based on the size of the viewport if the bookmark bar were completely visible, regardless of the current bookmark bar state. The result is that in the event the user has disabled the bookmark bar, the resulting thumbnail may be slightly shorter than it would otherwise be, which can result in a couple of pixels being trimmed off the right and left when the thumbnail is displayed. However, it's extremely hard to notice this slight reduction in fidelity compared to the benefit of having the thumbnail size constantly changing. Bug: 1066652 Change-Id: I05fc47ac6f99e1323ebe682a636151fb81360229 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462492 Auto-Submit: Dana Fried <dfried@chromium.org> Commit-Queue: Collin Baker <collinbaker@chromium.org> Reviewed-by:
Collin Baker <collinbaker@chromium.org> Cr-Commit-Position: refs/heads/master@{#815895}
-
Erik Chen authored
This CL has no intended behavior change. This is a prerequisite to passing base::Token as a stable UUID for interfaces. Bug: 1136652 Change-Id: Icab69978c7abc5e88fbb40bfb650fecf08c9e9d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461723 Auto-Submit: Erik Chen <erikchen@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Greg Kerr <kerrnel@chromium.org> Commit-Queue: Greg Kerr <kerrnel@chromium.org> Cr-Commit-Position: refs/heads/master@{#815894}
-
Matt Reynolds authored
content::HidService incorrectly decrements the active frame count when revoking a permission for a device with no open connections. This CL changes the behavior so the active frame count is only decremented if a connection was closed as a result of revoking the permission. BUG=1137031 Change-Id: I686d1e11193991c8e5ddeb7a90d50c77b848fe53 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463997 Commit-Queue: Matt Reynolds <mattreynolds@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Auto-Submit: Matt Reynolds <mattreynolds@chromium.org> Cr-Commit-Position: refs/heads/master@{#815893}
-
Shrek Shao authored
TBR=kbr@chromium.org Bug: 1135785 Change-Id: If06cf5b8d4d584acd23c768a645d9a23fc82bed9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464048Reviewed-by:
Shrek Shao <shrekshao@google.com> Commit-Queue: Shrek Shao <shrekshao@google.com> Cr-Commit-Position: refs/heads/master@{#815892}
-
chrome://safe-browsing/#tab-rt-lookupXinghui Lu authored
This flag was added for testing purpose. It is now misleading because RT Lookup can still be performed when this flag is false(e.g. for ESB users). The same information can also be checked in chrome://safe-browsing, so this flag is not needed anymore. Bug: 1069651 Change-Id: Ib96f9e456b6477bff77e18f9049309b5d75728b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2146795Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Xinghui Lu <xinghuilu@chromium.org> Cr-Commit-Position: refs/heads/master@{#815891}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/793a07e36636..1c25198384e7 2020-10-10 enrico.galli@intel.com Adding multiple of 4 validation on indirect draws and dispatches If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC kainino@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:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: None Tbr: kainino@google.com Change-Id: Ibee6c61f563c3375251eb8fabdee1dae7cb37d1d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463073Reviewed-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@{#815890}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/61003cde7688..13fc260c7080 2020-10-09 johnstiles@google.com Reject struct vardecls with modifiers. 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 jvanverth@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: jvanverth@google.com Change-Id: Ie67945dbfdf6fed2cbc30de18e804a57e5b5ee59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461133Reviewed-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@{#815889}
-
Ng Zhi An authored
There was a typo in arch64, the intention is aarch64, and the implementation uses ssat/usat, which is arm (32-bit) specific, and would not work for aarch64. So remove the check (and support) for aarch64. Change-Id: I9985c0c370781a8dcaccf55e2b6f182421848c97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440971 Commit-Queue: Zhi An Ng <zhin@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#815888}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/815983783a76..109c355900d1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I5af2117f10863c34c9f4e64ae982f9e345678bb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464088Reviewed-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@{#815887}
-
yiyix authored
After the cl, Merge ImageData::data_union into ImageData::data. Some code assume the data type is uint8 array and the size per pixel is 4. I left some TODOs in the code to address all the remaining problems. This cl addresses all issues in image_data and shape_detector. ref cl: https://chromium-review.googlesource.com/c/chromium/src/+/2379653 Bug: 1115317 Change-Id: If46ea1cc6986d8e2c1e4b40e8f98ecd55b31d9dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2437606 Commit-Queue: Yi Xu <yiyix@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Reviewed-by:
Aaron Krajeski <aaronhk@chromium.org> Reviewed-by:
ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#815886}
-
Rachel Wong authored
This CL does the following: - Remove the default focus action. This allows ChromeVox to immediately select and read out the notice when the search box is opened. - Make all of the text behave as one entity for ChromeVox, which can also activate it as the link. - Use the AppListColorProvider to get the correct shading color when the close button is focused. This ensures that the color will still contrast with the background in dark mode. Bug: 1112714 Change-Id: I798d532a030a0f331845ab440b8f6aabced46af0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463024 Commit-Queue: Rachel Wong <wrong@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#815885}
-
Ahmed Mehfooz authored
Modify mic slider view padding and overall detailed view padding to bring up to spec provided in the bug. Bug: 1086993 Change-Id: I2b8719811070234b9a06694053370bfb8c7eae82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464242Reviewed-by:
Tim Song <tengs@chromium.org> Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Cr-Commit-Position: refs/heads/master@{#815884}
-
Sammie Quon authored
Change the capture session widgets to be POPUP. If they are window, they need to be clicked first to gain capture. For example, press and drag of capture button. Then to close, you have to first click the bar and then click the exit button. This happens before this patch, but it is more noticeable with the inkdrop visuals. Change ShouldCaptureLabelEvent so that it can get events directed at it that have no location (ENTER, CAPTURELOST, EXIT). The capture button is now centered in the region if it can be with a minimum amount of padding on both sides. Otherwise it will go on either slightly above or below the region, based on its proximity to the root bottom edge. Test: ash_unittests CaptureModeTest.CaptureRegionCaptureButtonLocation Bug: 1131211 Change-Id: I732a1871120ffa9b9eee4fbadd5361fc9aec0ac0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2427865Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#815883}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/80d095c4dc41..d7d0b90f7d38 2020-10-09 ehmaldonado@google.com git-drover: Fix error printing. 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 ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@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 Bug: chromium:1135320 Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com Change-Id: I029f10f763dccce0abf4519e0971c4bf87c10d3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463796Reviewed-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@{#815882}
-
Leonid Baraz authored
Add ProcessGap entry point to Uploader, so that later on StorageQueue will be able to call it whenever there are no records in the queue that server expects to get, or if records are corrupt (e.g., file was deleted or altered). Currently StorageQueue enumerated records from the first it has and calls ProcessRecord for every one. If something goes wrong, it bails out. The plan is to change it like this in further CLs: 1) Separate last confirmed seq id from the first queue has. 2) If the last confirmed is lower, call ProcessGap for all those that are missing. 3) If anything is wrong, call ProcessGap for that one and switch to the next record regardless. ProcessRecord will add EncryptedRecord with data to the request to send. ProcessGap will add one or more Encrypted records with no data - that seems to be the easiest approach. Bug: b:169248924 Change-Id: I3cd600daf819a5ca8428c79788669be59f6d01dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463766 Commit-Queue: Leonid Baraz <lbaraz@chromium.org> Reviewed-by:
Zach Trudo <zatrudo@google.com> Cr-Commit-Position: refs/heads/master@{#815881}
-
Ahmed Mehfooz authored
Bug: 1136836 Change-Id: I36c98b23b714f36a09c7bce702bf9ef3c5347eba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464001Reviewed-by:
Tim Song <tengs@chromium.org> Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org> Cr-Commit-Position: refs/heads/master@{#815880}
-
Pavel Yatsuk authored
This CL: - Introduces MessageDispatcher and its implementation - Makes it available through WindowAndroid, following the pattern of BottomSheetController - Refactors MessageQueueManager removing it from public messages target BUG=1123947 R=lazzzis@chromium.org Change-Id: Ieeece06f4b2f6b31abfd9c6cfd8df5b428076f8d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2446195Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Pavel Yatsuk <pavely@chromium.org> Cr-Commit-Position: refs/heads/master@{#815879}
-
Akihiro Ota authored
Move method to a utilitiy file, since it doesn't need to be defined on the ChromeVox Next testing class. Bug: N/A Change-Id: I504be13fbaeb1012671102155b460d3cea256515 AX-Relnotes: N/A Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461947 Commit-Queue: Akihiro Ota <akihiroota@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#815878}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1602201537-a14eb3c04e9257cb13970f54aebda7b1af96d2f4.profdata to chrome-win32-master-1602255148-ebeb3da431f3690e2790a05d46a0bb31c66dafca.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC pgo-profile-sheriffs@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:win-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I0e259b7a52c42317d13d7bad6d40147ecc3e5ce1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463726Reviewed-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@{#815877}
-
Fabrice de Gans-Riberi authored
* Kill the Session after 10s of no activity in the audio or video stream. * Kill the Session after 5s if no offer message is received. * Fix reporting of messages from unknown namespaces in the Cast Streaming MessagePort. Bug: 1087528, 1087528 Change-Id: I69b44bea3b142e7cf12c32106a279ef37710c56b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2451525Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Cr-Commit-Position: refs/heads/master@{#815876}
-
Curt Clemens authored
ReceiveObserver originally had an OnIncomingTransfer method that was called when a transfer update occurred with status kAwaitingLocalConfirmation. Instead, all transfer updates are sent to the mojo interface to facilitate future work. Test: unit_tests, browser_tests Change-Id: I7f57db1b67ccb6fc9c3337419ca236a492d0c53f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2453914 Commit-Queue: Curt Clemens <cclem@google.com> Reviewed-by:
Alex Gough <ajgo@chromium.org> Reviewed-by:
James Vecore <vecore@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#815875}
-
George Burgess IV authored
This new clang-tidy check complains when param names /*in_comments=*/ don't match up with the actual parameter name they're being passed to. Sometimes this is benign, other times, actual bugs are found. Bug: 1136209 Change-Id: I06a90eb08ee7445e11f30409420ea5e1bef50aed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2459266Reviewed-by:
Nathan Parker <nparker@chromium.org> Commit-Queue: George Burgess <gbiv@chromium.org> Cr-Commit-Position: refs/heads/master@{#815874}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/6dc612511e7b..cdf1ab1f8288 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 lukasza@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:1093386 Tbr: lukasza@google.com Change-Id: I72bf3937791d0fddc635fa9329683e983a81dd6c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463792Reviewed-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@{#815873}
-
Bo Liu authored
Add an API that allows client to declare the action mode items it wants to override. When overridden, the selected text is passed to the client and whatever the default action in weblayer does not happen. Currently only support search and share since they are always enabled. Bug: b/159070012 Change-Id: Ibabc253a9d990fbf4c1746fd402068df4aeafff2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463989Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#815872}
-
Lukasz Anforowicz authored
ExtensionURLLoaderFactory's and CastExtensionURLLoaderFactory's lifetime cannot exceed the lifetime of the associated BrowserContext. This CL should fix https://crbug.com/1136214 and is likely to also fix https://crbug.com/1134050. Fixed: 1136214 Bug: 1134050 Change-Id: I06b0c17ebe308ec86f0341c7fa7b7bbdc1ffde4c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461222 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Sean Topping <seantopping@chromium.org> Cr-Commit-Position: refs/heads/master@{#815871}
-
Devlin Cronin authored
There are two main ctors for URLPattern - one takes valid schemes and is used with the Parse() method (which handles failure) - one that takes schemes and a string pattern, and assumes parsing must succeed The latter currently has a NOTREACHED() and an error log. Upgrade this to a DCHECK, since it should never, ever be hit (especially following the identifying case for crbug.com/856948). This should - hopefully - help ensure that no new cases are introduced, and flush out any other cases that may exist. Bug: 856948 Change-Id: I1982d10e92c72d1d8ca6da03644072a73a0c7580 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2450707Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#815870}
-
Shannon Chen authored
Enable point scanning using the point scanning button in the Switch Access menu. Signed-off-by:
Shannon Chen <shannc@chromium.org> Bug: 1061537 Change-Id: I5cdc94dec29870fbda276f01ff77124442922393 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461721Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Shannon Chen <shannc@google.com> Cr-Commit-Position: refs/heads/master@{#815869}
-
Kyle Horimoto authored
This KeyedService is responsible for generating dependencies to provide to the Nearby Connections utility process. Code is based off of the existing nearby_process_manager.cc file; the only changes are that we only fetch the Bluetooth adapter once and that we re-organize this code to be a KeyedService since it is based on the Profile. Bug: 1130069 Change-Id: I9c1efd2ff93e9a9aadba5c363d0af37097ab5e49 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454388 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Auto-Submit: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
James Vecore <vecore@google.com> Cr-Commit-Position: refs/heads/master@{#815868}
-
Daniel Classon authored
Adds a metric for the kAddAccount setting on the People Page. It records the number of the account the user was trying to add, e.g. a sample of 3 on the histogram means the user attempted to add a 3rd account. Bug: 1133553 Change-Id: I8befd66ce2ce78f1c07e6c5afb4f4a5fcb8ae09c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2454492 Commit-Queue: Daniel Classon <dclasson@google.com> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#815867}
-
Lukasz Anforowicz authored
I looked at other js-based tests and couldn't find a way to disable only on a single platform (it seems that the flakiness is specific to ChromeOS). Note that most of the changes in this CL comes from |git cl format --js chrome/browser/resources|. Bug: 1136991 Change-Id: Ib292732483b5f1d88f0a5b919c66eb484646f31b Tbr: anastasi@chromium.org Tbr: dtseng@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463928Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#815866}
-
Moe Ahmadi authored
Fixed: 1136649 Change-Id: Ie48f7b52c40fd040a23b16c63ec0bb970c8585b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463804 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#815865}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/daef088d069f..e21911cc1c77 2020-10-09 tsepez@chromium.org Make _TIFFBuiltinCODECS be const data. 2020-10-09 thestig@chromium.org Simplify struct TbConvertAdditional. 2020-10-09 thestig@chromium.org Avoid relocations in struct BarCodeInfo. 2020-10-09 tsepez@chromium.org Add pdfium::span::front() and back() 2020-10-09 tsepez@chromium.org Move CFXA_MapModule to CFXJSE_MapModule and add tests 2020-10-09 nigi@chromium.org Disable some Windows-only embedder tests which fail with Skia/SkiaPaths. 2020-10-09 nigi@chromium.org Update Skia/SkiaPaths embedder tests' macOS expected results. 2020-10-08 tsepez@chromium.org Move XFA_ATTRIBUTE_CALLBACK from fxjs/ to xfa/ directory 2020-10-08 tsepez@chromium.org Add some missing const in CFXJSE_Context 2020-10-08 tsepez@chromium.org Completely re-implement CFXA_MapModule data storage 2020-10-08 thestig@chromium.org Make more XFA attribute data truly read-only. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pdfium-autoroll Please CC pdfium-deps-rolls@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 Bug: None Tbr: pdfium-deps-rolls@chromium.org Change-Id: I49f4770debd38906e694eab2014de1aab0e63575 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463791Reviewed-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@{#815864}
-
Fabrice de Gans-Riberi authored
This leaves the fuchsia-namespaced versions of the constants and function of the call for out-of-tree callers. Bug: 1073821 Change-Id: I305fe5b84a00656fe83a5da1d92ce782e65cbd7d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2429810 Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#815863}
-
Sunny Sachanandani authored
Test suppressions for tests added in https://crrev.com/c/2432207 that fail because of existing bugs. TBR=kainino@chromium.org Bug: 1132381, 1132375 Change-Id: I10bb564ae4729a199bcea80c444d8b1b7f8e2e14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463805Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Cr-Commit-Position: refs/heads/master@{#815862}
-
Chan Li authored
Revert "Reland "Re-enable `chromium.resultdb.result_sink` experiment on chromium fyi builders at 30%"" This reverts commit 99db6496. Reason for revert: Found 2 more issues, disable the experiment for now. Original change's description: > Reland "Re-enable `chromium.resultdb.result_sink` experiment on chromium fyi builders at 30%" > > This is a reland of 6bcf1a99 > > Original change's description: > > Re-enable `chromium.resultdb.result_sink` experiment on chromium fyi builders at 30% > > > > Bug: 1108016 > > Change-Id: I90beaf78f9f4b8508f6229b17b07a44943f442d6 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462049 > > Reviewed-by: Takuto Ikuta <tikuta@chromium.org> > > Commit-Queue: Chan Li <chanli@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#815450} > > Bug: 1108016 > Change-Id: I257a1f13a7cefe4c95e0a93d9ba87b57b19c716f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463102 > Auto-Submit: Chan Li <chanli@chromium.org> > Reviewed-by: Takuto Ikuta <tikuta@chromium.org> > Commit-Queue: Chan Li <chanli@chromium.org> > Cr-Commit-Position: refs/heads/master@{#815483} TBR=chanli@chromium.org,tikuta@chromium.org Change-Id: I435ca3e7d459cdfe0ae4edbc29acb671570632f5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1108016 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464239Reviewed-by:
Chan Li <chanli@chromium.org> Commit-Queue: Chan Li <chanli@chromium.org> Cr-Commit-Position: refs/heads/master@{#815861}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/e9879d106b95..cd2eebd327e3 2020-10-09 ericlaw@microsoft.com NetLogViewer: Show Capture Mode in Import view If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC sadrul@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:chromeos-kevin-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:1057773 Tbr: sadrul@google.com Change-Id: Ic0fdabc37a77b2970e655e4b59fcad34c0b8c174 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463790Reviewed-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@{#815860}
-
Shakti Sahu authored
This test was disabled three years back due to flakiness of ChromeActivityTestRule.loadUrl. Since then a lot has changed, and possibly this test isn't flaky any more since tons of test use that method. Reenabling this test and will keep watching if flakiness returns. Bug: 786237 Change-Id: I9601394c0fb88617d252e61cf3d8d9d0f8b4da3a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463571Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#815859}
-