- 07 Nov, 2019 40 commits
-
-
David Bokan authored
This flag is useful both for debugging and accessibility. While we wait for an accessibility UI we shouldn't remove the flag. Bug: 817814 Change-Id: I85c8dbf2479f3ea36552d6b00794695f81284858 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898331Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#713522}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/62a814392cf5..50d2ff30828b git log 62a814392cf5..50d2ff30828b --date=short --no-merges --format='%ad %ae %s' 2019-11-07 fmayer@google.com Merge "Add disabled test-case for incorrect unique_retained." 2019-11-07 treehugger-gerrit@google.com Merge "Move CopyablePtr from base to protozero" Created with: gclient setdep -r src/third_party/perfetto@50d2ff30828b If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: None Change-Id: I681b87f4726d9cbddbcd55e1e2d4b8161789f82a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903655Reviewed-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@{#713521}
-
Shawn Gallea authored
This is needed to notify cast apps when they are covered by other activities. Bug: b/120809866 Test: Compile Change-Id: I312117a7b051183ad0056cfeff2d0edff49a6f82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900388Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Reviewed-by:
Sean Topping <seantopping@chromium.org> Commit-Queue: Shawn Gallea <sagallea@google.com> Cr-Commit-Position: refs/heads/master@{#713520}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/252d1eb47f23..c3284fa40ec6 git log 252d1eb47f23..c3284fa40ec6 --date=short --no-merges --format='%ad %ae %s' 2019-11-07 rharrison@chromium.org Migrate away from using recently deprecated APIs Created with: gclient setdep -r src/third_party/dawn@c3284fa40ec6 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 cwallez@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try: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 TBR=cwallez@google.com Bug: None Change-Id: I14860f2313eaef07f738d5113d6260ae48fafaeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904333Reviewed-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@{#713519}
-
Kurt Catti-Schmidt authored
This is a reland of cc2f9f85 There are two issues that occurred once the reland was checked in yesterday, which this "reland reland" addresses. First, https://chromium-review.googlesource.com/c/chromium/src/+/1872409 updated the AuraLinux dumps to output "[static]" instead of "[text]" for text nodes. Since this was a new test file, it was missed in the update and started reliably failing once that change went in. This was addressed by updating the expected dump file for Linux. Additionally, Windows 7 outputs a different date format, with commas in between the month and year. This is addressed by adding DENY filters for the name attribute on Windows on the node where formatting differs, as well as removing all IA2 Hypertext from the tree, which isn't relevant to this test either. TBR=sky@chromium.org TBR=nektar@chromium.org Original change's description: > Reland "Fix UIA Fragment Root Navigation in Inspect" > > This is a reland of 5e7f2837 > > This was reverted due to flaky tests. I was not able to reproduce the > failures, but I was able to see via the logs that the failed cases > were opening the first popup instead of the third. As a mitigation, I > have switched the logic to find the third popup. Previously, I had > added a count parameter to FindNode, so it would find the third node > with that name. This change adds a name to the parent node (in this > case the <input> node), and updated the logic to search for that node > as a parent. > > Original change's description: > > Fix UIA Fragment Root Navigation in Inspect > > > > We discovered that the fragment root change that was recently submitted > > here: https://chromium-review.googlesource.com/c/chromium/src/+/1823822 > > regressed fragment root navigation in Inspect. This was not the case > > when Kevin had first submitted it. I traced it down to the changes that > > I made for Controller-For elements, so this change undoes those > > modifications. > > > > In order to stabilize the content_browsertest that prompted the changes > > (DumpAccessibilityTreeTest.AccessibilityInputDateWithPopupOpen/uia), it > > was discovered that there were several issues in how we were > > handling date-time popups. > > > > The first issue is that we were failing to return the proper parent > > window for the fragment root. In > > RenderWidgetHostViewAura::GetParentNativeViewAccessible, we were > > returning nullptr as the parent of datetime popups, which was > > parenting the popup under the root desktop window in Inspect. This > > change now returns the host's window, which places the popup under > > the correct parent window in Inspect. > > > > The second issue is that there was a cardinality problem with datetime > > popup windows. Since they share a BrowserAccessibilityManager with > > the main window (but are under a different fragment root), > > LegacyRenderWidgetHostHWND::GetOrCreateBrowserAccessibilityRoot would > > always return the main window's root kRootWebArea instead of the > > popup window's kRootWebArea. So we'd have two different fragment roots > > that map to the same gfx::NativeViewAccessible. This would cause > > GetFragmentRootParentOf to return the incorrect parent most of the > > time. This change fixes that issue by adding an extra step for popup > > windows that traverses the root node and searches for the popup web > > root via OneShotAccessibilityTreeSearch. > > > > Bug: 1019016 > > Change-Id: I678425c8417334275727053243a39b1039e6f135 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885102 > > Commit-Queue: Kurt Catti-Schmidt <kschmi@microsoft.com> > > Reviewed-by: Scott Violet <sky@chromium.org> > > Reviewed-by: Kevin Babbitt <kbabbitt@microsoft.com> > > Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#711358} > > TBR=sky@chromium.org > > Bug: 1019016 > Change-Id: I73dc98eeb049788b3a7a086f5e6ac2927e2b0473 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894761 > Commit-Queue: Kurt Catti-Schmidt <kschmi@microsoft.com> > Reviewed-by: Nektarios Paisios <nektar@chromium.org> > Reviewed-by: Kevin Babbitt <kbabbitt@microsoft.com> > Cr-Commit-Position: refs/heads/master@{#712774} Bug: 1019016 Change-Id: I0fc44bf62612cc480f3bb4f818a584694bbaafe0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899202Reviewed-by:
Kurt Catti-Schmidt <kschmi@microsoft.com> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Kevin Babbitt <kbabbitt@microsoft.com> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Commit-Queue: Kurt Catti-Schmidt <kschmi@microsoft.com> Cr-Commit-Position: refs/heads/master@{#713518}
-
Sara Kato authored
Bug: none Change-Id: I7317204057092f41ee40128f6ecb264340e84da2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903233Reviewed-by:
Sara Kato <sarakato@chromium.org> Reviewed-by:
Akihiro Ota <akihiroota@chromium.org> Auto-Submit: Sara Kato <sarakato@chromium.org> Commit-Queue: Akihiro Ota <akihiroota@chromium.org> Cr-Commit-Position: refs/heads/master@{#713517}
-
Alex Ilin authored
Since the test file now contains real tests, it shouldn't be called demo. Change-Id: I46d4cc153543451ddc66265c334e91c100d778a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899505Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Alex Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#713516}
-
Abigail Klein authored
Observe AXEvents received by the RenderFrameHost. Log them in the the accessibility events recorder, which records events from the OS. chrome: accessibility page, intermixed with the events recorded by Bug: 785493 Change-Id: I7a72f786567c1f661de9f0945c01dc4d46789e72 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900209 Commit-Queue: Abigail Klein <abigailbklein@google.com> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#713515}
-
Patrick Monette authored
This CL changes RegisterWithOSAsDefaultClient() and CheckDefaultClientWithOS() to directly receive the callback instead of calling GetDefaultWebClientCallback() on the registry. Bug: 1021600 Change-Id: Ic71173e91121f0f3576b38532e030227f3f23797 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894698 Commit-Queue: Patrick Monette <pmonette@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#713514}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/efd8b7f5..500af51d Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I370b41ce9efbdfe83fae373012fb8f02d28b425e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902439Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#713513}
-
Vasiliy Telezhnikov authored
Some examples and tests create InProcessCommandBuffer without GpuChannelManagerDelegate. Some of them would crash trying to report new GpuBlockedTimeBetweenSwaps metric because of it. This CL fixes crash. Fixed: 1022412 Change-Id: I61a0a8a663d91bcf608ec2992d01b9d0752de8c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903841Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#713512}
-
Elly Fong-Jones authored
Something is wrong with the fallback font path that these tests hit, on 10.15.1 and probably other releases. Bug: 1022455 Change-Id: Ib82a5e27a5f0203f700363e663a08496da2df466 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902440Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#713511}
-
Aleks Totic authored
Change-Id: I06dc5c8977761e32c468fb43e91739d872f2a6aa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900925 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Auto-Submit: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#713510}
-
Nektarios Paisios authored
This is the first patch resulting from splitting into multiple patches. The CreateBoundaryStartPosition and CreateBoundaryEndPosition methods were re-written to work with boundaries that are not only at the beginning or the end of specific nodes, but could also fall on a text offset *within* a node. This patch doesn't create any new publicly exposed functionality in AXPosition. Even though all existing tests should be passing unmodified, this patch has uncovered a bug on how we used to compute upstream affinity which has now been fixed. A followup patch will re-write word navigation functionality to use the above two methods. A second followup patch will introduce support for document markers in AXPosition. R=kschmi@microsoft.com, aleventhal@chromium.org, ethavar@microsoft.com https: //crrev.com/c/1874254/ Bug: 1008031 Change-Id: I1ed67028c950cda1226c3a1b9b732a8d5a2af036 Change-Id: I1ed67028c950cda1226c3a1b9b732a8d5a2af036 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894159 Commit-Queue: Nektarios Paisios <nektar@chromium.org> Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Kurt Catti-Schmidt <kschmi@microsoft.com> Cr-Commit-Position: refs/heads/master@{#713509}
-
Vladimir Levin authored
This patch moves the directory for the intrinsic-size tests from css/css-intrinsic-size to css/css-sizing/intrinsic-size It also adds meta assert tags for the tests that explain what the test is testing. R=chrishtr@chromium.org, tabatkins@chromium.org Change-Id: Ie09b8185b0df4d641eec916f52e16b7cda371219 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903843 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#713508}
-
yilkal authored
This CL adds WebTimeLimitEnforcer. The enforcer class finds and reloads all WebContents that can be found by iterating through the TabStripModel of each of the browsers in BrowserList. Reloading triggers a new navigation which will trigger WebTimeLimitNavigationThrottle to block the WebContents. Bug: 1015661 Change-Id: I3d5e86f34c2eeb48f7a8ad4ca126cb4d4cde1cca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885510 Commit-Queue: Yilkal Abe <yilkal@chromium.org> Reviewed-by:
Aga Wronska <agawronska@chromium.org> Cr-Commit-Position: refs/heads/master@{#713507}
-
Lindsay Pasricha authored
Bug: 1015953 Change-Id: I6e08a727f800b7f0d0cedf40acdaa1669d829030 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900066Reviewed-by:
Justin Cohen <justincohen@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Commit-Queue: Lindsay Pasricha <lindsayw@chromium.org> Auto-Submit: Lindsay Pasricha <lindsayw@chromium.org> Cr-Commit-Position: refs/heads/master@{#713506}
-
Abhijeet Kandalkar authored
This CL has two goals, 1. Use To<HTMLTableRowElement> and DynamicTo<HTMLTableRowElement> as new downcast helper 2. Use IsA<HTMLTableRowElement>(element) in place of IsHTMLTableRowElement(element) Bug: 891908 Change-Id: I4095c5486fbdf63c3c692b295f23b18d626ab48d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899640Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#713505}
-
kylechar authored
conformance2/transform_feedback/switching-objects.html is flaky Bug: 1022410 Change-Id: Id7e28e5c82affda12612bbcd64f8d9e0c2da308a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903612 Auto-Submit: kylechar <kylechar@chromium.org> Commit-Queue: Brandon Jones <bajones@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#713504}
-
Hongchan Choi authored
See details in: https://analysis.chromium.org/p/chromium/flake-portal/flakes/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyYQsSBUZsYWtlIlZjaHJvbWl1bUBjb21wb25lbnRzX3VuaXR0ZXN0c0BBcmNoaXZlVmFsaWRhdG9yVGVzdC5HZXRTaXplQW5kQ29tcHV0ZURpZ2VzdE9uQ29udGVudFVyaQw TBR=petewil@chromium.org Bug: 1022323 Change-Id: I64b8bb53697f8538b8615d9548868f50c19ee6d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904156Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Commit-Queue: Hongchan Choi <hongchan@chromium.org> Auto-Submit: Hongchan Choi <hongchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#713503}
-
Mario Sanchez Prada authored
We need to check what UnhandledTapWebContentsObserver::FromWebContents() returns before using it to make sure it's not null before trying to bind the receiver, otherwise it will crash. See crrev.com/c/1887615 for extra context in the original CL. Bug: 936482,1006361,1021827 Change-Id: I52113a994722df552cbdf85c98efaed685dacd2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900004 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Donn Denman <donnd@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#713502}
-
Jun Mukai authored
Sometimes the screen width can be odd numbered, and the current logic will end up with a slight gap between the snapped bounds. This CL fixes the logic, the width will be based on the mid point of the work area. Bug: 1020351 Test: the new test case Change-Id: I8d4e3a7cac6ecf5ab4b18e5f6f5404223e88c9e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902766Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#713501}
-
Monica Basta authored
We never show the profile name if it is equal to GAIA name, e.g. Matt (Matt), in that case the profile name will be the GAIA name: Matt. In case of ambiguity, we prefer showing 'Person n' over showing redundancy. Bug: 1022438 Change-Id: I8f5e6a959a70ded588b435000a69e59bf7e2a762 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904189Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Monica Basta <msalama@chromium.org> Cr-Commit-Position: refs/heads/master@{#713500}
-
Yue Zhang authored
Bug: 1019101 Change-Id: Ie59028544a1953282a683f8740efc5fcc41e6d84 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895161Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Yue Zhang <yuezhanggg@chromium.org> Cr-Commit-Position: refs/heads/master@{#713499}
-
Abhijeet Kandalkar authored
While migrating code to new mojo type it is noticed that files versioning_apptest.cc and versioning_test_service.cc are not used by any platform. This CL removes unwanted code from codebase. Bug: 955171, 978694 Change-Id: Ifcbdb7d6bb4970ff960d31129cc0bd3dc159bf00 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903244Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#713498}
-
Abhijeet Kandalkar authored
This CL has two goals, 1. Use To<HTMLScriptElement> and DynamicTo<HTMLScriptElement> as new downcast helper 2. Use IsA<HTMLScriptElement>(element) in place of IsHTMLScriptElement(element) Bug: 891908 Change-Id: I7d041de56e0bfc57ba61a1b49f14f6f26d78f5a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899638Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#713497}
-
Mike Pennisi authored
Use an object with a null prototype for the internal mapping of names to cached values. This reduces the likelihood of unintended interaction with the environment (e.g. when new enumerable prototypes are defined on the Object prototype). Update the testing code so that this change is verifiable. Bug: 1014780 Change-Id: Ifbdc8d09c1049d0ec07f7a0099d774579423cdb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894707Reviewed-by:
John Chen <johnchen@chromium.org> Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#713496}
-
Mark Cogan authored
When closing the last incognito tab, the tab grid pauses briefly before scrolling to the regular tabs page. With VoiceOver enabled, the text on the Incognito page gets focused before this scrolling happens, which causes the scroll view to stay on the regular tabs page. (See the bug for more details). This CL removes the delay in scrolling when VoiceOver is enabled, immediately scrolling to the regular tabs page when the last incognito tab is closed. This prevents the timing issues that cause the text focus error. Bug: 980844 Change-Id: Idee32ce9863579372de9383644b3f24d02342960 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899995 Commit-Queue: Robbie Gibson <rkgibson@google.com> Reviewed-by:
Robbie Gibson <rkgibson@google.com> Auto-Submit: Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#713495}
-
Lucas Furukawa Gadani authored
The browser process will now notify the renderer to report a content security policy violation. Bug: 759184 Change-Id: I1a4f8fc82d522c67d7587c0dd9b65550c9354ac7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1857202Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Commit-Queue: Lucas Gadani <lfg@chromium.org> Cr-Commit-Position: refs/heads/master@{#713494}
-
Peter Boström authored
This accounts for additional internal padding which is present for leading/trailing buttons when maximized / in fullscreen. Bug: None Change-Id: If6841101172316174d2bce5b6091d2f31e9f197b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903652 Commit-Queue: Peter Boström <pbos@chromium.org> Commit-Queue: Caroline Rising <corising@chromium.org> Auto-Submit: Peter Boström <pbos@chromium.org> Reviewed-by:
Caroline Rising <corising@chromium.org> Cr-Commit-Position: refs/heads/master@{#713493}
-
Brian Salomon authored
Change-Id: I7fedc818b6fad5c08f37c8002268c65eefdf57a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757345 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by:
Mike Klein <mtklein@chromium.org> Cr-Commit-Position: refs/heads/master@{#713492}
-
Etienne Bergeron authored
The CL is fixing a bug when the rewriting loop iterate through the input text. If the codepoint is updated with a larger codepoint, the next_offset was not updated properly and was pointing in the middle of the surrogate pair. This was not happening since only the PUA codepoints were producing larger characters. Change-Id: I4c6eea9a1a02374d04a0271bd4d6140e123069ba Bug: 4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903808Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#713491}
-
Christopher Cameron authored
A few issues here: First, actually wire up NativeWidgetNSWindowBridge as a DisplayObserver. In the past, it didn't matter that this code was missing, because we were getting notified via windowDidChangeBackingProperties. In Catalina, that call is happening at a different time, resulting us using an invalid cached version (which is the second issue). Second, change GetDisplayNearestWindow to call UpdateDisplaysIfNeeded. There was a bug here wherein we would return displays_[0], even if we knew (because of displays_require_update_) that that value was out of date. Thid, make GetCachedDisplayForScreen be robust to getting a surprise NSScreen* that we didn't know about. On Catalina, it happens that we can read -[NSScreen screens] and see that it has changed, before having received any notifications that such a change would happen (!). Fourth, listen for NSApplicationDidChangeScreenParametersNotification notifications. Just because it sounds like a healthy thing to be doing. Bug: 1021340 Change-Id: Ibe5a6469d9e2c39cd81d0fb19ee2cfe3aedb1488 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902508Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#713490}
-
Roger Tawa authored
Bug: 1021949 Change-Id: I0b3ae7dcb2c53902f010820f839f8a85414f888d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903950Reviewed-by:
Daniel Rubery <drubery@chromium.org> Commit-Queue: Roger Tawa <rogerta@chromium.org> Cr-Commit-Position: refs/heads/master@{#713489}
-
Dmitry Torokhov authored
The dispatcher state will reside in daemon store/cryptohome instead of global stateful to avoid cross-user information disclosure, so it needs to know cryptohome ID of the session owner in order to bind-mount the right state directory. Bug: b:143443446 Change-Id: Id29b10f2981fe9e919fe41d3e0f4c3832bde1654 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899838Reviewed-by:
Timothy Loh <timloh@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Dmitry Torokhov <dtor@chromium.org> Cr-Commit-Position: refs/heads/master@{#713488}
-
Jonathan Ross authored
A few tests were timing out on Linux + Intel bots. However they pass when the timeout is increased. Marking these tests as slow on Linux to address this. TBR=backer@chromium.org Test=vulkan_native_blink_web_tests Bug: 1018884 Change-Id: Ib2c0197e5f163ced2f76b7ba929f7c589f973de8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900553Reviewed-by:
Jonathan Ross <jonross@chromium.org> Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#713487}
-
Etienne Bergeron authored
This CL is writing the invalid codepoint by the replacement characters. See https://unicode.org/reports/tr44/ """ Noncharacter_Code_Point : Code points permanently reserved for internal use. """ Bug: 1011818 Change-Id: Id39b42f52a8c36887fabcd81e794f46ebedec089 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902772Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#713486}
-
Sylvain Defresne authored
The class OCMockComplexTypeHelper does not really need any of the behaviour coming from GTMLightweightProxy, so reimplement it as a subclass of NSProxy directly. Bug: none Change-Id: If66eca04df012853d0e619c44ca3f5bc18f5ffdf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903135 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#713485}
-
Demetrios Papadopoulos authored
This is in preparation of turning on such PRESUBMIT checks for src/components. Bug: 1019312 Change-Id: I352cf547b8ea51cb6eba3a8715122c2129948cc6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902065 Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Esmael Elmoslimany <aee@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#713484}
-
Elly Fong-Jones authored
Some unit tests need access to the footnote view of a dialog. This change adds a method to DialogDelegate to get the footnote view, and changes a test suite (BookmarkBubbleTest) to use it. Bug: 1011446 Change-Id: Ica62c0512a36b6f58c26b3ad8e9694e6a7c2cbad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900363 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#713483}
-