Bug fix for the pointer lock string problem

chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller.mm

Index: chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller.mm
diff --git a/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller.mm b/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller.mm
index d6c0de1689d19206affeb70769be10d861a2380a..f319662dd98d26dedf496ee3dc35d98efc4a46c3 100644
--- a/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller.mm
+++ b/chrome/browser/ui/cocoa/exclusive_access_bubble_window_controller.mm
@@ -258,6 +258,17 @@ const float kHideDuration = 0.7;
   labelFrame.origin.x += NSWidth(labelFrame) - NSWidth(textFrame);
   labelFrame.size = textFrame.size;
   [exitLabel_ setFrame:labelFrame];
+
+  // Update the title of denyButton_ according to the current bubbleType_
+  if (bubbleType_ == EXCLUSIVE_ACCESS_BUBBLE_TYPE_FULLSCREEN_BUTTONS ||

[Mark Mentovai, 2015/01/27 21:32:16]

Call exclusive_access_bubble::ShowButtonsForType() instead of duplicating its
logic here.

[Jialiu Lin, 2015/01/28 02:47:09]

Done.

+      bubbleType_ ==
+          EXCLUSIVE_ACCESS_BUBBLE_TYPE_FULLSCREEN_MOUSELOCK_BUTTONS ||
+      bubbleType_ == EXCLUSIVE_ACCESS_BUBBLE_TYPE_MOUSELOCK_BUTTONS) {
+    NSString* denyButtonText =
+      SysUTF16ToNSString(
+        exclusive_access_bubble::GetDenyButtonTextForType(bubbleType_));

[Mark Mentovai, 2015/01/27 21:32:16]

Since you’re now always populating the deny button text (if the deny button is
to be shown) right here, I think you should remove the to-be-localized text from
the nib. That will prevent the work from being done to get the default
IDS_FULLSCREEN_DENY label if it’s just going to be overwritten, and the
duplicate work from being done to get it twice if it’s in fact the correct
label. It will also provide a better visual indication of a problem (label-less
button?) in case this code doesn’t execute properly.

[Jialiu Lin, 2015/01/28 02:47:09]

Done.

+    [denyButton_ setTitle:denyButtonText];
+  }

[Mark Mentovai, 2015/01/27 21:32:16]

Perhaps the showButtons:NO call should be here in an else instead of in
-showWindow. Conceptually, I believe it makes more sense to do it here because
it’s a choice between “set proper button text” and “don’t show button at all”.
The ShowButtonsForType() check in -showWindow isn’t as much about showing or
hiding the buttons as it is about arranging to dismiss the window when no
buttons are present. Hiding the buttons is more related to this code than that
code.

With this change, I’m starting to think that this method could be named
something better than -initializeLabel, since it’s now doing more than just
initializing its label.

[Jialiu Lin, 2015/01/28 02:47:08]

Added the else clause, and changed the function name to 
-initializeLabelAndButton

 }
 
 - (NSString*)getLabelText {