Revive tests for Document.createAttributeNS() and Element.setAttributeNodeNS()

Revive tests for Document.createAttributeNS() and Element.setAttributeNodeNS()

This brings back some tests removed in
https://codereview.chromium.org/158593003 and
https://codereview.chromium.org/143173003

They modify some of the same tests, so reverting together is simpler.

BUG=341296, 341830
R=pdr@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185721

[philipj_slow, 2014-11-20 15:44:40]

philipj@opera.com changed reviewers:
+ pdr@chromium.org

[philipj_slow, 2014-11-20 15:44:42]

PTAL. In some tests the use of these functions is incidental and can be avoided,
but I wasn't too picky and reverted rather blindly to get more test coverage.

[philipj_slow, 2014-11-20 19:30:05]

deprecation messages

[pdr., 2014-11-20 19:30:39]

On 2014/11/20 at 15:44:42, philipj wrote:
> PTAL. In some tests the use of these functions is incidental and can be
avoided, but I wasn't too picky and reverted rather blindly to get more test
coverage.

LGTM. This change is large but I skimmed it all and things look good. RS=me. It
looks like there are a few real failures. Feel free to fix them and land without
a second review.

[philipj_slow, 2014-11-20 19:33:53]

On 2014/11/20 19:30:39, pdr wrote:
> On 2014/11/20 at 15:44:42, philipj wrote:
> > PTAL. In some tests the use of these functions is incidental and can be
> avoided, but I wasn't too picky and reverted rather blindly to get more test
> coverage.
> 
> LGTM. This change is large but I skimmed it all and things look good. RS=me.
It
> looks like there are a few real failures. Feel free to fix them and land
without
> a second review.

Yeah, those were just missing console messages for the deprecation.

It was a plain revert with only 	LayoutTests/http/tests/security/javascriptURL/*
excluded, because those test expectations were no longer correct and the
setAttributeNode should cover the same thing.

[philipj_slow, 2014-11-20 19:34:01]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-11-20 19:35:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/740223003/20001

[commit-bot: I haz the power, 2014-11-20 20:41:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-20 20:41:25]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/2...)

[philipj_slow, 2014-11-20 22:20:43]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-11-20 22:21:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/740223003/20001

[commit-bot: I haz the power, 2014-11-20 22:45:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-20 22:45:38]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/2...)

[philipj_slow, 2014-11-20 23:17:35]

Presubmit will not forgive the tabs and trailing whitespace in these tests, I'll
have to dcommit them. (Not worth cleaning them up IMHO.)

[philipj_slow, 2014-11-20 23:27:50]

Committed patchset #2 (id:20001) manually as 185721 (presubmit successful).