acmh / webpagetest

Automatically exported from code.google.com/p/webpagetest
0 stars 0 forks source link

SetDomElement not working in Chrome/Firefox #138

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
http://www.webpagetest.org/forums/showthread.php?tid=12004&pid=20083#pid20083

Original issue reported on code.google.com by PatMee...@gmail.com on 28 Jan 2013 at 3:16

GoogleCodeExporter commented 9 years ago
Here's a suggestion to fix this issue for Firefox:
add "continue_processing = false;" in wpt_test.cc, line 569: 
<code> 
} else if (cmd == _T("setdomelement")) {
    if (command.target.Trim().GetLength()) {
      _dom_element_check = true;
      continue_processing = false;
      WptTrace(loglevel::kFrequentEvent, 
        _T("[wpthook] - WptTest::BuildScript() Setting dom element check."));
    }
    consumed = false;
</code>
This change implies sending "setDomElement" to the "WptDriver"-Firefox 
Extension, which is able to handle it.

Original comment by switaj...@googlemail.com on 23 Apr 2013 at 9:13

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r1846.

Original comment by PatMee...@gmail.com on 24 Apr 2013 at 2:24