Summary: | REGRESSION: Docked/Undocked state of inspector broken on Windows | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brady Eidson <beidson> | ||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Brady Eidson <beidson> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | eric, joepeck, pfeldman | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Brady Eidson
2010-03-09 18:49:11 PST
Created attachment 50365 [details]
The fix
This is the fix. Not putting it up for review yet, because the settings hack is something that isn't sustainable. Mac and Windows clients each have their own copy of the key that they use in multiple places, for example.
We should strongly consider moving the keys to some sort of interface that could be used in WebCore and by the WebKit clients so it'd be much harder for them to get out of sync.
Created attachment 50367 [details]
Real fix, with some more proper sharing of the settings key in quesiton
Attachment 50367 [details] did not build on mac: Build output: http://webkit-commit-queue.appspot.com/results/549005 Dammit, forgot to change the *right* WebCore.exp Created attachment 50368 [details]
Include WebCore.base.exp in the fix.
Landed in http://trac.webkit.org/changeset/55761 Brady, it would be great if you could use http://webkit.org/new-inspector-bug when filing inspector-related issues! (In reply to comment #7) > Brady, it would be great if you could use http://webkit.org/new-inspector-bug > when filing inspector-related issues! I'd be more than happy to, and love the idea. But the normal workflow for filing "any webkit bug" is to visit bugs.webkit.org. So: - How was I supposed to know about this deviation from the normal workflow before now? - Is it linked from the main WebKit site anywhere? - It seems to be a feature of http://webkit.org, so why don't I see any mention of it in the WebKitSite directory of the source tree? - Shouldn't it be linked from bugs.webkit.org when you're filing a new bug? - If there's any other such convenience templates for other bugs, what are they? and finally... - If there's any other such templates... answer all the above questions for them :) Too many questions I can't answer. The only place it is published is our channel. Now that you know about it, would be great if you could use it! (In reply to comment #8) > - It seems to be a feature of http://webkit.org, so why don't I see any mention > of it in the WebKitSite directory of the source tree? These are redirects set-up on the server, uneditable by the public. We got this link updated through emails to an admin. The only other ones I'm aware of are linked from: http://nightly.webkit.org/start/trunk/55516 They are "Forgotten Patches", "Patches awaiting Review", and "Approved Patches", each use the short link style. I agree its confusing, but its also very nice. I also haven't seen any other templates like it. (In reply to comment #10) > I agree its confusing, but its also very nice. I also haven't seen any other > templates like it. It wouldn't be confusing if it was described somewhere and easily accessible. I'm working with Bill on this. |