- 19 Aug, 2020 40 commits
-
-
Brian Malcolm authored
This is intended to replace the NativePrintersBulkBlacklist policy. The old policy will still be used in the absence of the new one for at least a few releases of Chrome. BUG=chromium:1098049 Change-Id: I473dbe34e2d4de7ffcd91b5254b6414a4364357d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2341218Reviewed-by:
Sean Kau <skau@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Brian Malcolm <bmalcolm@chromium.org> Cr-Commit-Position: refs/heads/master@{#799494}
-
Chromium WPT Sync authored
Using wpt-import in Chromium de77910b. 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: Ia5fdd50bb49f0b2c48f88279cadab7e41aa16067 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362631Reviewed-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@{#799493}
-
Hajime Hoshi authored
TBR=jkrcal@chromium.org Bug: 1021930 Change-Id: I3d542632c89a5834d5339a6a9d488dfb8d62eba6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362363Reviewed-by:
Hajime Hoshi <hajimehoshi@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#799492}
-
Hajime Hoshi authored
This reverts commit c8d093ac. Reason for revert: Test failures on ChromeOS https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-chrome/7829 Original change's description: > ambient: Use different backend servers > > This patch configs the dev channel to use testing server. But users can > overwrite it with a flag. > > Bug: b/148692733 > Test: manual > Change-Id: I967281bc1421c37d9c1ff74a0815d6a6da570da3 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342134 > Commit-Queue: Tao Wu <wutao@chromium.org> > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org> > Cr-Commit-Position: refs/heads/master@{#799318} TBR=xiyuan@chromium.org,xiaohuic@chromium.org,wutao@chromium.org Change-Id: Ic364ef7c1f701b7a36b6503e8d330995de5dcecb No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b/148692733 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363508Reviewed-by:
-
Andrew Grieve authored
When chrome_public_smoke_test has a crash, our test runner does not notice and so does not collect tombstones. This changes the "are there tombstones" logic to look for a tombstone message in logcat rather than looking to see if any tests crashed. Bug: 1117540 Change-Id: Ic0594f0d8dcb2cb8c7280ade9d8d5133c890614f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363075Reviewed-by:
Ben Pastene <bpastene@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#799490}
-
Richard Li authored
Although Intel's driver supports RGB10A2 overlay format, driver problems still exist in checking overlay's capability, which blocks playing hdr video through overlay. This CL adds a workaround for Intel's hardware (Icelake or later), to ensure play hdr video through RGB10A2 overlay. In future, this workaround could be remove if overlay capability checking bugs are fixed in Intel's driver. Bug: 1062184 Change-Id: If18383edb0339c3c7701febdb34bdd7d736f323a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2340414Reviewed-by:
kylechar <kylechar@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Commit-Queue: Richard Li <richard.li@intel.com> Cr-Commit-Position: refs/heads/master@{#799489}
-
Xiaodan Zhu authored
Bug: 1110661 Change-Id: Ie74c57d6221407e8583790ad9e1108bcb3875e5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363298 Commit-Queue: Xiaodan Zhu <zxdan@chromium.org> Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#799488}
-
Daniel Hosseinian authored
Add a deprecation warning message for Cloud and Privet printers under the destination select. Note: This change is only for non-CrOS desktop. CrOS changes will be applied in a chained CL. Screenshot of change: https://imgur.com/cZqYwba Bug: 1112581 Change-Id: I2057cab0daa8d6f169ae6255172970a84b2803f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2353731 Commit-Queue: Daniel Hosseinian <dhoss@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#799487}
-
Yuki Shiino authored
Makes V8ContextSnapshotImpl not depend on unnecessary code and data if !defined(USE_V8_CONTEXT_SNAPSHOT). Bug: 839389 Change-Id: I349292e52b465e5836e8ade7791be24a1e39af78 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362066Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#799486}
-
rbpotter authored
Adding closure compilation for: link_container_test media_size_settings_test margins_settings_test Bug: 1000989 Change-Id: Ie7b2a06e157591e3a27eec9d46933ae8853237ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360502 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
John Lee <johntlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#799485}
-
Hirokazu Honda authored
Bug: 1060775 Test: appr.tc/?vsbr=500&vsc=vp9 on rammus Test: video_encode_accelerator_tests --codec=vp9 on rammus Change-Id: I45ef96aec373b05383680c54245fc0d0000b7579 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2359816 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#799484}
-
Sky Malice authored
The old handling of exitFirstRun() simply relaunched the pending intent. This turned out to be a problem because the intent dispatcher read the durable settings for if the FRE was needed, and would decide it was. Interestingly, there can only be one FirstRunActivity active at a time, and so the relaunch was actually ignored. This would cause the existing FirstRunActivity get stuck because the replacement activity was never started. This fixes the problem by introducing a new "ephemeral" static variable that remembers we just decided to skip the FRE, so the next intent dispatch/routing will be aware of this decision. Bug: 1106987 Change-Id: Id8502956236f76c93091452fe580ae0d6fea2c80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2343553Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Wenyu Fu <wenyufu@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#799483}
-
David Maunder authored
To assist in the investigation of how uninitialized Tabs are appearing in getDomain() Bug: 1116613 Change-Id: Ia0ab0d7aca20d724fd1200b1fdbe1a2f951f9633 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363485 Commit-Queue: David Maunder <davidjm@chromium.org> Reviewed-by:
-
Daniel Hosseinian authored
Slightly clean up the CrOS destination status code by using a computed property to set |statusText_|. This allows the removal of the observer on |destination| and manual calls of updateStatusText_(). Additionally, remove the stale shouldShowStatus_() function. Change-Id: I50011717b9643f137fdd67f12c580adcae289776 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2357070 Commit-Queue: Daniel Hosseinian <dhoss@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/573931528077..5b4f6e3193cf 2020-08-19 jmadill@chromium.org Vulkan: Add overlay widget for RP buffer count. 2020-08-18 jmadill@chromium.org Feedback Loop Redesign 1/3: Framebuffer Serials. 2020-08-18 lehoangq@gmail.com Metal: autogen format caps from json 2020-08-18 jmadill@chromium.org Vulkan: Add DS ReadOnly mode to Framebuffer/RP caches. 2020-08-18 lehoangq@gmail.com Metal: Fix missing image view for texture from IOSurface. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC cnorthrop@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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: cnorthrop@google.com Change-Id: If969d7b027c7a322ddc7d3fe0a1148745c8886e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363498Reviewed-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@{#799480}
-
Fan Yang authored
R=dschinazi@chromium.org Change-Id: I954d49604fef3be1b7a50205db5d08441f4226a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360232 Commit-Queue: David Schinazi <dschinazi@chromium.org> Reviewed-by:
David Schinazi <dschinazi@chromium.org> Cr-Commit-Position: refs/heads/master@{#799479}
-
Kent Tamura authored
According to the HTML standard, 'align' attribute for a LEGEND element should be mapped to justify-self property. Our layout code handles text-align for LEGEND, and doesn't handle justify-self. We'd like to investigate possibility to change the behavior. This CL has no behavior changes. Bug: 880822 Change-Id: I86427f172d7ee317d13f60fa78d5478135543174 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362487 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#799478}
-
Hirokazu Honda authored
The commas are forgot in the kVaapiFunctionNames initialization. Thus it initializes the first element and the second or later element is nullptr. SIGSEGV is caused in attempting to log the name because the char's pointer is nullptr. Bug: b:162962069 Test: video_encode_accelerator_tests on atlas Change-Id: Ifbb081b8e92350df5b83510087a042b7213bc377 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362378 Auto-Submit: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
Andres Calderon Jaramillo <andrescj@chromium.org> Reviewed-by:
-
Hajime Hoshi authored
TBR=sandersd@chromium.org Bug: 1117613 Change-Id: If6ea1284f6733f490f8510c73f09040dbaac0605 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362069Reviewed-by:
-
Xinghui Lu authored
Currently HelpAndFeedback is not modularized, so a modularized fragment is not able to use it due to circular dependency. In this CL, create a feedback module and introduce two interfaces as a workaround: * HelpAndFeedbackLauncher, which is used to launch a help and feedback page in an activity. HelpAndFeedback implements this interface. * FragmentHelpAndFeedbackLauncher, which is used to inject HelpAndFeedbackLauncher to a fragment. Any fragment that needs to launch a help page implements this interface. The safe_browsing module will depend on this new module to launch a help page. Bug: 1115758,1117343 Change-Id: I3d9d76f0ba87db26ae5bbe853f7b6cc38eb9353a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2359749 Commit-Queue: Xinghui Lu <xinghuilu@chromium.org> Reviewed-by:
Natalie Chouinard <chouinard@chromium.org> Cr-Commit-Position: refs/heads/master@{#799475}
-
Sinan Sahin authored
This CL renames default_text_color_tertiary to default_text_color_disabled and replaces any uses that aren't disabled with default_text_color_secondary. It also updates TextAppearance.*.Tertiary and its uses accordingly. Bug: 1097357 Change-Id: Ib14fdfbaa3941f4c4ab3aaa6ee8175d25758a565 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360559 Commit-Queue: Sinan Sahin <sinansahin@google.com> Reviewed-by:
Lijin Shen <lazzzis@google.com> Reviewed-by:
-
Daniel Hosseinian authored
This is a reland of crrev.com/799230, which was rolled back due to a mismatch between the OS_CHROMEOS macro and the |isChromeOS| WebUI constant on Lacros. (crbug.com/1111985). Use grit preprocessing instead. Add a deprecation warning message for the "Save to Drive" destination under the destination select. Screenshot of change: https://imgur.com/ix1GjIB Bug: 1112581 Change-Id: Icf3caa35b0efce83f440f8c8583edbf96a209290 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364000Reviewed-by:
-
Noah Rose Ledesma authored
The global media controls UI displays the audio devices connected to the system. This change allows the UI to refresh the devices it shows as availability changes. Because of an existing bug in device update processing on linux (crbug.com/1112480) a new class for monitoring these changes was created. The implementation for linux will poll the system whereas the implementation for all other OSes will be notified of changes. Bug: 1113944 Change-Id: I0d285ff0486fc1fb1307b5d78a5a998dcde131ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2341181 Commit-Queue: Noah Rose Ledesma <noahrose@google.com> Reviewed-by:
Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#799472}
-
Hajime Hoshi authored
This reverts commit 132e1deb. Reason for revert: Android ASAN bot compilation failures: https://ci.chromium.org/p/chromium/builders/ci/Android%20ASAN%20%28dbg%29/26489 Original change's description: > Add -noverify to build commands to improve runtime (local builds only) > > The flag is guarded by treat_warnings_as_errors=false so that it > will apply only to local developer builds. > > Results from //tools/android/build_speed/benchmark.py base_java_sig: > With change: > base_java_sig: 68.0s avg (68.8s, 67.2s) > base_java_sig: 68.4s avg (68.6s, 68.2s) > > Without change: > base_java_sig: 69.4s avg (69.3s, 69.5s) > base_java_sig: 69.7s avg (69.7s, 69.7s) > > time out/Debug/bin/run_base_junit_tests: > With Change: > real 0m22.741s > real 0m23.543s > > Without change: > real 0m23.176s > real 0m24.328s > > Bug: 1117222 > Change-Id: I4bfd742165468bf9538e6fa0dc8f35daa9e0f23f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2313299 > Commit-Queue: Andrew Grieve <agrieve@chromium.org> > Auto-Submit: Andrew Grieve <agrieve@chromium.org> > Reviewed-by: Peter Wen <wnwen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#799285} TBR=wnwen@chromium.org,agrieve@chromium.org Change-Id: I82f411e5dab89c361a30c33e9103bd8561f19765 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1117222 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362493Reviewed-by:
-
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: I171b3d6aafc73c402f7b287cf0293363c72f9f98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363261Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#799470}
-
Renjie Tang authored
https://quiche.googlesource.com/quiche.git/+log/abf77f1b3891..58468984dc04 $ git log abf77f1b3..58468984d --date=short --no-merges --format='%ad %ae %s' 2020-08-18 renjietang Copy cached transport parameters and application states instead of using pointers in QUIC 0-RTT setup. 2020-08-18 quiche-dev CL generated by 'ykats' via Upkeep (go/upkeep). Resolves the following technical debt issues: Created with: roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src Bug: 1116526 Change-Id: Ia74cb3499997f24e8678290bd1d6474daaceec18 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363723 Auto-Submit: Renjie Tang <renjietang@chromium.org> Reviewed-by:
-
wutao authored
This patch adds strings to search for album page. Also changes some strings from ambient mode to screen saver based on UX writer. Bug: b/163421198, b/163417826 Test: manual Change-Id: I10759ef626ca3ec23e227ced1f687e0a1cc3fb1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2350334Reviewed-by:
Sam McNally <sammc@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Jimmy Gong <jimmyxgong@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: Tao Wu <wutao@chromium.org> Cr-Commit-Position: refs/heads/master@{#799468}
-
Alex Ilin authored
New colored avatars use the following colors: - Background color is equal to the frame color - Stroke color depends on whether the background color is dark: * If the background color is dark, the stroke color is white * Otherwise, the stroke color is computed from the background color by decreasing the lightness by 0.5 Bug: 1100835 Change-Id: I40d77064cf4baef339c931211eb12153e2f6e0c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362592 Commit-Queue: Alex Ilin <alexilin@chromium.org> Reviewed-by:David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#799467}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/5cff4e3b5cf3..25f1303ed83e 2020-08-19 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 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 apolito@google.com,ehmaldonado@google.com,sokcevic@google.com,ajp@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: apolito@google.com,ehmaldonado@google.com,sokcevic@google.com,ajp@google.com Change-Id: I8b4781c4e42ba0e56927cf41ccb8b9966895716d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364089Reviewed-by:
-
Hiroki Sato authored
Without this change, ChromeVoxBackgroundTest.NoFocusTalkBackEnabled was passing regardless of TalkBack toggled status. This is because MockFeedback runs all of the callbacks when replay() is called, and when utteranceInQueue is called in the test, any action has not been invoked yet. This change moves the utterance check after executing replay(). AX-Relnotes: n/a. Bug: none. Test: ChromeVoxBackgroundTest.NoFocusTalkBackEnabled, fails if TalkBack is disabled, and passes if TalkBack is enabled. Change-Id: Ide42b8811a7cb91d82d7dfb242ae318c4cf0c760 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362488 Commit-Queue: Hiroki Sato <hirokisato@chromium.org> Reviewed-by:
Akihiro Ota <akihiroota@chromium.org> Cr-Commit-Position: refs/heads/master@{#799465}
-
Thanh Nguyen authored
1. Create a white circle background vector icon for launcher chip. 2. Use vector icon for file launcher chips instead of pngs. 3. Remove resources that are no longer required. Bug: 1067326 Change-Id: I1a18e7c6c8458f84f50930ce47ac87dfcd94e1c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2354373 Commit-Queue: Thanh Nguyen <thanhdng@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#799464}
-
My Nguyen authored
Bug: 1117498 Change-Id: I09c348442df6a23143c1bbdadbf1f5ecbb1add0b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362239 Commit-Queue: My Nguyen <myy@chromium.org> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
-
Donn Denman authored
The server returns no pronunciation in some languages. This formats poorly - with a trailing dot. This is due to the formatting code relying on the mid-dot to determine if the card is a definition or not. Now we pass in the card type which is authoritative. BUG=1112792, 1117684 Change-Id: If9a4a4ddba9800a8a77212e1cf9d2d1296424dff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2358450 Commit-Queue: Donn Denman <donnd@chromium.org> Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Auto-Submit: Donn Denman <donnd@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Cr-Commit-Position: refs/heads/master@{#799462}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/59a3c3be94f0..81eabce6a3af 2020-08-19 bungeman@google.com SkCustomTypefaceBuilder to set SkFontStyle 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 stani@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: stani@google.com Change-Id: Ic089cd585906fa42e3a1ff0f947f2a62765795fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363969Reviewed-by:
-
Mike Wasserman authored
Use the active window-placement repo url and name instead. Leaves some mentions that are still abstractly applicable. Copy Screen TODO from third_party/blink/renderer/core/frame/screen.h Bug: 897300 Change-Id: I134516e21a74f0cec5f543306efa39fad6d76dc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2358134Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#799460}
-
Peter Kasting authored
* Make more things constexpr and/or make it possible to do so later * Inline SaturatedAdd/Sub(), FromDouble(), FromProduct() * Order the FromUnits{,D}() pairs together * Init members in declaration * Briefer implementations of various functions * IWYU * Fix declared-but-not-defined issue for FromTimeSpec() w/OS_FUSCHIA * Use more specific DCHECKs * No else after return * Omit needless qualifiers * EXPECT -> static_assert where possible * <atomic> is legal now * Don't handle DCHECK failure The inlines don't hurt size: this saves 4 KB off chrome.dll in my local release build. Bug: none TBR: stevenjb@chromium.org Change-Id: I269d6426ac1587569e7a4c785250a4dd5e95d5bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2358997Reviewed-by:Peter Kasting <pkasting@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#799459}
-
Ionel Popescu authored
This CL addresses an issue where the users where not able to enter data into the text fields of the color picker when the elements are inside a cross-domain iframe. For the top-level page and for same-domain iframes, when a char event is received while a popup is open, the event is routed from RenderWidgetInputHandler to WebViewFrameWidget shim, which hands it to the WebView. WebViewImpl::HandleCharEvent has code to check for an open popup and the event is sent to the popup if one exists. For the cross-domain iframe case, the RenderWidgetInputHandler hands the event to WebFrameWidgetImpl, but WebFrameWidgetImpl::HandleCharEvent doesn't check for an open popup, so the event goes to the page containing the popup. This CL adds the same popup check to WebFrameWidgetImpl as the one that WebViewImpl::HandleCharEvent already has. Bug: 1114052 Change-Id: I33f2b10b4ae5aa538ebb0d22c1d55f5d97019216 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364008Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Ionel Popescu <iopopesc@microsoft.com> Cr-Commit-Position: refs/heads/master@{#799458}
-
Yu Han authored
Intent to ship: https://groups.google.com/a/chromium.org/g/blink-dev/c/Jdw3VWbKvLY/m/0YBXYr-lBwAJ Bug: 869308 Change-Id: I75cab8d0c7c117ce67a8962a903b534e11620f95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2358608Reviewed-by:
Mason Freed <masonfreed@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#799457}
-
Sky Malice authored
This reverts commit 80207c25. Reason for revert: Flaking tests, see crbug.com/1117683 Original change's description: > Redesign ATrace integration > > Chrome and WebView have supported basic Android ATrace tracing, but the > current approach has some shortcomings: > > 1) Android R changes the way atrace session activations are broadcast, > breaking the code in WebView listening for activations > (OnTraceEnabledChangeListener). This means you can't trace a running > WebView app with atrace. > > 2) Neither Chrome nor WebView record early startup events to atrace, > which means all events before the native library has loaded are > lost. > > 3) It's not possible to specify trace categories via atrace, which means > we need more cumbersome alternative solutions (i.e., command line > flags) for startup tracing. > > 4) Writing ATrace events is only supported in WebView. > > This patch reworks the ATrace integration to resolve these problems and > to align the Chrome and WebView implementations. In short, ATrace > session management is moved to the common (TraceEvent) layer, and Chrome > and WebView only differ by which trace tags they listen to (APP vs. > WEBVIEW). > > We also add a way to declare trace categories through per-app tags: > > $ atrace -a org.chromium.chrome,org.chromium.chrome/<category_filter> > > (Note that the plain package name without any category filters must > always appear in the list on its own.) > > Multiple categories can be separated with a double colon: > > $ atrace -a org.chromium.chrome,org.chromium.chrome/cat1:cat2 > > Or by specifying the the app several times to get around the 91 > character limit for each entry: > > $ atrace -a org.chromium.chrome,\ > org.chromium.chrome/cat1,\ > org.chromium.chrome/cat2 > > Finally, to capture Java startup events into a tracing session > controlled by the system's Perfetto service instead of atrace, a > special "-atrace" category can be used to only write events into > Chrome's own tracing service instead of atrace. This way when we > connect to Perfetto later in the startup sequence and establish > the real tracing session, startup-related events can be flushed > into that session without emitting duplicate events into ATrace. > > # Basic trace while Chrome is running. Generates ATrace with Chrome > # events. > TEST=atrace -a org.chromium.chrome > > # Ditto for WebView. > TEST=atrace webview > > # Startup trace -- open Chrome after running this. Results in an ATrace > # with Chrome pre- and post-startup events. > TEST=atrace -a org.chromium.chrome > > # Ditto for WebView. > TEST=atrace webview > > # Trace with custom categories. Results in a ATrace with only > # "cc" events from Chrome. > TEST=atrace -a org.chromium.chrome,org.chromium.chrome/-*:cc > > # Ditto for WebView, using GMail as a test app. > TEST=atrace -a com.google.android.gm,com.google.android.gm/-*:cc webview > > # Chrome-only trace. No Chrome events written in the resulting ATrace. > TEST=atrace -a org.chromium.chrome,org.chromium.chrome/-atrace > > Bug: 1095587, b/160768681 > Change-Id: Ia0bee252ce398804c00be8a4179241b75479bf5e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332664 > Reviewed-by: Bo <boliu@chromium.org> > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Reviewed-by: Eric Seckler <eseckler@chromium.org> > Commit-Queue: Bo <boliu@chromium.org> > Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> > Cr-Commit-Position: refs/heads/master@{#798791} TBR=boliu@chromium.org,skyostil@chromium.org,agrieve@chromium.org,eseckler@chromium.org,nuskos@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1095587, 1117683 Bug: b/160768681 Change-Id: Iffcaebb70c6eb92da1cb3080e8034ea5c36e5394 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363564 Commit-Queue: Sky Malice <skym@chromium.org> Reviewed-by:
Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#799456}
-
Miguel Casas authored
This reverts commit 9881e6cd. Reason for revert: Needs a follow up crrev.com/c/2362068, but the code is still faulty because it'd hit a DCHECK. Let's revert this one and land a correct CL instead. Original change's description: > media/gpu/vaapi: use new |va_buffer_for_vpp_| for Vpp/BlitSurface() > > ToT VaapiWrapper uses |va_buffers_| for internal Vpp uses and also for > external VEA/JEA use. This CL starts refactoring these VABufferIDs > lifetime, by splitting the Vpp VABufferID out of |va_buffers_| and > into a ScopedID |va_buffer_for_vpp_|. > > Bug: b:162962069 > Change-Id: Ic54d0d26f5d17ea49fe896cf7ca5e1c931d07e0f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2352522 > Commit-Queue: Miguel Casas <mcasas@chromium.org> > Reviewed-by: Hirokazu Honda <hiroh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#799410} TBR=mcasas@chromium.org,hiroh@chromium.org Change-Id: I2b13e096fb7fce00c4dbbaed64bd4d231de34d15 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b:162962069 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363798Reviewed-by:
-
