[chromium] WebFrameImpl::m_totalMatchCount should be an int
Created attachment 83637 [details] Patch
No test possible?
This is just a trivial cleanup - the variable is documented as sometimes being -1, and is indeed initialized to -1 by the constructor, so shouldn't be using an unsigned type. In practice this won't change any behavior as the code happens to never depend on the signedness, hence it would be tricky to test.
Comment on attachment 83637 [details] Patch Thanks for the review. Could someone add this to the commit queue please?
Comment on attachment 83637 [details] Patch How do we test this?
> How do we test this? As John suggested, this is a trivial fix and the current behaviour is OK, so I don't think it's valuable to try to craft a regression test.
Comment on attachment 83637 [details] Patch Clearing flags on attachment: 83637 Committed r79767: <http://trac.webkit.org/changeset/79767>
All reviewed patches have been landed. Closing bug.