[IMP] website: perf - prefetch menus when serving a page

If we are rendering a page, the menus will most likely be rendered as well.
Note that even in a 403 case when the page is found but is not visible, the
menus will also be shown on the 403 page.

Fetching the menus before accessing the requested page will prefetch that page
as well in one go if that page is in the menus, without any costs for the pages
not in the menus.

There is a tradeoff for 'non-layout' pages, which only occurs in advanced
technical cases:
1. Create a page with specific extension as name suffix (page.css) in which
   case the page will be bootstraped accordingly, without call to layout.
2. Remove the call to layout in HTML editor or backend
3. Remove the call to submenus template in HTML editor or backend
For those cases, the menus will be prefetched for no reason.

Note that homepage '/' is rendered through a controller, same logic is applied
there.

task-2211013

1 files changed, 5 insertions, 5 deletions

diff --git a/addons/website/tests/test_performance.py b/addons/website/tests/test_performance.py
index c257de62b88..92e1616f29a 100644
--- a/addons/website/tests/test_performance.py
+++ b/addons/website/tests/test_performance.py
@@ -77,25 +77,25 @@ class TestWebsitePerformance(UtilPerf):
 
     def test_10_perf_sql_queries_page(self):
         # standard untracked website.page
-        self.assertEqual(self._get_url_hot_query(self.page.url), 14)
+        self.assertEqual(self._get_url_hot_query(self.page.url), 12)
         self.menu.unlink()
         self.assertEqual(self._get_url_hot_query(self.page.url), 14)
 
     def test_15_perf_sql_queries_page(self):
         # standard tracked website.page
         self.page.track = True
-        self.assertEqual(self._get_url_hot_query(self.page.url), 22)
+        self.assertEqual(self._get_url_hot_query(self.page.url), 20)
         self.menu.unlink()
         self.assertEqual(self._get_url_hot_query(self.page.url), 22)
 
     def test_20_perf_sql_queries_homepage(self):
         # homepage "/" has its own controller
-        self.assertEqual(self._get_url_hot_query('/'), 21)
+        self.assertEqual(self._get_url_hot_query('/'), 19)
 
     def test_30_perf_sql_queries_page_no_layout(self):
         # website.page with no call to layout templates
         self.page.arch = '<div>I am a blank page</div>'
-        self.assertEqual(self._get_url_hot_query(self.page.url), 8)
+        self.assertEqual(self._get_url_hot_query(self.page.url), 10)
 
     def test_40_perf_sql_queries_page_multi_level_menu(self):
         # menu structure should not impact SQL requests
@@ -113,7 +113,7 @@ class TestWebsitePerformance(UtilPerf):
         menu_bb.parent_id = menu_b
         menu_aa.parent_id = menu_a
 
-        self.assertEqual(self._get_url_hot_query(self.page.url), 14)
+        self.assertEqual(self._get_url_hot_query(self.page.url), 12)
 
     def test_50_perf_sql_web_content(self):
         # assets route /web/content/..