Bug 47827 - Web Inspector: disable private browsing for inspector
Summary: Web Inspector: disable private browsing for inspector
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Pavel Podivilov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-18 09:25 PDT by Pavel Podivilov
Modified: 2010-10-18 11:06 PDT (History)
13 users (show)

See Also:


Attachments
Patch. (2.37 KB, patch)
2010-10-18 09:28 PDT, Pavel Podivilov
timothy: review+
podivilov: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Podivilov 2010-10-18 09:25:54 PDT
Currently private browsing is enabled for inspector which makes localStorage inaccessible.
Comment says it's done to keep inspector out of history, but even with private browsing disabled, inspector don't appear in history. Debugging safari showed that inspector's WebView doesn't have history delegate, but I'm not sure because
history delegate is assigned in private safari code.

Joseph, could you please check if inspector doesn't have history delegate and thus private browsing isn't needed for inspector?
Comment 1 Pavel Podivilov 2010-10-18 09:28:44 PDT
Created attachment 71046 [details]
Patch.
Comment 2 Joseph Pecoraro 2010-10-18 09:41:52 PDT
Timothy would be more likely to know. I'll see what I can find.
Comment 3 Joseph Pecoraro 2010-10-18 09:45:33 PDT
By the way, your patch should remove the comment if it is no longer true. =)
Comment 4 Timothy Hatcher 2010-10-18 09:50:06 PDT
Comment on attachment 71046 [details]
Patch.

I do think this is correct now, since we have the history delegate concept and there isn't one hooked up to the Inspector. Assuming you tested and didn't see it show up in Safari's History menu.
Comment 5 Pavel Podivilov 2010-10-18 09:58:03 PDT
Comment on attachment 71046 [details]
Patch.

cq- to address Joseph's comment.
Comment 6 Pavel Podivilov 2010-10-18 10:22:23 PDT
Committed r69969: <http://trac.webkit.org/changeset/69969>
Comment 7 WebKit Review Bot 2010-10-18 11:06:22 PDT
http://trac.webkit.org/changeset/69969 might have broken Qt Windows 32-bit Release