Bug 70228 - [Chromium] Web Inspector: don't check for debugger pause details in interactive ui tests
Summary: [Chromium] Web Inspector: don't check for debugger pause details in interacti...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Yury Semikhatsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-17 04:41 PDT by Yury Semikhatsky
Modified: 2011-10-17 06:38 PDT (History)
10 users (show)

See Also:

Attachments
Patch (7.63 KB, patch)
2011-10-17 04:43 PDT, Yury Semikhatsky 		pfeldman: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2011-10-17 04:41:03 PDT 
There is a flakiness in Chromium interactive ui tests caused by debugger pause details validation. That functionality is well covered by the layout tests and there is no need in checking it again there.

See e.g.
http://build.chromium.org/p/chromium/builders/Linux%20Tests%20%28Views%20dbg%29%281%29/builds/6874/steps/interactive_ui_tests/logs/PauseInSharedWorkerInitialization

chrome/browser/debugger/devtools_sanity_unittest.cc:99: Failure
Value of: result
  Actual: "[FAILED] Unexpected execution line text.(Expected: 'debugger;', but was '')"
Expected: "[OK]"
Comment 1 Yury Semikhatsky 2011-10-17 04:43:18 PDT 
Created attachment 111244 [details]
Patch
Comment 2 Yury Semikhatsky 2011-10-17 06:38:27 PDT 
Committed r97613: <http://trac.webkit.org/changeset/97613>