Created attachment 32415 [details] Code patch for Interpreter.h On Symbian platforms we encounter include ambiguities of HashMap.h file, which is already present in default location. Since that header filename is very generic, it appears to be appropriate and saver to add a pathname qualifier "wtf/HashMap.h" to the include statement.
Comment on attachment 32415 [details] Code patch for Interpreter.h It's OK to change the include here to the normal style, but the normal style is to include <wtf/HashMap.h> and include in a separate paragraph.
Comment on attachment 32415 [details] Code patch for Interpreter.h r- per Darin's comment.
Created attachment 32756 [details] Second try. Updated the patch based on Darin's feedback (filling in for Norbert.) Note that since this patch reverses a subset of r44418 (presumably a build fix for WIN_CAIRO), the proposed fix might break WIN_CAIRO port (I CCd Brent on the bug).
This change should be fine for WIN_CAIRO. I added the HashMap.h because it was needed to compile. Using the fully-qualified wtf/HashMap.h path should not be a problem.
Landed in r45909