Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

html/selectelement_test times out on buildbot but completes quickly locally #15516

Closed
peter-ahe-google opened this issue Dec 8, 2013 · 4 comments
Labels
area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. closed-obsolete Closed as the reported issue is no longer relevant type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@peter-ahe-google
Copy link
Contributor

(view as text)
@@@BUILD_STEP dart2js-chrome tests failures@@@

FAILED: dart2js-chrome release_ia32 html/selectelement_test
Expected: Pass
Actual: Timeout
CommandOutput[dart2js]:

stdout:
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:26708:1: Warning: 'Window' doesn't implement 'clearInterval'.
Try adding an implementation of 'clearInterval'.
class Window extends EventTarget implements WindowBase, _WindowTimers, WindowBase64 native "Window,DOMWindow" {
^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:30378:8: Info: This is the method declaration.
  void clearInterval(int handle);
       ^^^^^^^^^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:26708:1: Warning: 'Window' doesn't implement 'clearTimeout'.
Try adding an implementation of 'clearTimeout'.
class Window extends EventTarget implements WindowBase, _WindowTimers, WindowBase64 native "Window,DOMWindow" {
^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:30380:8: Info: This is the method declaration.
  void clearTimeout(int handle);
       ^^^^^^^^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:26708:1: Warning: 'Window' doesn't implement 'setInterval'.
Try adding an implementation of 'setInterval'.
class Window extends EventTarget implements WindowBase, _WindowTimers, WindowBase64 native "Window,DOMWindow" {
^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:30382:7: Info: This is the method declaration.
  int setInterval(Object handler, int timeout);
      ^^^^^^^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:26708:1: Warning: 'Window' doesn't implement 'setTimeout'.
Try adding an implementation of 'setTimeout'.
class Window extends EventTarget implements WindowBase, _WindowTimers, WindowBase64 native "Window,DOMWindow" {
^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:30384:7: Info: This is the method declaration.
  int setTimeout(Object handler, int timeout);
      ^^^^^^^^^^
build/ReleaseIA32/dart-sdk/lib/html/dart2js/html_dart2js.dart:30512:1: Warning: '_ElementAttributeMap' doesn't implement 'addAll'.
Try adding an implementation of 'addAll'.
class _ElementAttributeMap extends _AttributeMap {
^^^^^
build/ReleaseIA32/dart-sdk/lib/core/map.dart:138:8: Info: This is the method declaration.
  void addAll(Map<K, V> other);
       ^^^^^^

CommandOutput[chrome]:

stdout:
message:
  Dom could not be fetched, since the test timed out.

stderr:
This test timed out. The delay until the test actually started was: 0:00:00.020002.

BrowserOutput while running the test (* EXPERIMENTAL *):
BrowserOutput.stdout:
  
BrowserOutput.stderr:
  

To retest, run: e:\b\build\slave\dart2js-chrome-win7-be\build\dart\tools\testing\bin\windows\dart.exe /e:/b/build/slave/dart2js-chrome-win7-be/build/dart/tools/testing/dart/http_server.dart -p 51010 -c 51011 --build-directory=e:\b\build\slave\dart2js-chrome-win7-be\build\dart\build\ReleaseIA32 --runtime=chrome

Command[dart2js]: build\ReleaseIA32\dart-sdk\bin\dart2js.bat --package-root=build/ReleaseIA32/packages/ --out=e:/b/build/slave/dart2js-chrome-win7-be/build/dart/build/ReleaseIA32/generated_compilations/dart2js/tests_html_selectelement_test/test.js /e:/b/build/slave/dart2js-chrome-win7-be/build/dart/tests/html/selectelement_test.dart
Took 0:00:09.522000

Command[chrome]: \e:\b\build\slave\dart2js-chrome-win7-be\build\dart\tools\testing\bin\windows\dart.exe tools/testing/dart/launch_browser.dart chrome http://127.0.0.1:51010/root_build/generated_tests/dart2js-chrome/tests_html_selectelement_test/test.html?crossOriginPort=51011
Took 0:01:00.020001

Short reproduction command (experimental):
    python tools/test.py -mrelease -cdart2js -rchrome --use-sdk --write-debug-log --write-test-outcome-log -t60 html/selectelement_test

===
=== 1 test failed
===

--- Total time: 42:01 ---
0:01:00.020001 - chrome - dart2js-chrome release_ia32/html/selectelement_test
0:00:26.310631 - chrome - dart2js-chrome release_ia32/lib/convert/streamed_conversion_utf8_decode_test
0:00:26.219622 - chrome - dart2js-chrome release_ia32/lib/convert/streamed_conversion_utf8_encode_test
0:00:20.071000 - dart2js - dart2js-chrome release_ia32/lib/mirrors/native_class_test
0:00:14.752000 - dart2js - dart2js-chrome release_ia32/samples/third_party/todomvc/test/listorder_test
0:00:14.530000 - dart2js - dart2js-chrome release_ia32/samples/third_party/todomvc/test/mainpage_test
0:00:14.470000 - dart2js - dart2js-chrome release_ia32/samples/third_party/todomvc/test/markdone_test
0:00:14.469000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/publish_inherited_properties_test
0:00:14.337000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/unbind_test
0:00:14.306000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/bind_test
0:00:14.285000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/property_change_test
0:00:14.222000 - dart2js - dart2js-chrome release_ia32/samples/third_party/todomvc/test/markdone_shadow_test
0:00:14.109000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/event_path_declarative_test
0:00:14.104000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/prop_attr_bind_reflection_test
0:00:14.100000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/prop_attr_reflection_test
0:00:14.086000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/noscript_test
0:00:14.075000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/example/canonicalization/test/canonicalization_deploy_test
0:00:14.043000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/register_test
0:00:13.995000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/publish_attributes_test
0:00:13.983000 - dart2js - dart2js-chrome release_ia32/pkg/polymer/test/nested_binding_test
@@@STEP_FAILURE@@@


started: Sun Dec 8 03:14:42 2013
ended: Sun Dec 8 03:14:42 2013
duration: 0 secs

@peter-ahe-google
Copy link
Contributor Author

Could be related to issue #15497.

@mkustermann
Copy link
Member

I don't think this is related to issue #15497.

This is dart2js-chrome. The environment variable mentioned in issue #15497 has only an effect in dartium (it was introduced in the dartium bindings layer and is not available in chrome).

@ricowind
Copy link
Contributor

ricowind commented Jan 8, 2014

I am reenabling this test. We added extra logging functionality to the browser controller. In this specific case it may very well be that the browser actually crashed (which we could have seen in the debug log)

Peter: do you remember if this was something that you saw more than once.

@peter-ahe-google
Copy link
Contributor Author

I think comment #­2 is incorrect and this was a dupe of issue #7.

@peter-ahe-google peter-ahe-google added Type-Defect area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. labels Jan 9, 2014
@kevmoo kevmoo added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed priority-unassigned labels Feb 29, 2016
@matanlurey matanlurey added the closed-obsolete Closed as the reported issue is no longer relevant label Jun 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. closed-obsolete Closed as the reported issue is no longer relevant type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

5 participants