Revert "Re-enable elements/styles-disable-inherited_test.ts"

This reverts commit 5a62b4b.

Reason for revert: still flaky in CI

Bug: 440335793
Original change's description:
> Re-enable elements/styles-disable-inherited_test.ts
>
> Bug: 440335793
> Change-Id: I05908c930fe828e75d5c4c10f419bee4ac3a4a08
> Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6874622
> Reviewed-by: Simon Zünd <[email protected]>
> Commit-Queue: Alex Rudenko <[email protected]>

Bug: 440335793
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I1a530ecb59be0414d840a827ed2d4d1e83bd024d
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6883610
Commit-Queue: Rubber Stamper <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Auto-Submit: Alex Rudenko <[email protected]>

Author: OrKoN

front_end/ui/visual_logging/Debugging.ts

@@ -767,16 +767,13 @@ function formatInteraction(e: TestLogEntry): string {
   return '';
 }
 
-function formatVeEvents(events: Array<TestLogEntry|IntuitiveLogEntry|AdHocAnalysisLogEntry>): string {
+function formatVeEvents(events: TestLogEntry[]): string {
   return events
       .map(e => {
         if ('interaction' in e) {
           return formatInteraction(e);
         }
-        if ('impressions' in e) {
-          return formatImpressions(e.impressions);
-        }
-        return JSON.stringify(e);
+        return formatImpressions(e.impressions);
       })
       .join('\n');
 }
@@ -796,7 +793,7 @@ export async function expectVeEvents(expectedEvents: TestLogEntry[]): Promise<vo
       pendingEventExpectation.fail(new Error(
           '\nMissing VE Events:\n' + formatVeEvents(pendingEventExpectation.missingEvents) +
           '\nUnmatched VE Events:\n' + formatVeEvents(pendingEventExpectation.unmatchingEvents) + '\nAll events:\n' +
-          formatVeEvents(veDebugEventsLog)));
+          JSON.stringify(veDebugEventsLog, null, 2)));
     }
   }, EVENT_EXPECTATION_TIMEOUT);
 

test/e2e/helpers/elements-helpers.ts

@@ -1040,10 +1040,8 @@ export const toggleClassesPaneCheckbox =
 
 export const uncheckStylesPaneCheckbox =
     async (checkboxLabel: string, devToolsPage = getBrowserAndPagesWrappers().devToolsPage) => {
+  console.error('uncheckStylesPaneCheckbox', checkboxLabel);
   const initialValue = await getContentOfSelectedNode(devToolsPage);
-  await devToolsPage.hover(`.enabled-button[aria-label="${checkboxLabel}"]`);
-  // To detect an impression, we need to hover for some time.
-  await new Promise(resolve => setTimeout(resolve, 250));
   await devToolsPage.click(`.enabled-button[aria-label="${checkboxLabel}"]`);
   await waitForSelectedNodeChange(initialValue, devToolsPage);
   await expectVeEvents(

test/e2e_non_hosted/elements/styles-disable-inherited_test.ts

@@ -10,8 +10,9 @@ import {
 } from '../../e2e/helpers/elements-helpers.js';
 
 describe('The Elements tab', function() {
-  it(
-      'does not break further style inspection if inherited style property was disabled',
+  // Skip since this test seems to be consistently failing on mac and linux.
+  it.skip(
+      '[crbug.com/440335793] does not break further style inspection if inherited style property was disabled',
       async ({devToolsPage, inspectedPage}) => {
         await inspectedPage.goToResource('elements/styles-disable-inherited.html');
         await expandSelectedNodeRecursively(devToolsPage);