Bug 282039 – ODR violation due to redefinition of maxTitleLength contant

• Component: WebKit Process Model
• Status: RESOLVED FIXED

[Bartosz Moczulski]

Created attachment 473032 [details]
Fix proposal

With UnifiedSources `--max-bundle-size` set to 16 it is possible for WebLocalFrameLoaderClient.cpp and WebChromeClient.cpp to end up in a single compilation unit. The fact that they both contain a static `maxTitleLength` constant in `WebKit` namespace results in an ODR violation during an attempt to compile them both together.

Fix proposal (attached): move maxTitleLength into functions where they are actually needed, like in SessionStateConversion.cpp.

[Radar WebKit Bug Importer]

<rdar://problem/138989376>

[Elliott Williams]

Thank you! Looks good, though you'll need to update your patch's commit message to follow WebKit's template. The template should be configured by default in git repositories once you've run `Tools/Scripts/git-webkit setup`. You can also copy it manually from <https://github.com/WebKit/WebKit/blob/main/.github/pull_request_template.md#template>.

Contributions via GitHub PR are also welcome, see <https://github.com/WebKit/WebKit/wiki/Contributing#setup> for details.

[Bartosz Moczulski]

Created attachment 473152 [details]
Fix proposal (amended to conform to style requirements)

[Bartosz Moczulski]

Thanks for the tips how to contribute properly, first ones are always hardest. If it's not too much trouble let's stick to .patch flow here. I'll try the PR approach starting with the next improvement, whenever I happen come up with one.

[EWS]

Committed 286230@main (44aa509992c1): <https://commits.webkit.org/286230@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 473152 [details].