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

Unified Diff: chrome/browser/resources/bug_report.html

Issue 5514001: Crash fix + OS specific ss's enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Build fix for linux_view. Created 10 years 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/bug_report.html
diff --git a/chrome/browser/resources/bug_report.html b/chrome/browser/resources/bug_report.html
index 3d3d058f51b084784787be1701382c4aea8fd688..38c05a49493c5b19ab8db696db0355f5f3a984b5 100644
--- a/chrome/browser/resources/bug_report.html
+++ b/chrome/browser/resources/bug_report.html
@@ -19,9 +19,10 @@
* Window onload handler, sets up the page.
*/
function load() {
- $('sysinfo-url').onclick = function(event) {
- chrome.send('openSystemTab');
- };
+ if ($('sysinfo-url'))
+ $('sysinfo-url').onclick = function(event) {
+ chrome.send('openSystemTab');
+ };
oshima 2010/12/02 18:03:30 please use if () {} for multi-line
rkc 2010/12/02 19:54:04 Done.
var menuOffPattern = /(^\?|&)menu=off($|&)/;
var menuDisabled = menuOffPattern.test(window.location.search);
@@ -181,10 +182,10 @@ window.addEventListener('DOMContentLoaded', load);
</table>
</td>
</tr>
+<if expr="pp_ifdef('chromeos')">
<!-- User e-mail -->
<tr>
<td>
-<if expr="pp_ifdef('chromeos')">
<table id="user-email-table" class="bug-report-table">
<tr>
<td class="bug-report-fieldlabel">
@@ -199,7 +200,6 @@ window.addEventListener('DOMContentLoaded', load);
</td>
</tr>
</table>
-</if>
</td>
</tr>
<!-- System Information -->
@@ -219,6 +219,7 @@ window.addEventListener('DOMContentLoaded', load);
</table>
</td>
</tr>
+</if>
<!-- Screenshot -->
<tr>
<td>

Powered by Google App Engine
This is Rietveld 408576698