Summary: | [Gtk] No need to set text encoding in the provisional phase | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joone Hur <joone> | ||||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, gustavo, joone.hur, mrobinson, xan.lopez | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | OS X 10.5 | ||||||||
Attachments: |
|
Description
Joone Hur
2011-02-01 05:12:56 PST
Created attachment 80748 [details]
Proposed Patch
I recall looking at this issue briefly, but at this point I no longer remember why our implementation differs from Qt. Does this allow us to unskip any tests?j (In reply to comment #2) > I recall looking at this issue briefly, but at this point I no longer remember why our implementation differs from Qt. Does this allow us to unskip any tests?j I tried to find any tests to be unskipped, but I couldn't find them. Actually, there seems to be no problem without this patch until now, but I'm sure that this patch allows WebKitGtk+ to work like other ports. This may prevent unexpected errors. Created attachment 81057 [details]
Proposed Patch2
I added more details.
Comment on attachment 81057 [details]
Proposed Patch2
Thank you!
Comment on attachment 81057 [details] Proposed Patch2 Clearing flags on attachment: 81057 Committed r77596: <http://trac.webkit.org/changeset/77596> All reviewed patches have been landed. Closing bug. |