Add 'genhtml_frames' to lcovrc config file - corresponds to

'genhtml --frames'
Clarify implications of chrome/firefox bug when using 'frames'.

Signed-off-by:  Henry Cox <[email protected]>

Author: henry2cox

bin/genhtml

@@ -5257,6 +5257,7 @@ my %genhtml_rc_opts = (
                "genhtml_no_source"          => \$no_sourceview,
                "genhtml_num_spaces"         => \$tab_size,
                "genhtml_highlight"          => \$highlight,
+               "genhtml_frames"             => \$frames,
                "genhtml_legend"             => \$legend,
                "genhtml_html_prolog"        => \$html_prolog_file,
                "genhtml_html_epilog"        => \$html_epilog_file,
@@ -5359,6 +5360,7 @@ $fn_hi_limit  = $hi_limit if (!defined($fn_hi_limit));
 $fn_med_limit = $med_limit if (!defined($fn_med_limit));
 $br_hi_limit  = $hi_limit if (!defined($br_hi_limit));
 $br_med_limit = $med_limit if (!defined($br_med_limit));
+$frames       = undef unless (defined($frames) && $frames);
 
 @datebins                   = @rc_date_bins unless @datebins;
 @SourceFile::annotateScript = @rc_annotate_script

man/genhtml.1

@@ -644,6 +644,21 @@ code character has to be inspected once. Note also that the GD.pm Perl
 module has to be installed for this option to work (it may be obtained
 from http://www.cpan.org).
 
+This option can also be controlled from the
+.I genhtml_frames
+entry of the
+.B lcovrc
+file.
+
+Please note that there is a bug in firefox and in chrome, such that
+enabling frames will disable hyperlinks from the 'directory' level summary
+table entry to the first line in the corresponding file in the particular
+category - e.g., to the first 'MIS' line (vanilla coverage report - see the
+.i \-\-show\-navigation
+option, below), to
+the first 'UNC' branch (differential coverage repot), etc.
+Hyperlinks from the summary table at the top of the 'source detail' page are not affected.
+
 .RE
 .B \-s
 .br
@@ -1446,6 +1461,13 @@ Hyperlinks are always generated for differential coverage reports.
 
 This feature enables developers to find and understand coverage issues more quickly than they might otherwise, if they had to rely on scrolling.
 
+See the
+.I \-\-frames
+description above for a description of a browser bug which disables
+these hyperlinks in certain conditions.
+
+Navigation hyperlinks are always enabled in differential coveage report.
+
 .RE
 .BI "\-\-show\-proportions "
 .RS

man/lcovrc.5

@@ -855,6 +855,17 @@ age of the corresponding block (as reported by your annotation script).   This o
 Default is 5.
 .PP
 
+.BR genhtml_frames " ="
+.I 0 | 1
+.IP
+Specify whether source detail view should contain a navigation image.
+See the
+.I \-\-frame
+entry in the 
+.B genhtml
+man page.
+.PP
+
 .BR genhtml_overview_width " ="
 .I pixel_size
 .IP