Landing http://codereview.chromium.org/7033038 for jshin.

src/extensions/experimental/collator.cc

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 135 matching lines @@
   // by a caller in JavaScript. Otherwise, we don't touch because
   // we don't want to change the locale-dependent default value.
   // The three options below are very likely to have the same default
   // across locales, but I haven't checked them all. Others we may add
   // in the future have certainly locale-dependent default (e.g.
   // caseFirst is upperFirst for Danish while is off for most other locales).
 
   bool ignore_case, ignore_accents, numeric;
 
   if (ExtractBooleanOption(options, "ignoreCase", &ignore_case)) {
+    // We need to explicitly set the level to secondary to get case ignored.
+    // The default L3 ignores UCOL_CASE_LEVEL == UCOL_OFF !
+    if (ignore_case) {
+      collator->setStrength(icu::Collator::SECONDARY);
+    }
     collator->setAttribute(UCOL_CASE_LEVEL, ignore_case ? UCOL_OFF : UCOL_ON,
                            status);
     if (U_FAILURE(status)) {
       delete collator;
       return ThrowExceptionForICUError("Failed to set ignoreCase.");
     }
   }
 
   // Accents are taken into account with strength secondary or higher.
   if (ExtractBooleanOption(options, "ignoreAccents", &ignore_accents)) {
@@ skipping 43 matching lines @@
   wrapper->SetPointerInInternalField(0, collator);
 
   // Make object handle weak so we can delete iterator once GC kicks in.
   wrapper.MakeWeak(NULL, DeleteCollator);
 
   return wrapper;
 }
 
 } }  // namespace v8::internal