This adds a webui overlay on the extensions page for showing what Extension keybindings are active.…

chrome/browser/resources/shared/js/load_time_data.js

Index: chrome/browser/resources/shared/js/load_time_data.js
===================================================================
--- chrome/browser/resources/shared/js/load_time_data.js	(revision 137495)
+++ chrome/browser/resources/shared/js/load_time_data.js	(working copy)
@@ -42,6 +42,9 @@
      */
     getValue: function(id) {
       expect(this.data_, 'No data. Did you remember to include strings.js?');
+      // If your value is not found here, inspect the contents of this.data_.
+      // If you find none of your values there, then make sure that
+      // GetLocalizedValues() in your handler class (in C++) is called.

[Evan Stade, 2012/05/18 21:08:26]

this comment is inaccurate, GetLocalizedValues() does not exist for all
WebUIMessageHandler classes. I don't think it's worth including --- this isn't
where I would expect most people to look for documentation.

[Finnur, 2012/05/18 22:02:04]

Having gone through the exercise of trying to figure out why the messages aren't
showing up properly this is where the dead-end was reached. It wasn't obvious
where to go from here in the C++ code. 

Is there any generic comment we can use here instead (to point people in the
right direction) or should we just remove it?

On 2012/05/18 21:08:26, Evan Stade wrote:

[Evan Stade, 2012/05/18 22:14:23]

not obvious where to go in the C++ code? You added the C++ code which attempted
but failed to set new strings, so I would have thought it pretty clear you have
to go there to see why it wasn't working.

On 2012/05/18 22:02:04, Finnur wrote:

[Finnur, 2012/05/18 22:27:01]

I added GetLocalizedValues() but overlooked the fact that you have to explicitly
call it. Then it looks like the C++ code is correct, but it isn't actually doing
anything so you blame the .js side of things. Anyway, I was just trying to help
the next guy. I'll just remove the comment. 

On 2012/05/18 22:14:23, Evan Stade wrote:

       var value = this.data_[id];
       expect(typeof value != 'undefined', 'Could not find value for ' + id);
       return value;