Bug 43065 - Provide a Helpful Link for Web Inspector Bugs in Bugzilla
Summary: Provide a Helpful Link for Web Inspector Bugs in Bugzilla
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-27 11:54 PDT by Joseph Pecoraro
Modified: 2010-07-27 15:26 PDT (History)
10 users (show)

See Also:


Attachments
[PATCH] Add a fake "Inspector" component, which uses the link (1.19 KB, patch)
2010-07-27 11:56 PDT, Joseph Pecoraro
ddkilzer: review-
Details | Formatted Diff | Diff
[IMAGE] What it should look like (78.02 KB, image/png)
2010-07-27 11:57 PDT, Joseph Pecoraro
no flags Details
[PATCH] New template in the custom directory. (3.63 KB, patch)
2010-07-27 12:06 PDT, Joseph Pecoraro
ddkilzer: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2010-07-27 11:54:48 PDT
It would be nice if there was a listing for the http://webkit.org/new-inspector-bug
link right from Bugzilla.
Comment 1 Joseph Pecoraro 2010-07-27 11:56:50 PDT
Created attachment 62718 [details]
[PATCH] Add a fake "Inspector" component, which uses the link
Comment 2 Joseph Pecoraro 2010-07-27 11:57:07 PDT
Created attachment 62719 [details]
[IMAGE] What it should look like
Comment 3 David Kilzer (:ddkilzer) 2010-07-27 12:00:36 PDT
Comment on attachment 62718 [details]
[PATCH] Add a fake "Inspector" component, which uses the link

> diff --git a/BugsSite/template/en/default/global/choose-product.html.tmpl b/BugsSite/template/en/default/global/choose-product.html.tmpl

This looks great, except you need to copy template/en/default/global/choose-product.html.tmplm to template/en/custom/global/choose-product.html.tmpl and then modify that.

r- to copy and modify a custom copy of the template file.
Comment 4 Joseph Pecoraro 2010-07-27 12:06:24 PDT
Created attachment 62722 [details]
[PATCH] New template in the custom directory.
Comment 5 Joseph Pecoraro 2010-07-27 12:07:21 PDT
I would have thought git would have determined it was a copy.
Apparently it didn't. =/
Comment 6 David Kilzer (:ddkilzer) 2010-07-27 12:17:21 PDT
Comment on attachment 62722 [details]
[PATCH] New template in the custom directory.

r=me
Comment 7 WebKit Commit Bot 2010-07-27 13:04:27 PDT
Comment on attachment 62722 [details]
[PATCH] New template in the custom directory.

Rejecting patch 62722 from commit-queue.

Unexpected failure when processing patch!  Please file a bug against webkit-patch.
Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'land-attachment', '--force-clean', '--build', '--non-interactive', '--ignore-builders', '--build-style=both', '--quiet', 62722, '--test', '--parent-command=commit-queue', '--no-update']" exit_code: 1
Last 500 characters of output:
webkitpy/tool/commands/stepsequence.py", line 64, in _run
    step(tool, options).run(state)
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/tool/steps/validatereviewer.py", line 66, in run
    if self._has_valid_reviewer(changelog_entry):
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/tool/steps/validatereviewer.py", line 48, in _has_valid_reviewer
    if changelog_entry.reviewer():
AttributeError: 'NoneType' object has no attribute 'reviewer'
Comment 8 WebKit Commit Bot 2010-07-27 13:58:28 PDT
Comment on attachment 62722 [details]
[PATCH] New template in the custom directory.

Rejecting patch 62722 from commit-queue.

Unexpected failure when processing patch!  Please file a bug against webkit-patch.
Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'land-attachment', '--force-clean', '--build', '--non-interactive', '--ignore-builders', '--build-style=both', '--quiet', 62722, '--test', '--parent-command=commit-queue', '--no-update']" exit_code: 1
Last 500 characters of output:
webkitpy/tool/commands/stepsequence.py", line 64, in _run
    step(tool, options).run(state)
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/tool/steps/validatereviewer.py", line 66, in run
    if self._has_valid_reviewer(changelog_entry):
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/tool/steps/validatereviewer.py", line 48, in _has_valid_reviewer
    if changelog_entry.reviewer():
AttributeError: 'NoneType' object has no attribute 'reviewer'
Comment 9 Joseph Pecoraro 2010-07-27 15:26:16 PDT
Thanks for the idea and review Dave!
Its working on https://bugs.webkit.org/enter_bug.cgi

Landed Manually.
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	BugsSite/ChangeLog
	A	BugsSite/template/en/custom/global/choose-product.html.tmpl
Committed r64168 = 1b66af104c145b0c752bd5aac6ebffe8f44387bd (refs/remotes/trunk)
http://trac.webkit.org/changeset/64168