Bug 55774 - Split JNIBridge.cpp/h into JavaString.h and JavaMethod.cpp/h
Summary: Split JNIBridge.cpp/h into JavaString.h and JavaMethod.cpp/h
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Steve Block
URL:
Keywords:
Depends on: 55772
Blocks: 55765 55879 55881
  Show dependency tree / graph
 
Reported: 2011-03-04 08:08 PST by Steve Block
Modified: 2011-03-07 10:10 PST (History)
2 users (show)

See Also:

Attachments
Patch 1 (35.43 KB, patch)
2011-03-07 07:53 PST, Steve Block 		jorlow: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Steve Block 2011-03-04 08:08:41 PST 
Split JNIBridge into JavaString.h and JavaM:qethod.cpp/h
Comment 1 Steve Block 2011-03-04 08:11:04 PST 
JNIBridge.cpp/h contains only two classes - JavaMethod and JavaString - and it would aid clarity to split these into their own classes
Comment 2 Steve Block 2011-03-07 07:53:00 PST 
Created attachment 84945 [details]
Patch 1
Comment 3 Jeremy Orlow 2011-03-07 09:54:23 PST 
Comment on attachment 84945 [details]
Patch 1

style seems good.  rubber stamp = me
Comment 4 Steve Block 2011-03-07 10:10:25 PST 
Committed r80472: <http://trac.webkit.org/changeset/80472>