- 27 Feb, 2019 40 commits
-
-
Yuli Huang authored
BUG=b:117816926 TEST=Tested by taking pictures and checking GA traffic. Change-Id: I984a0cd69d9977f2b3faf2b3054c1156340e268c Reviewed-on: https://chromium-review.googlesource.com/c/1484876Reviewed-by:
Sheng-hao Tsao <shenghao@google.com> Commit-Queue: yuli <yuli@chromium.org> Cr-Commit-Position: refs/heads/master@{#635865}
-
Nigel Tao authored
BUG=826982 Change-Id: I5bee3696fe24dd611ab51a6bf82e974373d55165 Reviewed-on: https://chromium-review.googlesource.com/c/1484960Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Nigel Tao <nigeltao@chromium.org> Cr-Commit-Position: refs/heads/master@{#635864}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=abdulsyed@chromiue.org Change-Id: Id89db5937272e38fe969e01ea7fbfc61a2c9e59f Reviewed-on: https://chromium-review.googlesource.com/c/1490955Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#635863}
-
Tien Mai authored
This better reflects the use of this class as something used to validate access for users associated with GCPW. Bug: 924520 Change-Id: I78e8b5a72b1a9f4030b016ad9c920d53b2f789e6 Reviewed-on: https://chromium-review.googlesource.com/c/1489103 Commit-Queue: Tien Mai <tienmai@chromium.org> Reviewed-by:
Roger Tawa <rogerta@chromium.org> Cr-Commit-Position: refs/heads/master@{#635862}
-
Martin Kreichgauer authored
Some U2F devices enter an unresponsive state after receiving a CTAP2 GetInfo command. This change checks for these devices by their VID/PID during initialization and forces them into U2F mode. Bug: 935131 Change-Id: I11c18808c272aa06d839ae7bc5ceba351634fe09 Reviewed-on: https://chromium-review.googlesource.com/c/1488718 Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#635861}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d28f8754..9e723d07 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I37d9c422e9f5dfbb100394210ca9fe1f90f91424 Reviewed-on: https://chromium-review.googlesource.com/c/1490741Reviewed-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@{#635860}
-
rbpotter authored
This feature has been enabled by default since M71. Bug: 775999 Change-Id: Ia2a6af706f799fdd770515c04f6df0566335d370 Reviewed-on: https://chromium-review.googlesource.com/c/1487527Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#635859}
-
Dan Beam authored
Screenshot: https://imgur.com/a/k0Y3J7N R=hcarmona@chromium.org BUG=923597 Change-Id: Ib2d782d084cf02eacf68f5296a884a09d0b2be70 Reviewed-on: https://chromium-review.googlesource.com/c/1487882 Auto-Submit: Dan Beam <dbeam@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Dan Beam <dbeam@chromium.org> Cr-Commit-Position: refs/heads/master@{#635858}
-
Dan Beam authored
Screenshots: https://imgur.com/a/hQji9EA Bug: 926658 Change-Id: I7ee67f123e67ea43f4681d14dce89ac55c657a1e Reviewed-on: https://chromium-review.googlesource.com/c/1487190Reviewed-by:
-
Dan Beam authored
Screenshot: https://imgur.com/a/wLZIEi0 R=hcarmona@chromium.org BUG=935238 Change-Id: I080af135e2754b2cc723d276eebbe465f6e2a958 Reviewed-on: https://chromium-review.googlesource.com/c/1488033 Auto-Submit: Dan Beam <dbeam@chromium.org> Reviewed-by:
-
Dan Beam authored
R=hcarmona@chromium.org BUG=923224 Change-Id: I1fe33c412952be065cb8c5794030b1bdef8fd526 Reviewed-on: https://chromium-review.googlesource.com/c/1487842Reviewed-by:
-
Joonghun Park authored
Before this CL, the number specified as font-style's value is just passed not considering its unit type. R=ericwilligers@chromium.org Bug: 889277 Change-Id: Ifb5be52be5342390c27c4d4905f7d5e424bc2023 Reviewed-on: https://chromium-review.googlesource.com/c/1484958 Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#635854}
-
Morten Stenshorne authored
Also get rid of local NGBoxStrut variables for border/padding. This saves 128 bytes [*] of stack space per in-flow layout recursion. [*] Linux 64bit release. Bug: 930637 Change-Id: I2a52f164634d22347282bd0eb08a6a7c2a64b5c0 Reviewed-on: https://chromium-review.googlesource.com/c/1489426 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#635853}
-
Tsuyoshi Horo authored
Bug: 910516 Change-Id: I0768c0f7379778cb78c783a2d3e31ab02163085b Reviewed-on: https://chromium-review.googlesource.com/c/1490359Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Commit-Queue: Tsuyoshi Horo <horo@chromium.org> Cr-Commit-Position: refs/heads/master@{#635852}
-
Sam McNally authored
Bug: 934549 Change-Id: Idd9d1f6c5733e4d993a0bd5d1bc4d33311622e4a Reviewed-on: https://chromium-review.googlesource.com/c/1488394Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Sam McNally <sammc@chromium.org> Auto-Submit: Sam McNally <sammc@chromium.org> Cr-Commit-Position: refs/heads/master@{#635851}
-
Brandon Wylie authored
This allows fetching gifs from java using new api methods. Involved some additions/changes to the customization of CachedImageFetcher: * Allow skipping transcoding option when only fetching image_data. * Pipe up FetchImageData to Java. * Wrap FetchImageData into FetchGif. * Write code to potentially fetch/decode gif from disk in Java. Bug: 919915 Change-Id: I8d825272f67b136e5c0d79bdff236d37e3f70f73 Reviewed-on: https://chromium-review.googlesource.com/c/1474799 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#635850}
-
Abhishek Bhardwaj authored
This is a rework of https://crrev.com/c/1302866 monitoring all wake locks rather than just ARC++ wake locks. This change adds functionality to support lock screen notifications for ARC++, Assistant on Chrome OS and other future clients. It adds a new module that listens to dark resume events from the power manager. It - 1. Starts a timer to check for any current app suspension wake locks. 2. After the timer in 1 expires, if no app suspension wake lock is acquired the power manager is requested to re-suspend the system. If an app suspension wake lock is acquired then another hard timeout timer is set and also an observer to the wake lock being released is set. 3. If the wake lock is released, the system is re-suspended immediately. 4. If the hard timeout in 2 expires and a wake lock is still acquired then the system is re-suspended immediately. 5. If the system transitions to a full resume all dark resume related state and timers are cleared. On top of this change - 1. Changes were added to fix linker errors by declaring static constants in the source file as well. 2. Refactored some functions in the test code. BUG=chromium:898297 TEST=Unit tests and end to end test with Android applications and Assistant. Change-Id: I695a7dc666b26fce00f6b0f5d0d2017f6556f83a Reviewed-on: https://chromium-review.googlesource.com/c/1372477 Commit-Queue: Abhishek Bhardwaj <abhishekbh@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#635849}
-
Charlie Reis authored
BUG=935175 Change-Id: Id4a9198d5dff823bc3d324b9de9bff2ee86dc499 Reviewed-on: https://chromium-review.googlesource.com/c/1488152 Commit-Queue: Charlie Reis <creis@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#635848}
-
Kristi Park authored
Do not use dark mode chips (i.e. Most Visited icons, Vasco, fakebox, etc.) if a background image is applied to the NTP. This includes custom backgrounds and images from themes. In order to keep compatibility with M73 versions and Vasco, we use the negation ":not(.light-chip)" instead of a positive selector such as ".dark-chip". Bug: 928548 Change-Id: I0284e08485810ce3aad29a288ce4ad21042e7567 Reviewed-on: https://chromium-review.googlesource.com/c/1475207Reviewed-by:
Kyle Milka <kmilka@chromium.org> Commit-Queue: Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#635847}
-
Sam McNally authored
https://crrev.com/510729 updated renames outside the directory tree; update renames in the directory tree to match. Also fix a "Redundant double negation" eslint spotted 40 lines away and demanded be fixed. Bug: 934683 Change-Id: I1a915fa4359e87f807ccd0cdc346f3f7b7fba664 Reviewed-on: https://chromium-review.googlesource.com/c/1488478 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
-
Tien Mai authored
- Now when a user needs to reauth their local login privilege is denied and the user is removed from the user list for winlogon through a registry setting. - Sign in of the user through GCPW will unlock the user and allow them to sign in. - These permissions are only revoked during Winlogin to ensure a user's state is not completely broken by accident. - Added a new credential provider filter used mainly to ensure that the privileges for a user that needs a reauth is correctly revoked before any credential providers try to create a credential. - Correctly support "Other User" credential. when it appears. - When a user needs a reauth and an entry is not found in the user array in SetUserArray then an anonymous credential associated to this user is added so that they can sign in with gaia to reauth themselves. Bug: 924520 Change-Id: Iaed674025606dc0319a0c1a6f5cf5d4ff662f824 Reviewed-on: https://chromium-review.googlesource.com/c/1481858 Commit-Queue: Tien Mai <tienmai@chromium.org> Reviewed-by:
-
Meng-Huan Yu authored
In CL:1423978, we add UMA to measure the async dbus call at server side. BUG=920850, b:124719803 Change-Id: Ia767981f0cfee72e056f2088c66b0c88f19a2053 Reviewed-on: https://chromium-review.googlesource.com/c/1477131Reviewed-by:
-
Joshua Peraza authored
The custom browser ptype exists to differentiate single process webview crashes from multi-process webview crashes, but was mistakenly set for child process crashes instead of browser crashes. Bug: 936243 Change-Id: I9e37f42323ae39a7a3fa8b81e71331c08aff35bb Reviewed-on: https://chromium-review.googlesource.com/c/1490891Reviewed-by:
Mark Mentovai <mark@chromium.org> Commit-Queue: Joshua Peraza <jperaza@chromium.org> Cr-Commit-Position: refs/heads/master@{#635843}
-
Esmael El-Moslimany authored
Bug: 935334 Change-Id: Ib82d26cda4b4056da4dfa474a376d226949e72bd Reviewed-on: https://chromium-review.googlesource.com/c/1490396Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#635842}
-
Anand K. Mistry authored
Only "very long" API calls are logged to the user's log. Regular "long" APIs are logged to the drive-internals log. BUG=904658 Change-Id: Ic959174f9927e7fe398706b8270a60c35df82e70 Reviewed-on: https://chromium-review.googlesource.com/c/1488393 Commit-Queue: Anand Mistry <amistry@chromium.org> Reviewed-by:
Sam McNally <sammc@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#635841}
-
Kevin Marshall authored
The ids.txt file was being created without newlines between entries, preventing it from being read correctly. Also enable symbolization for binaries provided in the SDK, now that the symbolizer supports multiple sources of symbols. Bug: 772252 Change-Id: I2c0e179d09a26248f5d0c17c61bda7b2ccd05ee6 Reviewed-on: https://chromium-review.googlesource.com/c/1488576Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#635840}
-
Salvador Guerrero authored
Some leveldb_proto clients attempt to destroy their database when it fails to initialize, this would fail because we needed to have an initialized instance in order to destroy it. This change calls the static method leveldb::DestroyDB which will try to erase the database directory. This will ensure that clients using a DB selector will be able to recover from a failed initialization Renamed wrapper to DB impl on proto_database_impl_unittest to match new class name Bug: 870813 Change-Id: I9cbf4482cde132f68abeecb3b210c289bba0b1b2 Reviewed-on: https://chromium-review.googlesource.com/c/1484086 Commit-Queue: Salvador Guerrero <salg@google.com> Reviewed-by:
ssid <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#635839}
-
Kyle Milka authored
Use the real local_ntp.html instead of local_ntp_browsertest.html. Previous attempts at this were flaky (see bug), this avoids the flakiness by using FocusView instead of ClickOnView to bring focus to the NTP. ClickOnView clicks the center of the page, which in tests sometimes seems to be the fakebox. Bug: 791066 Change-Id: If3d0a10a79ede06297d35ead3f389316d639b5a0 Reviewed-on: https://chromium-review.googlesource.com/c/1468786 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#635838}
-
Robert Liao authored
BUG= Change-Id: I8aa80def6c134c944a29791420a649b183acd763 Reviewed-on: https://chromium-review.googlesource.com/c/1490895 Commit-Queue: Robert Liao <robliao@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Auto-Submit: Robert Liao <robliao@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#635837}
-
Joel Hockey authored
This reverts commit c0c53eb2. Reason for revert: breaks compilation when is_chrome_branded=true OobeBaseTest::kFakeUserEmail is referenced at: https://cs.chromium.org/chromium/src/chrome/browser/chromeos/login/sync_consent_interactive_ui_test.cc?l=141&rcl=95ffa536956a89b861c39e6df1c8670638dd31a0 https://cs.chromium.org/chromium/src/chrome/test/BUILD.gn?l=2010&rcl=80199d3ea8218c1e8568bcc087c6d77f0ebaf94e Original change's description: > cros: Make FakeGaia in OobeBaseTest a mixin > > We may want to just use FakeGaia directly instead of introducing a > mixin, but that can be decided later. For now let's focus on simplifying > all of the test bases. > > Bug: 912704 > Change-Id: Id4a7ab8e0c23343b7e6e59c7b7d2aefc16891515 > Reviewed-on: https://chromium-review.googlesource.com/c/1366895 > Commit-Queue: Jacob Dufault <jdufault@chromium.org> > Reviewed-by: Sergey Poromov <poromov@chromium.org> > Reviewed-by: Denis Kuznetsov <antrim@chromium.org> > Reviewed-by: Alexander Alekseev <alemate@chromium.org> > Cr-Commit-Position: refs/heads/master@{#635236} TBR=antrim@chromium.org,alemate@chromium.org,jdufault@chromium.org,poromov@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 912704 Change-Id: I31882d938465fb8404c90b84379e34aff7354512 Reviewed-on: https://chromium-review.googlesource.com/c/1490353Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#635836}
-
Anastasia Helfinstein authored
This is a pure refactor, with no functional changes. Bug: None Change-Id: Id66cf951a8b060fdb2a0d4458dd48620d28b9d05 Reviewed-on: https://chromium-review.googlesource.com/c/1487873 Commit-Queue: Anastasia Helfinstein <anastasi@google.com> Auto-Submit: Anastasia Helfinstein <anastasi@google.com> Reviewed-by:
Katie Dektar <katie@chromium.org> Cr-Commit-Position: refs/heads/master@{#635835}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/46d32e0fcd84..009696c56169 git log 46d32e0fcd84..009696c56169 --date=short --no-merges --format='%ad %ae %s' 2019-02-26 geofflang@chromium.org Vulkan: Support EGL_ANDROID_image_native_buffer on Android. Created with: gclient setdep -r src/third_party/angle@009696c56169 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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_optional_gpu_tests_rel TBR=ynovikov@chromium.org Change-Id: Id5afbbd8f59361160d1087b4f9a1754de1f2d7dd Reviewed-on: https://chromium-review.googlesource.com/c/1490375Reviewed-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@{#635834}
-
Fernando Serboncini authored
We may have pending mojo messages after GC, so we want to drop the dispatcher as soon as possible. Bug: 929757,913964 Change-Id: I5789bcbb55aada4a74c67a28758f07686f8911c0 Reviewed-on: https://chromium-review.googlesource.com/c/1489175Reviewed-by:
-
Stefan Zager authored
It makes sense to have someone from the rendering team in OWNERS. I'm nominating myself because my code review load is not too heavy, I'm reasonably familiar with the code, and I anticipate working more in this area going forward. R=dcheng,haraken,eae,chrishtr Change-Id: Id3141c2aa100a26afaef1dd07980fbb63dea7945 Reviewed-on: https://chromium-review.googlesource.com/c/1490632Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#635832}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/a502bde5ced6..ee85330e0ea1 git log a502bde5ced6..ee85330e0ea1 --date=short --no-merges --format='%ad %ae %s' 2019-02-26 rsavitski@google.com shared ring fuzzer: clear spinlock field to not make fuzzer's read spin indefinitely. Created with: gclient setdep -r src/third_party/perfetto@ee85330e0ea1 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=perfetto-bugs@google.com Change-Id: I942ffa9f05af2a133dbf3aa33166391d570e67fd Reviewed-on: https://chromium-review.googlesource.com/c/1490674Reviewed-by:
-
kylechar authored
Post[Delayed]TaskWithTraits() takes a OnceCallback. Replace usage of base::Bind(), which produces a RepeatingCallback, with base::BindOnce() when the callback is created as a temporary inside of PostTask(). The following regex was used to find instances that could be replaced: (Post(?:Delayed)?TaskWithTraits)\((?:\n\s*)?FROM_HERE,\n?\s*(\{.*?\n?.*?\}),(?:\n)?\s*base::Bind\( Also replace any usage of base::Passed(&var) with std::move(var) for variables passed to base::BindOnce(). base::Passed() isn't needed for move-only types with OnceCallbacks. This CL was uploaded by git cl split. R=seantopping@chromium.org Bug: 714018 Change-Id: I5d71bfb093f9358dace00438d21cf2d861451399 Reviewed-on: https://chromium-review.googlesource.com/c/1486152 Auto-Submit: kylechar <kylechar@chromium.org> Reviewed-by:Sean Topping <seantopping@chromium.org> Commit-Queue: Sean Topping <seantopping@chromium.org> Cr-Commit-Position: refs/heads/master@{#635830}
-
Piotr Pawliczek authored
A list of enterpise printers is not ready immediately after creation of CupsPrintersManager. It caused a problem for testing procedure since from the level of autotest_private_api we do not know if the final list is ready or there are still some ongoing calculations. A new method called OnEnterprisePrintersInitialized() was added to the observer of CupsPrintersManager to solve this issue. Also whole code related to enterprise printers were refactored, large part of functionality was moved from SyncedPrintersManager to new class EnterprisePrintersProvider. Classes ExternalPrintersPrefBridge and DeviceExternalPrintersSettingsBridge have been merged into one class called CalculatorsPoliciesBinder. BUG=chromium:816996 TEST=Tested on nautilus with the use of CL:1451591 Change-Id: I50745f18beacab213216e5f367db0d023190e9e4 Reviewed-on: https://chromium-review.googlesource.com/c/1220547 Commit-Queue: Piotr Pawliczek <pawliczek@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Sean Kau <skau@chromium.org> Auto-Submit: Piotr Pawliczek <pawliczek@chromium.org> Cr-Commit-Position: refs/heads/master@{#635829}
-
John Rummell authored
When unable to pre-provision origin IDs, it may be due to the device not being able to connect to a provisioning server. Updating the code to watch for network events. When connected to a network, try again to pre-provision origin IDs. BUG=917527 TEST=new unit_tests pass Change-Id: Ia41bbedf543556e59ecd03bef51d989c6c0f7210 Reviewed-on: https://chromium-review.googlesource.com/c/1475005 Commit-Queue: John Rummell <jrummell@chromium.org> Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#635828}
-
Paul Dyson authored
* Add more metrics to UKM event AppListAppLaunch. * Add a new UKM metric AppListAppClickData to send the accumulated click data for an app. On an app launch click, an AppListAppClickData event is sent for the app clicked on and for five other apps chosen at random. Also makes user_activity_ukm_logger_helpers a public_dep so it can be used here. Bug: 899123 Change-Id: Ie1d6f17673b9548ef6823d05a362fea8992126b9 Reviewed-on: https://chromium-review.googlesource.com/c/1460750Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Jia Meng <jiameng@chromium.org> Commit-Queue: Paul Dyson <pdyson@chromium.org> Cr-Commit-Position: refs/heads/master@{#635827}
-
rbpotter authored
Bug: 935625 Change-Id: Ib2e8c1e934f213916b90354686ab42a2da3a078c Reviewed-on: https://chromium-review.googlesource.com/c/1487521Reviewed-by:
-
