- 28 Apr, 2020 40 commits
-
-
Devlin Cronin authored
//chrome/browser/extensions/extension_action.h and extension_action.cc have no //chrome dependencies. Move them to //extensions. This is a pure move; there are no functional changes. Bug: 1073590 Change-Id: Iedf4fcbaf871470e727046dce0976270a239b035 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2162140Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#763368}
-
Ben Pastene authored
This will let folks check out multiple board SDKs at once without having to modify their .gclient file each time. Needs to land after crrev.com/c/2163589. Bug: 937821 Change-Id: I191985b4d5044c84009b8a7d05e33785b6e81e37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2163849Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#763367}
-
Nate Chapin authored
This means DeviceOrientationController and DeviceMotionController become Supplements of LocalDOMWindow instead of Document, too. Bug: 1029822 Change-Id: I16511f45f1ac1eb343e466217b324b1217f14a83 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2166586Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#763366}
-
Pavol Marko authored
This CL makes TestNavigationObserver track state per-WebContents instead of globally: (*) Whether a navigation is in progress (in |navigation_started|) (*) Whether the last navigation that finished matched the |target_url| / |target_error| filter (in |last_navigation_matches_filter|). When TestNavigationObserver is configured with matching against a target_url, only set last_* member variables if the target_url actually matches. This solves a race condition that TestNavigationObserver suffered from previously when receiving events from multiple WebContents: TestNavigationObserver with target_url_ == A, and default wait_event_ == WaitEvent::kNavigationFinished. (1) OnDidFinishNavigation for URL A Set last_navigation_url_ == A (2) OnDidFinishNavigation for URL B Set last_navigation_url_ == B (3) OnDidStopLoading (probably matching (1)) Does not trigger EventTriggered() because (target_url_ == A) != (last_navigation_url_ == B) (4) OnDidStopLoading (probably matching (2)) Same as above. Also, use base::Optional to make it obvious if |target_url| / |target_error| filters are active. Bug: 830337 Change-Id: Id1d47a0e4ed20d6df38ac222e5eb3eed97bcbc63 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2161032 Commit-Queue: Pavol Marko <pmarko@chromium.org> Reviewed-by:Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#763365}
-
CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-betty-pi-arc-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-eve-compile-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-kevin-compile-chrome BUG=762641 TBR=chrome-os-gardeners@google.com Change-Id: I43dfde446232343dabb16ab223396199251d0524 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170136Reviewed-by:
ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Commit-Queue: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#763364}
-
Liviu Tinta authored
Adding missing wpt test description. Bug: 1031745 Change-Id: I118731dfb49dc7b454ada3ecf8d9b4edabcb5061 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169760 Commit-Queue: Liviu Tinta <liviutinta@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#763363}
-
Xing Liu authored
The build failure histogram is added to track an android bug. This can be removed now. TBR=bcwhite@chromium.org Bug: 1075635 Change-Id: I67acd66efcfdef186528a0076724175f8e789b68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2168008 Commit-Queue: Xing Liu <xingliu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#763362}
-
Nate Chapin authored
Because it (and its subclasses) no longer use a Document, we can remove a couple of functions that are duplicated between Document and LocalDOMWindow. Bug: 1029822 Change-Id: I8d023642ef82818e9b75d9041cb861be0996947b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2168511Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/2e08eda8..d50c00bf 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,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I38c7777b388d9c8d949a42838f73bb7752a876b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170187Reviewed-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@{#763360}
-
Andrei Salavei authored
Attempt to recover Access Token in case request to the |ExchangeToken| endpoint failed with 4xx(unauthorized) http error. Bug: 1071397 Test: browser_tests --gtest_filter=ArcAuthServiceTest.* Change-Id: I39d89a7ace885de589030cea297a58ce924d1e2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2151592Reviewed-by:
Anastasiia N <anastasiian@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Commit-Queue: Andrei Salavei <solovey@google.com> Cr-Commit-Position: refs/heads/master@{#763359}
-
Rushan Suleymanov authored
This patch extends checks for the bookmarks metadata while loading from the disk. If the duplicate tombstone is detected then the whole metadata is rejected with a new reason DUPLICATED_CLIENT_TAG_HASH. Bug: 1050716 Change-Id: I61d9e5dded864a1a3ce4ee3c9295b01ca82a5ca3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2162906 Commit-Queue: Rushan Suleymanov <rushans@google.com> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#763358}
-
Trent Begin authored
This splits cros_network_config mojo interface into a shared group of enums that are reused in Network Health's interface. Bug: chromium:1057739 Change-Id: I5bc2673f0c856dbbcb689a96282982e795562271 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2157769Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Trent Begin <tbegin@chromium.org> Cr-Commit-Position: refs/heads/master@{#763357}
-
Colin Blundell authored
In the initial bringup of ExternalNavigationDelegateImpl this method was not implemented, and //weblayer's flow was configured to ensure that closing tabs was never requested. However, WebLayer's flow now properly detects cases where tabs should be closed, and hence ExternalNavigationDelegateImpl can actually be requested to close a tab. This CL implements that closing via the same flow as //weblayer's InterceptNavigationDelegateClientImpl#closeTab(). The only case in which this flow currently occurs is an extreme corner case that is unlikely to occur in practice: - The app using WebLayer does not have storage permissions - The user goes to a file URL in a context where the tab should be closed if the navigation is overridden (essentially, the navigation is the initial navigation in the tab) - The user denies the app storage permissions in the dialog that WebLayer pops up I looked at testing this flow. However, it turns out that it would require adding a private AIDL API to configure the WindowAndroid instance to not regard the app as having the READ_EXTERNAL_STORAGE permission. This seemed like an unnecessary amount of overhead for a flow that is basically non-existent in the product. When I implement the warning dialog for launching intents from incognito, that will also go through this closeTab() method. At that point I will look at constructing a test for that more-realistic use case. Bug: 1074301 Change-Id: Ic342877c569298b2a0e29cda10316c0cbe5074d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2162750Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#763356}
-
Harald Alvestrand authored
This will happen if invoking functions on an object after its context has been destroyed. Added test. Bug: chromium:1072412 Change-Id: Icc2e8a5ad47398acffb2d56a299a51b11386c9f2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2160909 Commit-Queue: Harald Alvestrand <hta@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
-
Michael Ershov authored
When key_name_for_spkac_ is specified, return public key for it. In other cases return public key for key_name_. Bug: 1045895, 1075840 Change-Id: I5af345ecda8c4c61fe9eaaedaa7f059a1810ed8f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169994 Commit-Queue: Michael Ershov <miersh@google.com> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Cr-Commit-Position: refs/heads/master@{#763354}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/012aa375b177..4381af48b40b git log 012aa375b177..4381af48b40b --date=short --first-parent --format='%ad %ae %s' 2020-04-28 landrey@webrtc.org Change connection ASSERT into metrics for the PC level framework. 2020-04-28 landrey@webrtc.org Add cpu_usage metrics. Created with: gclient setdep -r src/third_party/webrtc@4381af48b40b 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/+/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: I98c7d9807d1f4a5c1d3cded5c17bac4a25a947ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170287Reviewed-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@{#763353}
-
Nicolás Peña Moreno authored
This CL attempts to fix test flakiness. * For the click counts, remove tests about mouseout/mouseover since those could fail under certain starting positions of the mouse, and the test has a pretty high flakiness score right now. * For other tests, fix flakiness by making sure that PerformanceObserver callbacks where mousedown is not present are ignored. This is possible since a click triggers mousedown, mouseup, click, etc., so we cannot just assume that mousedown will always be present at the callback. Bug: 1074048 Change-Id: I95a082948c105cf18b51376c9d6421195b23b718 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2163775Reviewed-by:
Yoav Weiss <yoavweiss@chromium.org> Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Cr-Commit-Position: refs/heads/master@{#763352}
-
Mehran Mahmoudi authored
This CL implements a new system for managing custom Views for Tab. The TabViewManager class acts as the entry point for all users who want to display a View on top of Tab's Content view. TabViewManager knows the priority of all its users and always displays the TabViewProvider with the highest priority. Internal design doc: http://go/tab-view-clients Bug: 1073948 Change-Id: I5d5d0384b512d6722dd0357295de5542b9b77306 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2159248Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Patrick Noland <pnoland@chromium.org> Reviewed-by:
Mehran Mahmoudi <mahmoudi@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Commit-Queue: Mehran Mahmoudi <mahmoudi@chromium.org> Cr-Commit-Position: refs/heads/master@{#763351}
-
Bartek Nowierski authored
This is needed because of how argument-dependent lookup works (it looks in the namespace of the type for an unqualified function name). According to go/using-std-swap, the recommended usage pattern is: using std::swap; swap(thing1, thing2); Having swap outside of base namespace wouldn't work with this recommendation as slower std::swap would be silently used instead. Bug: 1073933 Change-Id: Ib4485b51ba52ad491306bec9b4402a154adae16c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2167796 Auto-Submit: Bartek Nowierski <bartekn@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
-
Gabriel Charette authored
Jank analysis uncovered that Windows uses native ::PeekMessage calls as an opportunity to yield to other threads according to some heuristics (e.g. presumably when there's no input but perhaps a single WM_USER message posted later than another thread was readied). MessagePumpForUI doesn't intend to give this opportunity to the kernel when invoking ::PeekMessage however as it runs most tasks out-of-band. Hence, PM_NOYIELD should be used to tell ::PeekMessage it's not the only source of work for this thread. R=fdoray@chromium.org Bug: 1075960 Change-Id: Ic8808cab8c15aa690f7dfd8405741323285c107b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169728 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#763349}
-
Alexander Hendrich authored
These all seem to be related to each other: ProfileHelperTest.DeleteActiveProfile (WIN) ProfileHelperTest.DeleteSoleProfile (WIN) ProfileManagerBrowserTest.DeleteAllProfiles (WIN) ProfileManagerBrowserTest.DeleteCurrentProfile (WIN + LINUX) ProfileManagerBrowserTest.DeleteSingletonProfile (WIN) TBR=droger@chromium.org,rhalavati@chromium.org Bug: 1073451 Change-Id: Id3817808337330664a04210ed6a3a1c94639ac1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169104Reviewed-by:
Alexander Hendrich <hendrich@chromium.org> Commit-Queue: Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#763348}
-
Victor Hugo Vianna Silva authored
Bug: None Change-Id: I9d6c7023c3ec0bd016a84606fda509a3f56e6496 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169990Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Victor Vianna <victorvianna@google.com> Cr-Commit-Position: refs/heads/master@{#763347}
-
Rohit Agarwal authored
URL navigation in incognito with intent based scheme "intent://" currently results in one of these behaviour: 1) Launches an incognito intent with a user dialog that requires permission to leave incognito. This is shown only if chrome can't handle the resultant intent via willChromeHandleIntent method. The method willChromeHandleIntent returns true, if target intent has the same package name as chrome; if not then android is asked to resolve this check; the method returns false otherwise. |OVERRIDE_WITH_ASYNC_ACTION| 2) Default case, launches the resultant intent without any user dialog. |OVERRIDE_WITH_EXTERNAL_INTENT| Behaviour 2 is a bug that leaks user session from incognito without the user dialog. For example, clicking on an intent url that opens custom tabs from incognito doesn't show any dialog. This is because, the package name of intent is that of chrome which fails condition 1. This CL modifies the behaviour in incognito for external navigation from intent based schemes as : a) Same as behaviour 1. b) In case chrome can handle the resulting intent it will continue to do so in the incognito session. c) The default case, i.e behaviour 2, is no longer true for incognito session, we will fall in either a) or b) always. It also adds unit tests and browser test. Bug: 1062548, 1062550 Change-Id: I1dcab96f054e2696c6993a9f11975e436bc12ebe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2148738Reviewed-by:
Rohit Agarwal <roagarwal@chromium.org> Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Commit-Queue: Rohit Agarwal <roagarwal@chromium.org> Cr-Commit-Position: refs/heads/master@{#763346}
-
Xida Chen authored
This CL makes the numbers in the debug string in the FrameSequenceTracker % 1000 such that the debug string won't be too long, and will be easier for debugging. Bug: 1072482 Change-Id: I732eced597a568dc365eafa6d12b8661469ca486 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2166394Reviewed-by:
Robert Flack <flackr@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#763345}
-
Yoshifumi Inoue authored
This patch changes "paste-4039777-fix.html" to utilize selection_test() for ease of maintenance and help to implementing EditingNG. Bug: 707656, 679977 Change-Id: I1b16bde5e3b6d3bb4a3d82551dbda7787e7b47c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170007 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Auto-Submit: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#763344}
-
Kush Sinha authored
Extend the expiry of AccountManager.TokenLoadStatus histogram by 1 year. Bug: 1075621 Change-Id: Ia5d7e664c2e4d42098bd24608326f9d2f9363bc2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169893 Commit-Queue: Kush Sinha <sinhak@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#763343}
-
Alice Wang authored
This CL adds a render test for AccountPickerDialogFragment.updateSelectedAccount to check that this call updates the account selection mark. Bug: 967374 Change-Id: I0ebd2abfa0f7782daa3b2690101271af65dc1f0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2166904Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Commit-Queue: Alice Wang <aliceywang@chromium.org> Cr-Commit-Position: refs/heads/master@{#763342}
-
Vincent Boisselle authored
Change-Id: Ib2f8514e4c1c54cf92bb25c66109fbe3a243564f Bug: 1075680 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2145249 Commit-Queue: Vincent Boisselle <vincb@google.com> Reviewed-by:
Dan H <harringtond@chromium.org> Reviewed-by:
-
Alex Chau authored
Bug: 1074512 Change-Id: Ic7d1436b78fb5786cb451a33e4c7d0448fd13721 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2167761 Commit-Queue: Alex Chau <alexchau@chromium.org> Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Cr-Commit-Position: refs/heads/master@{#763340}
-
Alex Gough authored
This is a reland of 0823e911 Plus: missed the nacl-broker process type (used for x86 on x64 nacl). This previously reported kInvalid as a sandbox type, and acted as if it had no sandbox, so now has kNoSandbox following components/nacl/browser/nacl_broker_host_win.cc:36. TBR'ing wez & jam as those parts are unchanged. Original change's description: > Exchange SandboxType::kInvalid for a CHECK. > > No processes should be launched with an invalid combination of flags for > sandboxing. This eliminates SandboxType::kInvalid and adds a CHECK at > the point where the command line is parsed when child processes start. > > Unit tests exist for this code and continue to pass. Tests that > verified that bad command lines were recognized are now removed as > Chrome will safely CHECK in these cases. > > The following changes in default or behavior:- > > * cloud print service (--type=service) now reports kNoSandbox > * NaCl loader process on non-Mac now reports kUtility > - The NaCl loader defines its own sandbox but its command line > is processed in some tests and needs a valid value. > * Fuchsia: sandbox types are now enumerated and the type must be > specified when the low level sandbox policy is constructed. > * Linux: type added for Zygote to have before it grows up to get a > real sandbox > > Bug: 1065087 > Change-Id: Ibe277153fa58771d12bae7e3c2f9c6b92b9370a4 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120049 > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Reviewed-by: Wez <wez@chromium.org> > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Commit-Queue: Alex Gough <ajgo@chromium.org> > Cr-Commit-Position: refs/heads/master@{#762525} TBR=wez,jam Bug: 1065087 Change-Id: Ic66f04e7ac05de694d16eff7af3387d333d8149c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2167995Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Alex Gough <ajgo@chromium.org> Cr-Commit-Position: refs/heads/master@{#763339}
-
Colin Blundell authored
This CL adapts //chrome's translate_manager_browsertest.cc tests of several error conditions to //weblayer. There are more that can be adapted, but this seemed like a reasonable initial cutpoint. Bug: 1025620 Change-Id: Ia1c79c23e2193bf5f35a75a5be1fccbc42bb5a1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2161017 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Cr-Commit-Position: refs/heads/master@{#763338}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/df658be0297c..3623f72383d9 Created with: gclient setdep -r src-internal@3623f72383d9 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 yukishiino@google.com,hendrich@google.com,tommycli@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.chrome.try:linux-chromeos-chrome Bug: chromium:1019685 Tbr: yukishiino@google.com,hendrich@google.com,tommycli@google.com Change-Id: Id4d22619f55168ce1c26c142455620e4cfed2905 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170249Reviewed-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@{#763337}
-
Robert Sesek authored
Firefox has not stored search engine data in XML files in the past five years (https://bugzilla.mozilla.org/show_bug.cgi?id=1203167). Something about the search engines importer is causing crashes, but with no tests and no good way to get an old-style profile to test with, fixing that bug is difficult. Just remove the code instead, since it has bitrotted and it's unlikely that anyone is importing a 3+ year-old Firefox profile that has not been migrated to the new storage scheme. Firefox now stores search engines in an LZ4-compressed JSON file called search.json.mozlz4, which would be the thing to read if Chromium wanted to re-add support for importing Firefox search engines. Bug: 1073905 Change-Id: Ib1643d64407a550805105908df40075d85cfa118 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2163872Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#763336}
-
Javier Fernandez authored
This CL defines new performance tests to evaluate 2 different cases. One is measuring how efficient we implement the propagation of a custom property when its declaration changes. The other test evaluates the performance of a case that apparently forces a style recalc which implies resolving all the variable references. Change-Id: Ib4576670ee9914aca8d1eea9eac3e085493350ca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2153124Reviewed-by:
Anders Hartvoll Ruud <andruud@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Javier Fernandez <jfernandez@igalia.com> Cr-Commit-Position: refs/heads/master@{#763335}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/bed1eba707ac..03fcfdd74f97 git log bed1eba707ac..03fcfdd74f97 --date=short --first-parent --format='%ad %ae %s' 2020-04-28 eseckler@google.com processor: Fix a potential use-after-free Created with: gclient setdep -r src/third_party/perfetto@03fcfdd74f97 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 Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I31a9ecf51781e592dde256e59e19af3c5362b017 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170248Reviewed-by:
-
Lily Chen authored
Was left over from previous cleanup: https://crrev.com/c/1287209 Now confusing due to newly added kEnableExperimentalCookieFeatures flag. Bug: 840359 Change-Id: If7342f18acec92307292fde1948baa5bfbefc450 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169575 Commit-Queue: Matt Menke <mmenke@chromium.org> Auto-Submit: Lily Chen <chlily@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#763333}
-
Aaron Leventhal authored
Also adds stubbed out support for ATK for follow-up work. Bug: 1074380 Change-Id: Ifeaf86aa1581c5e374e09edfe4ddea89c63aa084 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2165568 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#763332}
-
Alexander Hendrich authored
This reverts commit b28a9077. Reason for revert: consistent test failures on linux TSan bots (see https://crbug.com/1075983) Original change's description: > heap: Whitelist bailout for Vectors > > This CL introduces VectorTraits<T>::kCanTraceConcurrently which serves > as a whitelist for skipping the bailout in Vector/Deque concurrent > marking. > Default is set to false. Only Member nad Persistent types are currently > set to true. > > Bug: 986235 > Change-Id: Ibbddde09221bc9ac29366c1538a13a03ce84672c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2148741 > Commit-Queue: Omer Katz <omerkatz@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > Cr-Commit-Position: refs/heads/master@{#763302} TBR=haraken@chromium.org,mlippautz@chromium.org,omerkatz@chromium.org Change-Id: Idcc8a3c39b4ae7d6410a4da331d123bd20a84d12 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 986235 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2169105Reviewed-by:
-
Ian Clelland authored
This implements the change in https://github.com/httpwg/http-extensions/issues/1083, where dictionary members are specified to default to boolean True if their value is omitted from headers, and are required to be omitted from serialization if their value is True, even if they have associated parameters. This allows dictionary headers of the form Header: key;param=value where 'key' will be assigned the value 'true' by default. Bug: 1069784 Change-Id: I1d08ef812bf7316fd69b36094e4ea932ac367be2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2091712Reviewed-by:
Asanka Herath <asanka@chromium.org> Commit-Queue: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#763330}
-
Mike Wittman authored
The performance bug on Trend Micro's side that necessitated disabling the profiler is reported to be fixed. So we can remove the workaround that disables the profiler. This is an effective revert of 7ce02641. Bug: 1018291 Change-Id: Id45265b8992f547bc0b68da7b65ffebc06d8e704 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2168796 Auto-Submit: Mike Wittman <wittman@chromium.org> Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
Etienne Pierre-Doray <etiennep@chromium.org> Cr-Commit-Position: refs/heads/master@{#763329}
-
