Skip to content

Commit

Permalink
Update to v9.3.12, Revert "Link in vk.com profile doesn't work (close #…
Browse files Browse the repository at this point in the history
…846) (#854)" (#867)

* Revert "Link in vk.com profile doesn't work (close #846) (#854)"

This reverts commit 8f0578e.

* Update to v9.3.12
  • Loading branch information
churkin authored Oct 17, 2016
1 parent 776ddb0 commit dc0ccbe
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "testcafe-hammerhead",
"description": "A powerful web-proxy used as a core for the TestCafe testing framework (https://github.com/DevExpress/testcafe).",
"version": "9.3.11",
"version": "9.3.12",
"homepage": "https://github.com/DevExpress/testcafe-hammerhead",
"bugs": {
"url": "https://github.com/DevExpress/testcafe-hammerhead/issues"
Expand Down
2 changes: 1 addition & 1 deletion src/client/sandbox/node/element.js
Original file line number Diff line number Diff line change
Expand Up @@ -578,7 +578,7 @@ export default class ElementSandbox extends SandboxBase {
var target = this._getEffectiveTargetValue(el);

if (!ElementSandbox._isKeywordTarget(target) && !windowsStorage.findByName(target))
el.target = '_top';
el.target = '_self';

}

Expand Down
2 changes: 1 addition & 1 deletion src/client/sandbox/node/window.js
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ export default class WindowSandbox extends SandboxBase {
var newArgs = [];

newArgs.push(getProxyUrl(arguments[0]));
newArgs.push('_top');
newArgs.push('_self');

if (arguments.length > 2)
newArgs.push(arguments[2]);
Expand Down
2 changes: 1 addition & 1 deletion src/processing/dom/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@ export default class DomProcessor {
attrValue = attrValue && attrValue.replace(/\s/g, '');

if (attrValue === '_blank' || attrValue === 'blank')
this.adapter.setAttr(el, 'target', '_top');
this.adapter.setAttr(el, 'target', '_self');
}

_processUrlAttrs (el, urlReplacer, pattern) {
Expand Down
2 changes: 1 addition & 1 deletion test/client/fixtures/sandbox/node/document-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,7 @@ asyncTest('multiple document.write with html and body tags should not break mark

strictEqual(doc.querySelector('h1').innerHTML, 'Header');
ok(/Text( text){19}/.test(doc.querySelector('p').innerHTML));
strictEqual(doc.querySelector('a').target, '_top');
strictEqual(doc.querySelector('a').target, '_self');
strictEqual(doc.querySelectorAll('body > table tr > td > a > img').length, 1);

document.body.removeChild(iframe);
Expand Down
12 changes: 6 additions & 6 deletions test/client/fixtures/sandbox/node/dom-processor-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -534,7 +534,7 @@ module('should ensure that target contains the existing window name (GH-247) (GH
link.href = 'http://example.com';
link.target = 'wrong_window_name';
link.addEventListener('click', function (e) {
strictEqual(link.target, '_top');
strictEqual(link.target, '_self');

e.preventDefault();
link.parentNode.removeChild(link);
Expand Down Expand Up @@ -567,7 +567,7 @@ module('should ensure that target contains the existing window name (GH-247) (GH
base.target = 'wrong_window_name';
link.href = 'http://example.com';
link.addEventListener('click', function (e) {
strictEqual(link.target, '_top');
strictEqual(link.target, '_self');

e.preventDefault();
link.parentNode.removeChild(link);
Expand All @@ -586,7 +586,7 @@ module('should ensure that target contains the existing window name (GH-247) (GH
base.target = '_Parent';
link.href = 'http://example.com';
link.addEventListener('click', function (e) {
strictEqual(link.target, '_top');
strictEqual(link.target, '_self');

e.preventDefault();
link.parentNode.removeChild(link);
Expand Down Expand Up @@ -631,7 +631,7 @@ module('should ensure that target contains the existing window name (GH-247) (GH
area.target = 'wrong_window_name';
area.href = 'http://example.com';
area.addEventListener('click', function (e) {
strictEqual(area.target, '_top');
strictEqual(area.target, '_self');

e.preventDefault();
map.parentNode.removeChild(map);
Expand All @@ -655,7 +655,7 @@ module('should ensure that target contains the existing window name (GH-247) (GH
form.appendChild(input);

form.addEventListener('submit', function (e) {
strictEqual(form.target, '_top');
strictEqual(form.target, '_self');

e.preventDefault();
form.parentNode.removeChild(form);
Expand Down Expand Up @@ -684,7 +684,7 @@ module('should ensure that target contains the existing window name (GH-247) (GH
var storedNativeFormSubmit = nativeMethods.formSubmit;

nativeMethods.formSubmit = function () {
strictEqual(form.target, '_top');
strictEqual(form.target, '_self');

nativeMethods.formSubmit = storedNativeFormSubmit;
start();
Expand Down

0 comments on commit dc0ccbe

Please sign in to comment.