[Devtools] Proper filter set from panel jump into network panel

[Devtools] Proper filter set from panel jump into network panel

Made it so when link is clicked that jumps and pre-filters results
in the network panel it now properly adjusts regex and other filters.

BUG=609441
R=lushnikov

Committed: https://crrev.com/cdf66f4f2270f5e5fbe8b45f9f53a87f268614ab
Cr-Commit-Position: refs/heads/master@{#396383}

[allada, 2016-05-24 21:49:13]

PTL

[lushnikov, 2016-05-24 22:50:36]

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

https://codereview.chromium.org/2006143004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js:1060:
setTextFilterValue: function(filterString)
Let's make this method reset regexChecked checkbox and dataURLFilterUI
checkboxes.

https://codereview.chromium.org/2006143004/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js (right):

https://codereview.chromium.org/2006143004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js:527:
this._toggleTypeFilter(WebInspector.NamedBitSetFilterUI.ALL_TYPES, false);
nit: a comment to name literal would help

... false /* allowMultiSelect */);

[allada, 2016-05-26 21:33:07]

PTL

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

https://codereview.chromium.org/2006143004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js:1060:
setTextFilterValue: function(filterString)
On 2016/05/24 22:50:36, lushnikov wrote:
> Let's make this method reset regexChecked checkbox and dataURLFilterUI
> checkboxes.

Done.

https://codereview.chromium.org/2006143004/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js (right):

https://codereview.chromium.org/2006143004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js:527:
this._toggleTypeFilter(WebInspector.NamedBitSetFilterUI.ALL_TYPES, false);
On 2016/05/24 22:50:36, lushnikov wrote:
> nit: a comment to name literal would help
> 
> ... false /* allowMultiSelect */);

Done.

[lushnikov, 2016-05-26 23:25:13]

lgtm

[allada, 2016-05-26 23:28:20]

The CQ bit was checked by allada@chromium.org

[commit-bot: I haz the power, 2016-05-26 23:29:19]

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

[commit-bot: I haz the power, 2016-05-27 01:32:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-27 01:32:31]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_TIMED_OUT, no build
URL)

[allada, 2016-05-27 03:17:12]

The CQ bit was checked by allada@chromium.org

[commit-bot: I haz the power, 2016-05-27 03:17:34]

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

[commit-bot: I haz the power, 2016-05-27 04:35:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-27 04:37:57]

Description was changed from

==========
[Devtools] Proper filter set from panel jump into network panel

Made it so when link is clicked that jumps and pre-filters results
in the network panel it now properly adjusts regex and other filters.

BUG=609441
R=lushnikov
==========

to

==========
[Devtools] Proper filter set from panel jump into network panel

Made it so when link is clicked that jumps and pre-filters results
in the network panel it now properly adjusts regex and other filters.

BUG=609441
R=lushnikov

Committed: https://crrev.com/cdf66f4f2270f5e5fbe8b45f9f53a87f268614ab
Cr-Commit-Position: refs/heads/master@{#396383}
==========

[commit-bot: I haz the power, 2016-05-27 04:37:59]

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