[Webkit-unassigned] [Bug 34861] Web Inspector: Focusing a resource in the inspector should default to content not header

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=34861


Joseph Pecoraro <joepeck at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #48603|                            |review?
               Flag|                            |




--- Comment #3 from Joseph Pecoraro <joepeck at webkit.org>  2010-02-11 18:32:31 PST ---
Created an attachment (id=48603)
 --> (https://bugs.webkit.org/attachment.cgi?id=48603)
[PATCH] Default to Content Tab

The initial default is "headers" (from Settings.js):

> this._installSetting("resourceViewTab", "resource-view-tab", "headers");

But the relevant code does look at the last user action (from ResourceView.js):

> _selectTab: function()
> {
>     if (this._headersVisible) {
>         if (WebInspector.settings.resourceViewTab === "headers")
>             this._selectHeadersTab();
>         else
>             this._selectContentTab();
>     } else
>         this._innerSelectContentTab();
> },

Sounds like we should just empty out the default, or change it to something
like "content" to make it future proof for more tabs. Patch uses "content".

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.