[views_vtab] only pay attention to top_time if selection is the same

Related to #625

Author: tstack

src/views_vtab.cc

@@ -610,7 +610,9 @@ CREATE TABLE lnav_db.lnav_views (
             if (!tc.is_selectable()) {
                 selection = top_row;
             }
-        } else if (top_time != nullptr && time_source != nullptr) {
+        } else if (top_time != nullptr && time_source != nullptr
+                   && selection == tc.get_selection())
+        {
             auto sel = tc.get_selection();
             date_time_scanner dts;
             timeval tv;
@@ -623,7 +625,6 @@ CREATE TABLE lnav_db.lnav_views (
 
                 if (last_ri_opt) {
                     auto last_time = last_ri_opt->ri_time;
-                    last_time.tv_usec = rounddown(last_time.tv_usec, 1000);
                     if (tv != last_time) {
                         time_source->row_for_time(tv) |
                             [&tc, &selection](auto row) {
@@ -744,7 +745,7 @@ CREATE TABLE lnav_db.lnav_views (
             auto wrap_words = vo.vo_word_wrap.value() == word_wrap_t::normal;
             tc.set_word_wrap(wrap_words);
             if (wrap_words) {
-                left = 0; // reset horizontal scroll position
+                left = 0;  // reset horizontal scroll position
             }
         }
         if (vo.vo_hidden_fields) {

test/expected/Makefile.am

@@ -1402,6 +1402,8 @@ dist_noinst_DATA = \
     test_sql_views_vtab.sh_2180f813a15d33b5aa1f96a2625534c329d0df47.out \
     test_sql_views_vtab.sh_28e23f4e98b1acd6478e39844fd9306b444550c3.err \
     test_sql_views_vtab.sh_28e23f4e98b1acd6478e39844fd9306b444550c3.out \
+    test_sql_views_vtab.sh_2e9cf255d69c1bc29ead5db05a67d82c10adf60d.err \
+    test_sql_views_vtab.sh_2e9cf255d69c1bc29ead5db05a67d82c10adf60d.out \
     test_sql_views_vtab.sh_32acc1a8bb5028636fdbf08f077f9a835ab51bec.err \
     test_sql_views_vtab.sh_32acc1a8bb5028636fdbf08f077f9a835ab51bec.out \
     test_sql_views_vtab.sh_362e2302a2ccec4438d822a909c04f810d6d86f6.err \

test/expected/test_sql_views_vtab.sh_2e9cf255d69c1bc29ead5db05a67d82c10adf60d.err

@@ -0,0 +1,10 @@
+  bytes_read: 0
+  duration: 0.000724
+2024-05-26T11:44:00.654197Z 10.10.10.132  "GET mirrors.tnonline.net:400/gentoo/gentoo-distfiles/distfiles/b6/libvirt-python-9.9.0.tar.gz HTTP/2.0" 200 246357 handled request
+  request/headers/User-Agent#: Mozilla/5.0 AppleWebKit/537.36 (KHTML, like Gecko; compatible; bingbot/2.0; +http://www.bing.com/bingbot.htm) Chrome/116.0.1938.76 Safari/537.36
+  bytes_read: 0
+  duration: 0.024627
+2024-05-26T14:16:03.414651Z 240e:1:1::1234  "GET mirrors.tnonline.net/res/browse.css HTTP/3.0" 200 1577 handled request
+  request/headers/User-Agent#: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/117.0.5938.60 Safari/537.36
+  bytes_read: 0
+  duration: 0.000276

test/expected/test_sql_views_vtab.sh_2e9cf255d69c1bc29ead5db05a67d82c10adf60d.out

@@ -4,6 +4,11 @@ export TZ=UTC
 export YES_COLOR=1
 unset XDG_CONFIG_HOME
 
+run_cap_test ${lnav_test} -n \
+    -c ";UPDATE lnav_top_view SET selection=6" \
+    -c ";UPDATE lnav_top_view SET selection=6" \
+    ${test_dir}/logfile_caddy_log.1
+
 run_test ${lnav_test} -n \
     -c ";SELECT view_name,basename(filepath),visible FROM lnav_view_files" \
     -c ":write-csv-to -" \