[DevTools] Hide "Service Worker Fallback" response from the Network tab.

[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.
http://stackoverflow.com/questions/34035667/service-worker-making-extra-call

So this cl removes the CORS fallback response from the network panel.

BUG=477685
Committed: https://crrev.com/2e1de2f8f4ee2c620c43f00b2825224689046c3a
Cr-Commit-Position: refs/heads/master@{#365441}

[horo, 2015-12-14 06:36:01]

Description was changed from

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for the internal response which is sent from the browser
process to the page's renderer process when the Service Worker didn't handle the
CORS request to consider the CORS preflight logic.
This message is not only meaningless for Web developers but also confusing them.
So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

to

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.

So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

[horo, 2015-12-14 06:38:17]

Description was changed from

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.

So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

to

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.
http://stackoverflow.com/questions/34035667/service-worker-making-extra-call

So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

[horo, 2015-12-14 06:38:55]

horo@chromium.org changed reviewers:
+ pfeldman@chromium.org

[horo, 2015-12-14 06:38:56]

pfeldman@
Could you please review this?

[pfeldman, 2015-12-16 00:48:12]

lgtm

https://codereview.chromium.org/1518383002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
(right):

https://codereview.chromium.org/1518383002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js:1638: if
(request.statusText == "Service Worker Fallback Required")
===

[horo, 2015-12-16 02:27:06]

Thank you!!

https://codereview.chromium.org/1518383002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
(right):

https://codereview.chromium.org/1518383002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js:1638: if
(request.statusText == "Service Worker Fallback Required")
On 2015/12/16 00:48:12, pfeldman wrote:
> ===

Done.

[horo, 2015-12-16 02:27:12]

The CQ bit was checked by horo@chromium.org

[horo, 2015-12-16 02:27:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1518383002/#ps20001
(title: "incorporated pfeldman's comment")

[commit-bot: I haz the power, 2015-12-16 02:35:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1518383002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1518383002/20001

[commit-bot: I haz the power, 2015-12-16 04:36:59]

Description was changed from

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.
http://stackoverflow.com/questions/34035667/service-worker-making-extra-call

So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

to

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.
http://stackoverflow.com/questions/34035667/service-worker-making-extra-call

So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

[commit-bot: I haz the power, 2015-12-16 04:36:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-16 04:38:03]

Description was changed from

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.
http://stackoverflow.com/questions/34035667/service-worker-making-extra-call

So this cl removes the CORS fallback response from the network panel.

BUG=477685
==========

to

==========
[DevTools] Hide "Service Worker Fallback" response from the Network tab.

This message is used for an internal response which is sent from the browser
process to the page's renderer process to consider the CORS preflight logic
when the Service Worker didn't handle the CORS request .
This message is not only meaningless for Web developers but also confusing them.
http://stackoverflow.com/questions/34035667/service-worker-making-extra-call

So this cl removes the CORS fallback response from the network panel.

BUG=477685

Committed: https://crrev.com/2e1de2f8f4ee2c620c43f00b2825224689046c3a
Cr-Commit-Position: refs/heads/master@{#365441}
==========

[commit-bot: I haz the power, 2015-12-16 04:38:04]

Patchset 2 (id:??) landed as
https://crrev.com/2e1de2f8f4ee2c620c43f00b2825224689046c3a
Cr-Commit-Position: refs/heads/master@{#365441}