CLOSED FIXED 11604
[S60] Remove RD_32_BROWSER flag from WebCore and WebKit
https://bugs.webkit.org/show_bug.cgi?id=11604
Summary [S60] Remove RD_32_BROWSER flag from WebCore and WebKit
Pat
Reported 2006-11-15 12:29:28 PST
Remove the obsolete flag RD_32_BROWSER from the WebCore and WebKit code base.
Attachments
Remove obsolete flags from WebKit (76.41 KB, patch)
2006-11-17 12:32 PST, Pat
Sachin.Padma: review-
S60Headers.zip file (240.81 KB, application/octet-stream)
2006-11-22 08:44 PST, Pat
no flags
Patch with S60Headers.zip file separate, see attachement of S60Headers.zip for the latest zip file. (76.50 KB, patch)
2006-11-22 08:48 PST, Pat
Sachin.Padma: review+
Pat
Comment 1 2006-11-15 12:32:34 PST
Will create two patches. One for WebCore and one for WebKit for approval and inclusion into the tree.
Pat
Comment 2 2006-11-17 12:32:34 PST
Created attachment 11555 [details] Remove obsolete flags from WebKit
Pat
Comment 3 2006-11-17 12:33:31 PST
Portions reviewed by the responsible engineers. No changes in WebCore, the flags were not used there. Two header files updated in the \S60\S60Internals\S60Headers.zip, HttpDownloadMgrCommon.h and BrowserUiInternalCRKeys.h. The WebKit.loc and WebKit_sdk.loc, and WebKit.rss and WebKit_sdk.rss files changed to include each others new entries.
Sachin Padma
Comment 4 2006-11-20 15:00:16 PST
Comment on attachment 11555 [details] Remove obsolete flags from WebKit Can you please remove the S60headers from the patch. When I apply the patch it deletes this zip file.
Sachin Padma
Comment 5 2006-11-21 15:20:11 PST
This is not fixed yet.
Pat
Comment 6 2006-11-22 08:44:25 PST
Created attachment 11609 [details] S60Headers.zip file The zip files in S60Internal directory have to be added manually.
Pat
Comment 7 2006-11-22 08:48:32 PST
Created attachment 11610 [details] Patch with S60Headers.zip file separate, see attachement of S60Headers.zip for the latest zip file.
Sachin Padma
Comment 8 2006-11-28 09:05:20 PST
Comment on attachment 11610 [details] Patch with S60Headers.zip file separate, see attachement of S60Headers.zip for the latest zip file. r=me
Sachin Padma
Comment 9 2006-11-28 09:33:33 PST
committed as r17905
Krishna
Comment 10 2006-12-05 10:57:26 PST
Verified and Closed.
Note You need to log in before you can comment on or make changes to this bug.