Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(20)

Side by Side Diff: chrome/browser/resources/chromeos/neterror.js

Issue 2503453003: Remove all calls to domAutomationController.setAutomationId.
Patch Set: Fix nacl_browsertest_util.cc Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 function toggleHelpBox() { 5 function toggleHelpBox() {
6 var helpBoxOuter = $('details'); 6 var helpBoxOuter = $('details');
7 helpBoxOuter.classList.toggle('hidden'); 7 helpBoxOuter.classList.toggle('hidden');
8 var detailsButton = $('details-button'); 8 var detailsButton = $('details-button');
9 if (helpBoxOuter.classList.contains('hidden')) { 9 if (helpBoxOuter.classList.contains('hidden')) {
10 detailsButton.innerText = detailsButton.detailsText; 10 detailsButton.innerText = detailsButton.detailsText;
11 } else { 11 } else {
12 detailsButton.innerText = detailsButton.hideDetailsText; 12 detailsButton.innerText = detailsButton.hideDetailsText;
13 } 13 }
14 } 14 }
15 15
16 function diagnoseErrors() { 16 function diagnoseErrors() {
17 window.domAutomationController.setAutomationId(1);
18 window.domAutomationController.send('open_connectivity_diagnostics'); 17 window.domAutomationController.send('open_connectivity_diagnostics');
19 } 18 }
20 19
21 // Subframes use a different layout but the same html file. This is to make it 20 // Subframes use a different layout but the same html file. This is to make it
22 // easier to support platforms that load the error page via different 21 // easier to support platforms that load the error page via different
23 // mechanisms (Currently just iOS). 22 // mechanisms (Currently just iOS).
24 if (window.top.location != window.location) 23 if (window.top.location != window.location)
25 document.documentElement.setAttribute('subframe', ''); 24 document.documentElement.setAttribute('subframe', '');
26 25
27 // Re-renders the error page using |strings| as the dictionary of values. 26 // Re-renders the error page using |strings| as the dictionary of values.
(...skipping 18 matching lines...) Expand all
46 if (oldClass !== undefined) 45 if (oldClass !== undefined)
47 classList.remove(oldClass); 46 classList.remove(oldClass);
48 47
49 classList['last_icon_class'] = newClass; 48 classList['last_icon_class'] = newClass;
50 49
51 if (newClass == 'icon-offline') { 50 if (newClass == 'icon-offline') {
52 document.body.classList.add('offline'); 51 document.body.classList.add('offline');
53 new Runner('.interstitial-wrapper'); 52 new Runner('.interstitial-wrapper');
54 } 53 }
55 } 54 }
OLDNEW
« no previous file with comments | « chrome/browser/resources/chromeos/merge_session_load.html ('k') | chrome/browser/site_per_process_interactive_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698