- 31 Mar, 2020 40 commits
-
-
Andrew Grieve authored
Targets should start depending on these directly rather than going through the legacy androidx_appcompat_appcompat_java target. Bug: 896775 Change-Id: Ic1a6371882f70fb6da6731409c34d32d27017bf8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119542 Commit-Queue: Sam Maier <smaier@chromium.org> Commit-Queue: benjamin joyce <bjoyce@chromium.org> Auto-Submit: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Sam Maier <smaier@chromium.org> Reviewed-by:
benjamin joyce <bjoyce@chromium.org> Cr-Commit-Position: refs/heads/master@{#755121}
-
Ben Schwartz authored
Both the Secure DNS Policy Handler and the Secure DNS Settings UI support entry of multiple DoH templates. However, the Policy Handler requires all the entered templates to be valid, while the Settings UI only requires at least one of them to be valid. This change harmonizes the two, to require that all entered templates be syntactically valid. It also reduces duplication of string manipulation for splitting groups of templates. Change-Id: Ia66d939a93b6efae70e2fbdd31a6811c110be36f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2125874 Commit-Queue: Ben Schwartz <bemasc@chromium.org> Auto-Submit: Ben Schwartz <bemasc@chromium.org> Reviewed-by:
Eric Orth <ericorth@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#755120}
-
Andrew Comminos authored
To support an isInputPending implementation that supports off-main-thread attribution, propagate compositor attribution decisions to the main thread scheduler. Issue: 910421 Change-Id: I23d643f790b7c7b8a4f9d32ec4fc88d930ae76ed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2083684Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Stefan Zager <szager@chromium.org> Reviewed-by:
Scott Haseley <shaseley@chromium.org> Commit-Queue: Andrew Comminos <acomminos@fb.com> Cr-Commit-Position: refs/heads/master@{#755119}
-
David Bokan authored
Mac key event handling works by first sending a key to the renderer. When the renderer ACKs the event as unconsumed, the browser re-injects the NSEvent back into the app. This is done so that the browser can perform commands if the renderer didn't consume them (e.g. Command-T). The Event.Latency.OS_NO_VALIDATION metric was temporarily added to investigate the cause of long instances in the InputDelay metric. It measures the delta between the event's OS-provided timestamp and the current time when a WebKeyboardEvent is built from an NSEvent. This metric is meant to narrow down where these long InputDelay values are coming from. OS_NO_VALIDATION assumes that the call to it happens close to where the OS event is received by Chrome. However, the reinjection mechanism mentioned at the top breaks this assumption. When the event is reinjected, there are cases where it must be passed to the content/ layer (e.g. when being passed to the Views FocusManager[1]) and a WebKeyboardEvent is reconstructed. Hence we record this metric multiple times for such events, the second time causing the metric to depend on delays in the jank. This can reproduced by visiting a page with a very busy main thread (e.g. https://rbyers.github.io/scroll-latency.html and set a high periodic jank) and performing "key equivalents" (i.e. browser commands like Command-C) while the main thread is blocked. You'll notice very high values reported in [2]. This CL adds a temporary constructor to NativeWebKeyboardEvent on Mac that causes us to record this metric only on the code path that will eventually send the event to the renderer (i.e. avoid reinjected events). This is temporary to prove this hypothesis and determine how much of the problematic reports are due to other causes. It will be reverted after a few days of data is collected. Note: In https://crrev.com/4a7404eceeb I added a DumpWithoutCrash in cases where we saw a big discrepancy in the Now() and event timestamp. The issue addressed in this CL shows up in these stacks but there are other cases that have a big discrepancy and are captured from RenderWidgetHostViewCocoa::keyEvent. This CL will help cut down on the noise so we can keep diagnosing the root cause. [1] https://cs.chromium.org/chromium/src/chrome/browser/ui/cocoa/chrome_command_dispatcher_delegate.mm?l=44&rcl=fb1021a791d9141b4df6d639f0d50bfe0c0ce944 [2] chrome://histograms/Event.Latency.OS_NO_VALIDATION.POSITIVE.KEY_PRESSED Bug: 1039833 Change-Id: I17af7f76db0f69024ec75b45822d69178a4cd565 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129238 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#755118}
-
Kurt Horimoto authored
This reverts commit e033ca47. Reason for revert: Not needed yet. Original change's description: > [iOS] Add Safe Browsing opt out setting. > > Bug: none > Change-Id: Iaf328b22a6fba65a1d9a6deb837f379cd9405b5e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2088187 > Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> > Reviewed-by: Rohit Rao <rohitrao@chromium.org> > Reviewed-by: Ali Juma <ajuma@chromium.org> > Cr-Commit-Position: refs/heads/master@{#751135} TBR=rohitrao@chromium.org,ajuma@chromium.org,kkhorimoto@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: none Change-Id: Ibc388798926a634d76b014b2f8dd931d22b6f7b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130666Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#755117}
-
Brandon Wylie authored
Bug: 1059898 Change-Id: I22482839c28491a00d7f473e924de0710e8c94db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127741 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by:
Ender <ender@google.com> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#755116}
-
Rebekah Potter authored
This reverts commit 8aee0918. Reason for revert: Test being added is flaky. See https://bugs.chromium.org/p/chromium/issues/detail?id=1066619 Original change's description: > Add MainSettingsFragmentTest > > MainSettings.java have bit less test coverage over some of the > preferences, for example, the HomepageSettings at this moment. Adding > MainSettingsFragmentTest is aiming for more test coverage for the > list of preferences on the settings screen. > > Bug: 1052060, 1066362 > Change-Id: Ib51c133f2a73f4ec85cc185c4101833bc3fa0b58 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124729 > Commit-Queue: Wenyu Fu <wenyufu@chromium.org> > Reviewed-by: Theresa <twellington@chromium.org> > Reviewed-by: Natalie Chouinard <chouinard@chromium.org> > Cr-Commit-Position: refs/heads/master@{#754957} TBR=twellington@chromium.org,wychen@chromium.org,chouinard@chromium.org,wenyufu@chromium.org Change-Id: If15e22de962484fc87b3b5dc5eb1cfcf550cf024 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1052060, 1066362 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2131017Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#755115}
-
Victor Vasiliev authored
https://quiche.googlesource.com/quiche.git/+log/1bcd0c3f96e1..0abcd8febd32 $ git log 1bcd0c3f9..0abcd8feb --date=short --no-merges --format='%ad %ae %s' 2020-03-30 wub gfe-relnote: (n/a) When QUIC switches from BbrSender to Bbr2Sender, let Bbr2Sender copy the bandwidth sampler states from BbrSender. Protected by --gfe2_reloadable_flag_quic_bbr_copy_sampler_state_from_v1_to_v2. 2020-03-30 renjietang Remove QuicStreamIdManager::DelegateInterface::OnCanCreateNewOutgoingStream(). 2020-03-30 wub gfe-relnote: (n/a) Deprecate --gfe2_reloadable_flag_quic_use_quic_time_for_received_timestamp2. 2020-03-30 vasilvv Ensure QuicTransportStream::Visitor::OnFinRead() is only called once. 2020-03-30 bnc Fold QuicReceiveControlStream::HttpDecoderVisitor into QuicReceiveControlStream to remove one layer of indirection. 2020-03-30 fayang gfe-relnote: Reset consecutive_pto_count and loss detection when discarding keys. Protected by existing quic_enable_version_draft_25_v3 and quic_enable_version_draft_27. 2020-03-30 bnc Do not send a MAX_PUSH_ID frame if SetMaxPushId() has not been called yet. 2020-03-27 bnc Simplify Http3DebugVisitor methods. 2020-03-27 bnc Pass payload length or compressed header block length to some HttpDecoder::Visitor methods. 2020-03-27 fayang Enable IETF QUIC for more gfe/gfe2/e2e tests. Also let quic_spdy_client process MAX_STREAMS if it cannot open new streams. 2020-03-27 dschinazi Move some QUIC tests to QuicTest Created with: roll-dep src/net/third_party/quiche/src R=bnc@chromium.org Change-Id: I5fb2f81dd0a8d7aa81edc19831554e7335de35b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128716Reviewed-by:
Bence Béky <bnc@chromium.org> Commit-Queue: Victor Vasiliev <vasilvv@chromium.org> Cr-Commit-Position: refs/heads/master@{#755114}
-
Sammie Quon authored
Moves the enums I thought could belong to either class that uses them to a shared location. Left ones that I thought were clearly associated with their class (OverviewItemView::HeaderVisibility, OverviewWindowDragController::DragBehavior). Also removes unused enum OVERVIEW_ANIMATION_SELECTION_WINDOW. Test: compiles Bug: none Change-Id: I1b280f29e3330b238754e019509f0da35ce4d5ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2125553Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#755113}
-
Peter Kotwicz authored
This CL makes WebAPKs use TwaSharingController as part of merging the WebAPK and TWA codebases. BUG=1059580 Change-Id: I32db42b90854937318b06ed2a295078659ae7bd1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100817 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Glenn Hartmann <hartmanng@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#755112}
-
Kyle Milka authored
Use the activity context instead of the application context to tint the icons in the share sheet. Bug: 1009124 Change-Id: I5f0fc4175dce70031c41b4f8f7e0d31578a267ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130670Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#755111}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/17ffd9875c34..a680c23e7859 git log 17ffd9875c34..a680c23e7859 --date=short --first-parent --format='%ad %ae %s' 2020-03-31 ehmaldonado@google.com git-cl: Fix checking if parent branch has been uploaded. 2020-03-31 ahaas@chromium.org [man] Fix order of depot_tools in PATH 2020-03-31 ehmaldonado@google.com git-cl: Move GCE detection when creating a connection. Created with: gclient setdep -r src/third_party/depot_tools@a680c23e7859 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: None Tbr: agable@chromium.org Change-Id: I429219759bd40684408b8c78063d53c401a91ead Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130828Reviewed-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@{#755110}
-
Hesen Zhang authored
Bug: 1060805c Change-Id: Id2ca8e91cd3f9626d570f68f69563d830f2ff7b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128940Reviewed-by:
Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
Hesen Zhang <hesen@chromium.org> Commit-Queue: Hesen Zhang <hesen@chromium.org> Cr-Commit-Position: refs/heads/master@{#755109}
-
Xiaocheng Hu authored
This reverts commit 950810de. Reason for revert: Loading behavior fixed Original change's description: > Revert "Record split UMA histograms when rendering has been delayed for font preloading" > > This reverts commit 4a7fe529. > > Reason for revert: The loading behavior is designed wrong. > > The objective of these split histograms is to study, when there's font preloading, how effective delaying rendering is. > > Currently: the behavior is observed when rendering is delayed by font preloading. > > What's actually needed: the behavior is observed when font preloading occurs > > Original change's description: > > Record split UMA histograms when rendering has been delayed for font preloading > > > > This patch adds split histograms of the following metrics when > > rendering delay for web font preloading has been triggered: > > - PaintTiming.NavigationToFirstPaint > > - PaintTiming.NavigationToFirstContentfulPaint > > - PaintTiming.NavigationToLargestContentfulPaint > > - LayoutInstability.CumulativeShiftScore > > > > This will help us understand the effectiveness of this optimization, and > > adjust the length of the delay to the best value. > > > > Bug: 1040632 > > Change-Id: I6f8aa151c576c9d8d583647a3a7ef6fcb8fb833d > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109251 > > Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org> > > Reviewed-by: Charlie Harrison <csharrison@chromium.org> > > Reviewed-by: Chris Harrelson <chrishtr@chromium.org> > > Reviewed-by: Bryan McQuade <bmcquade@chromium.org> > > Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#754771} > > TBR=chrishtr@chromium.org,ksakamoto@chromium.org,skobes@chromium.org,bmcquade@chromium.org,csharrison@chromium.org,xiaochengh@chromium.org > > Change-Id: I7829822fea47183df9d4ed27c0eec2a1f888aa0b > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1040632 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2131010 > Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org> > Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#755081} TBR=chrishtr@chromium.org,ksakamoto@chromium.org,skobes@chromium.org,bmcquade@chromium.org,csharrison@chromium.org,xiaochengh@chromium.org Change-Id: Ibae7230089fe9e56e91cf1480d19c7bc70778567 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1040632 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2131011Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#755108}
-
Maksim Sisov authored
After TooltipControllerCaptureTest was renamed to TooltipControllerTest, the filters were not updated. The CL that did that - https://crrev.com/c/2117335 Fix that. Bug: 1065474 Change-Id: Ibc8df1137f45d001eb81ce3624a59e2c1c81b4ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130248 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#755107}
-
Christian Dullweber authored
An extension can use the privacy.websites.ThirdPartyCookiesAllowed extension api to set the kBlockThirdPartyCookies preference. kCookieControlsMode needs to be set to the corresponding value if kBlockThirdPartyCookies is controlled by extension. Bug: 1065392 Change-Id: Ib450349cb33251b3eccc06ff2dcc18a0899a028e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127034 Commit-Queue: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#755106}
-
Elly Fong-Jones authored
This provides access via the system menus to a new setting that is currently only present in the omnibox context menu. To achieve that, this change: 1) Adds a new IDC_SHOW_FULL_URLS command and handling logic 2) Adds a new conditionally-present item to the main menu bound to that command 3) Has OmniboxViewView listen for changes in the backing pref and update its appearance 4) Remove the logic in OmniboxViewView to manually update appearance when changing the pref, since that is now accomplished via listening to changes on the pref This change doesn't have any automated tests. I spent some hours trying to write one, but ran persistently into two problems: 1) All the existing tests of OmniboxViewView are Views unittests and do not have a "real" LocationBarView/Browser/PrefService 2) Adding a new test, using BrowserWithTestWindowTest instead, means having to construct a real LocationBarView and quite a few other backing services/classes/etc. The effort involved and brittleness of the resulting test quickly exceeded the value of the test, so I decided to just do without. Bug: 1061576 Change-Id: I17d86124e334fb7f85407a16f2c8c31d9f77ebbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120854 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#755105}
-
Qijiang Fan authored
Add missing trailing semicolon. Bug: 1065504 Change-Id: I98fe899a3934a1c4e6df34f7e08b171647bd9be4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130447 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Auto-Submit: Qijiang Fan <fqj@chromium.org> Cr-Commit-Position: refs/heads/master@{#755104}
-
Richard Coles authored
This reverts commit 350ec69f. Reason for revert: After this change, changing android_channel in args.gn and rebuilding doesn't reliably cause a manifest rebuild and some of the built APKs end up still using the package name/etc from the previous channel. crbug.com/1066594 Original change's description: > Android: Add md5check to compile_resources.py > > Using md5check speeds up the build by not re-compiling resources if its > dependencies have not changed. > > Shaves 5 seconds off for Chrome Java Signature Change (77s -> 72s). > Shaves 12 seconds off for Base Java Signature Change (174s -> 162s). > > Bug: 906803 > Change-Id: Idccbc080dfdfb75b1ce801f3fcd934b93c2c3bb1 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120047 > Commit-Queue: Peter Wen <wnwen@chromium.org> > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Auto-Submit: Peter Wen <wnwen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#754507} TBR=wnwen@chromium.org,agrieve@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 906803 Change-Id: I66045a49a1076b1c2644cc1a1b96a5d4c3cf547d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130866Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Ben Mason <benmason@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#755103}
-
Regan Hsu authored
* Keydown 'Enter' causes the selected row to trigger a route change. * If the selected row is focused, keydown 'Enter' triggers route change. * Clicking on specific row causes the row to trigger a route change (iron-list also causes the row to become the selected row on click). * If a user deselects a row, re-select the row (one row should always be selected). Deselection is an <iron-list> behavior we must work around. * Add test to confirm <os-search-result-row> has the correct behaviors. * A route change won't cause the cr-input to lose its text, nor will it cause the results to be lost, nor the selected item to be deselected. Bug: 1056909 Change-Id: I00c151acd7439b916b4b01bec188cb48197c775d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2123787 Commit-Queue: Regan Hsu <hsuregan@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#755102}
-
Kurt Catti-Schmidt authored
Problem : AXObjectCacheImpl accumulates events when it's not clear whether layout is in a clean state then fires these events when layout is clean. Unfortunately this means that if an event is enqueued that isn't associated with a layout change then there's no guarantee the event will be fired. This CL attempts to fire these deferred events more regularly by hooking up an accessibility DocumentLifecycle state, and scheduling a lifecycle update whenever a deferred event is enqueued to guarantee the events are fired. Bug: 1050792 Change-Id: Iceb9fe5252aff5538435710b1d285b93c609bba3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2055361 Commit-Queue: Kurt Catti-Schmidt <kschmi@microsoft.com> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Cr-Commit-Position: refs/heads/master@{#755101}
-
Archana Simha authored
Extends the expiration of Extensions.Checkup.TimeSpent, Extensions.Checkup.NtpPromoClicked, and Extensions.Checkup.NtpPromoShown to after M85. Bug: 1066034 Change-Id: I082734ceacb0ea66dfd8893dfc35b7b8fb88dbb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129027Reviewed-by:
Alex Ilin <alexilin@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Archana Simha <archanasimha@chromium.org> Cr-Commit-Position: refs/heads/master@{#755100}
-
Roman Sorokin authored
Measure time between start showing the screen and the ready state. Bug: 1066489 Change-Id: I6e705c222db8b5bd178f8edfbb35393669f10f59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129551 Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#755099}
-
Andrew Grieve authored
Keeping R8 revision the same, but updated local modifications: * Expose dexsplitter-related R8 flags * Taken from reverted change: https://chromium-review.googlesource.com/c/chromium/src/+/2014309 * Disable stateless lambda "optimization" * Makes horizontal & vertical class merging separate flags. * Make --file-per-class use DexFilePerClassFile rather than DexFilePerClass (fixes incremental dexing /w desugar_with_d8). Also updates update instructions. Bug: 1032609 Change-Id: I1553785fa103ed3164206e0397fe320cce980f61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127512Reviewed-by:
Sam Maier <smaier@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#755098}
-
Dominique Fauteux-Chapleau authored
TBR=msw@chromium.org Bug: 1066607 Change-Id: I745a6851628f5c59431a4a28f7294b1829b361ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130968Reviewed-by:
Dominique Fauteux-Chapleau <domfc@chromium.org> Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org> Cr-Commit-Position: refs/heads/master@{#755097}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/8cbedf985570..8b99b9e1d11c git log 8cbedf985570..8b99b9e1d11c --date=short --first-parent --format='%ad %ae %s' 2020-03-31 mtklein@google.com flatten skvm type nesting 2020-03-31 herb@google.com Update header list for spirv-tools 2020-03-31 bsalomon@google.com Revert "Revert "Assert that all GrCaps colortype/format -> swizzle queries are legal."" 2020-03-31 herb@google.com Introduce Liveness and Uses into existing scheduler 2020-03-31 egdaniel@google.com Add rasterizer and blend state to d3d pipeline state. Created with: gclient setdep -r src/third_party/skia@8b99b9e1d11c 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 michaelludwig@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: michaelludwig@google.com Change-Id: I7d5d59d0d4d34c99eb78dfba13f7808cbf46e78c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130607Reviewed-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@{#755096}
-
Navid Zolghadr authored
Change-Id: Ieff28b1dbafd39f3b38b047553ea67d3f6cea959 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129997Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Navid Zolghadr <nzolghadr@chromium.org> Cr-Commit-Position: refs/heads/master@{#755095}
-
Becca Hughes authored
Index names are at the DB level and some of them are duplicated so this prefixes them all with the table name. BUG=1024351 Change-Id: I9e8ba700625e9d2c42f2fbeaab3ba216f450f0a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129898Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#755094}
-
Michael Spang authored
../../chromecast/media/cma/backend/volume_control.cc:57:7: error: unused function 'DbFsToScale' [-Werror,-Wunused-function] float DbFsToScale(float db) { ^ 1 error generated. Bug: b/152723969 Test: Cast CQ Change-Id: Ie8553fbf2c5b537d8e493d652e442bb6199030a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130772Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#755093}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: I8a3cc83c8e2d08e77bd8b1a4831c81db8952a468 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130312Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#755092}
-
Danyao Wang authored
This allows adding new payments-specific JNI files without always having to get //chrome/android OWNERS review. Bug: 1063118 Change-Id: Ia2c86780c7527ea871902664496105908d2532ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129026 Commit-Queue: Danyao Wang <danyao@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#755091}
-
Aya ElAttar authored
After OSSettings is set to be disabled in ExtensionManagement, it gets re-enabled because it's ComponentLocation & from_bookmark, so an exception has to be added to enable admins to disable it. This exception will go away when OSSettings is switched to WebApps, which should happen very soon. Bug: 1063404 Change-Id: I457fb5e86aedc54930fb795d90e9fa7650a4f938 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124790Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Commit-Queue: Aya Elsayed <ayaelattar@google.com> Cr-Commit-Position: refs/heads/master@{#755090}
-
Anqing Zhao authored
Remove unnecessary words to make sure our UI is easy to localize and free of cultural references that may confuse users. Bug: 899414 Change-Id: Id5474ea5edd0adf46226c00f00963710dc084234 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128317 Commit-Queue: Anqing Zhao <anqing@google.com> Reviewed-by:
Anatoliy Potapchuk <apotapchuk@chromium.org> Cr-Commit-Position: refs/heads/master@{#755089}
-
Daniel Hosseinian authored
The histogram can be insightful during the transition of the macOS printing backend to CUPS IPP. Meanwhile add a secondary owner. Bug: 1053247 Change-Id: Idb3f5717b89fd5a9fe026d2be4782454d7280589 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130328 Commit-Queue: Daniel Hosseinian <dhoss@chromium.org> Commit-Queue: Ilya Sherman <isherman@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Auto-Submit: Daniel Hosseinian <dhoss@chromium.org> Cr-Commit-Position: refs/heads/master@{#755088}
-
Rainhard Findling authored
Bug: 1015841 Change-Id: I0a3103628117bd5a46750999a4f6020915df61d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2116219Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: Rainhard Findling <rainhard@chromium.org> Cr-Commit-Position: refs/heads/master@{#755087}
-
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/+/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: Id352eb5c44a590e4b272a2ea1bc239864dd4f558 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130826Reviewed-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@{#755086}
-
Tim Song authored
BUG=1052916 Change-Id: I5b402cbda97f83d9abac29d105c469fd890e2e53 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112335Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Tim Song <tengs@chromium.org> Cr-Commit-Position: refs/heads/master@{#755085}
-
Dave Tapuska authored
input-events-typing.html and caretnavigation.html both use test_driver.Actions which isn't supported yet by content_shell. BUG=893480 Change-Id: Ic4b0f56ac6a32a97c5b2344cd11536401fa67e2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2130673Reviewed-by:
Robert Ma <robertma@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#755084}
-
Oksana Zhuravlova authored
Since only Chrome binds InstalledAppProvider, but a default no-op behaviour is expected for other embedders (tests, WebView, WebLayer), this change removed the 'non-Android' condition for the default InstalledAppProvider binder to avoid renderer termination for the 'no binder found' case. Bug: 1064644, 1064827 Test: No crash when opening reddit.com in webview shell on an emulator Change-Id: Ia9e6d98f0abff94929bf10dcd778166c90907f40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124872Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#755083}
-
edchin authored
UIKit has fixed orphaned word in UILabel since iOS 11. Our custom implementation of fixing orphaned words is no longer necessary. Furthermore, adding an arbitrary new line in a UILabel has unintended consequences, such as failing EG tests that would otherwise pass. Change-Id: Ifc5ca16d8fca83aa67306d94ceac9db132ded45a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129241Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#755082}
-