Improve <adview> implementation and add tests.

chrome/test/data/extensions/platform_apps/ad_view/src_not_exposed/chrometest.js

Index: chrome/test/data/extensions/platform_apps/ad_view/src_not_exposed/chrometest.js
diff --git a/chrome/test/data/extensions/platform_apps/ad_view/src_not_exposed/chrometest.js b/chrome/test/data/extensions/platform_apps/ad_view/src_not_exposed/chrometest.js
new file mode 100644
index 0000000000000000000000000000000000000000..767ec6eefb397bc2aefaee39cc80315a893aec6e
--- /dev/null
+++ b/chrome/test/data/extensions/platform_apps/ad_view/src_not_exposed/chrometest.js
@@ -0,0 +1,26 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+function runTests() {

[benwells, 2013/04/03 02:40:03]

Please add an initial comment here like in all the other test js files.

[rpaquay, 2013/04/03 18:30:05]

I am slightly confused about where to put comments :). I thought you asked me to
move the comments back to the ".cc" file. Did you mean to duplicate each comment
in both the ".cc" file and in the ".js" files?

[benwells, 2013/04/04 04:33:52]

Yes sorry that was what I meant. They are really useful in both places.

[rpaquay, 2013/04/05 16:12:08]

Done.

+  chrome.test.runTests([
+    function test() {
+      var adview = document.getElementsByTagName("adview")[0];
+      adview.addEventListener("loadcommit", function(e){
+        chrome.test.assertEq(false, adview.hasAttribute("src"));
+        chrome.test.assertEq(null, adview.getAttribute("src"));
+        chrome.test.assertEq(undefined, adview['src']);
+        chrome.test.assertEq(undefined, adview.src);
+        console.log("Properties and attributes are inactive.");
+        chrome.test.succeed();
+      });
+      adview.setAttribute('ad-network', 'admob');
+    }
+  ]);
+}
+
+window.onload = function() {
+  chrome.test.getConfig(function(config) {
+    runTests();
+  });
+}