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

Side by Side Diff: net/docs/bug-triage-labels.md

Issue 2572513002: [net] Remove downloads bug triage from //net triage responsibilities. (Closed)
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « net/docs/bug-triage.md ('k') | net/docs/bug-triage-suggested-workflow.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Chrome Network Bug Triage : Components and labels 1 # Chrome Network Bug Triage : Components and labels
2 2
3 ## Some network component caveats 3 ## Some network component caveats
4 4
5 * **UI>Browser>Downloads**
6
7 Despite the name, this covers all issues related to downloading a file
8 except saving entire pages (which is **Blink>SavePage**), not just UI
9 issues. Most downloads bugs will have the word "download" or "save as" in
10 the description. Issues with the HTTP server for the Chrome binaries are
11 not downloads bugs.
12
13 * **UI>Browser>SafeBrowsing**
14
15 Bugs that have to do with the process by which a URL or file is determined
16 to be dangerous based on our databases, or the resulting interstitials.
17 Determination of danger based purely on content-type or file extension
18 belongs in **UI>Browser>Downloads**, not SafeBrowsing.
19
20 * **Internals>Network>SSL** 5 * **Internals>Network>SSL**
21 6
22 This includes issues that should be also tagged as **Security>UX** 7 This includes issues that should be also tagged as **Security>UX**
23 (certificate error pages or other security interstitials, omnibox indicators 8 (certificate error pages or other security interstitials, omnibox indicators
24 that a page is secure), and more general SSL issues. If you see requests 9 that a page is secure), and more general SSL issues. If you see requests
25 that die in the SSL negotiation phase, in particular, this is often the 10 that die in the SSL negotiation phase, in particular, this is often the
26 correct component. 11 correct component.
27 12
28 * **Internals>Network>DataProxy** 13 * **Internals>Network>DataProxy**
29 14
(...skipping 30 matching lines...) Expand all
60 45
61 Covers SDCH and gzip issues. `ERR_CONTENT_DECODING_FAILED` indicates a 46 Covers SDCH and gzip issues. `ERR_CONTENT_DECODING_FAILED` indicates a
62 problem at this layer, and bugs here can also cause response body 47 problem at this layer, and bugs here can also cause response body
63 corruption. 48 corruption.
64 49
65 ## Common non-network components 50 ## Common non-network components
66 51
67 Bugs in these areas often receive the **Internals>Network** component, though 52 Bugs in these areas often receive the **Internals>Network** component, though
68 they fall largely outside the purview of the network stack team: 53 they fall largely outside the purview of the network stack team:
69 54
55 * **UI>Browser>Downloads**
56
57 Despite the name, this covers all issues related to downloading a file
58 except saving entire pages (which is **Blink>SavePage**), not just UI
59 issues. Most downloads bugs will have the word "download" or "save as" in
60 the description. Issues with the HTTP server for the Chrome binaries are
61 not downloads bugs.
62
63 * **UI>Browser>SafeBrowsing**
64
65 Bugs that have to do with the process by which a URL or file is determined
66 to be dangerous based on our databases, or the resulting interstitials.
67 Determination of danger based purely on content-type or file extension
68 belongs in **UI>Browser>Downloads**, not SafeBrowsing.
69
70 * **Blink>Forms** 70 * **Blink>Forms**
71 71
72 Issues submitting forms, forms having weird data, forms sending the wrong 72 Issues submitting forms, forms having weird data, forms sending the wrong
73 method, etc. 73 method, etc.
74 74
75 * **Blink>Loader** 75 * **Blink>Loader**
76 76
77 Cross origin issues are sometimes loader related. Blink also has an 77 Cross origin issues are sometimes loader related. Blink also has an
78 in-memory cache, and when it's used, requests don't appear in 78 in-memory cache, and when it's used, requests don't appear in
79 about:net-internals. Requests for the same URL are also often merged there 79 about:net-internals. Requests for the same URL are also often merged there
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 beyond the lifetime of incognito tabs, etc). Generally used in conjunction 164 beyond the lifetime of incognito tabs, etc). Generally used in conjunction
165 with other components. 165 with other components.
166 166
167 ## Common labels 167 ## Common labels
168 168
169 * **Type-Bug-Security** 169 * **Type-Bug-Security**
170 170
171 Security related bug (Allows for code execution from remote site, allows 171 Security related bug (Allows for code execution from remote site, allows
172 crossing security boundaries, unchecked array bounds, etc) should be tagged 172 crossing security boundaries, unchecked array bounds, etc) should be tagged
173 with this label. 173 with this label.
OLDNEW
« no previous file with comments | « net/docs/bug-triage.md ('k') | net/docs/bug-triage-suggested-workflow.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698