Automated g4 rollback of changelist 81082112.

*** Reason for rollback ***

Rolling forward now that tests are fixed.

*** Original change description ***

Automated g4 rollback of changelist 81074155.

*** Reason for rollback ***

Breaks a test.

*** Original change description ***

Mark goog.module.ModuleManager as @struct.

This CL also moves all property declarations into the constructor, and tightens up a few loose types.

***

***
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81346841
Change goog.ui.Button to listen for goog.events.KeyHandler.EventType.KEY events on SPACE instead of goog.events.EventType.KEYUP events. This fixes an issue where hitting SPACE on a button inside of a goog.ui.Container (ex. toolbar) does not trigger the button, since goog.ui.Container does not listen for goog.events.EventType.KEYUP events (only goog.events.KeyHandler.EventType.KEY).

As an FYI, the change to listen for KEYUP events for SPACE happened in cr/8612470 to fix  b/1397374. But since the goog.events.KeyHandler that goog.ui components use bundles up all of the key events into one single goog.events.KeyHandler.EventType.KEY event, this is probably what we should be looking for instead.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81383445
Correct previously undetected type error
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81387244
Move mockControl.$tearDown from tearDownPage to tearDown.

When the environment creates a mockControl object, it should take full ownership of it and manage its state correctly. Previously it could happen that one test affected another because the mockControl.$tearDown was not automatically executed on env.tearDown.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81402364
Update depswriter for goog.module.

Fixes #382 on github
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81435962
Automated g4 rollback of changelist 81383445.

*** Reason for rollback ***

Breaks i18n address widget

*** Original change description ***

Change goog.ui.Button to listen for goog.events.KeyHandler.EventType.KEY events on SPACE instead of goog.events.EventType.KEYUP events. This fixes an issue where hitting SPACE on a button inside of a goog.ui.Container (ex. toolbar) does not trigger the button, since goog.ui.Container does not listen for goog.events.EventType.KEYUP events (only goog.events.KeyHandler.EventType.KEY).

As an FYI, the change to listen for KEYUP events for SPACE happened in cr/8612470 to fix  b/1397374. But since the...

***
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81437164
setHighlightedIdInternal doesn't appear to be called anywhere.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81441495
Automated g4 rollback of changelist 81387244.

*** Reason for rollback ***

Logic should not short circuit

*** Original change description ***

Correct previously undetected type error

***
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81443791
Automated g4 rollback of changelist 81443791.

*** Reason for rollback ***

fixed. The test now passes: //javatests/com/google/trix/client/jscript/waffle/view/embeddedobject:all_js_tests_chrome-linux

*** Original change description ***

Automated g4 rollback of changelist 81387244.

*** Reason for rollback ***

Logic should not short circuit

*** Original change description ***

Correct previously undetected type error

***

***
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81455727
Introduce a shared definition for the varaibles used across
files in the mechanism unit tests.

When the test files are shifted around (when new test files
are introduced), this is causing compilation errors.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81471360
Fix component addChild method to not move the child element if it's already in
the right place.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81557870
Listens to all "Text to display" changes in link dialog
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81568764
Remove use of innerHTML in flashobject.js, instead use goog.dom.safe.setInnerHtml().

In addition:
- Sanitize string URLs, since they could be dangerous (javascript: and friends).
- Unit tests to check that the same tags are being created as before.

Note: this is the second attempt at this CL. The first one was rolled back in cl/81087255. The only difference compared to the original one is that now we set allownetworking=true in the <embed> which is the Flash default and hence what was being used before using goog.html.flash to generate the <embed>/<object>.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81659525
Remove usage on innerHTML, instead use document.createElement() to create <table> and its children.

As per discussion in gws-js-bloat, the previous attempt at doing this, using goog.dom.SafeHtml.create() added substantial amount of JS to GWS primary request (https://groups.google.com/a/google.com/d/msg/gws-js-bloat/ysKIpdlBFP4/ry1MgdmZ1QAJ).
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81671629
Make HtmlFormatter.formatRecordAsHtml() display information generated by goog.debug.exposeException() once again. It was previously inserted into LogRecord by Logger but removed in cl/81254136.

To do this there's now a goog.debug.exposeExceptionAsHtml which returns SafeHtml.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81676235
Fix depwriter broken due to bad edit.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81677841
Add a link to the goog.module doc in the goog.module JSDoc.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81686223
clean up lint errors in base.js
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81691912
Adding the unicode character MYANMAR SIGN LITTLE SECTION (e.g U+104A) as a valid address separator.

Character was added to the list of address separators (proposed by the internationalization team) in https://docs.google.com/a/google.com/spreadsheets/d/1yARkO1nu0gyYGs9zoVMxMkYhK1bR-wVzOoDw5uVLZGM/edit#gid=0

This CL is related to cl/81584082.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81760018
Replace innerHTML usage with goog.dom.setTextContent() and remove fpsdisplay.js for innerHTML Conformance whitelist.
-------------
Created by MOE: http://code.google.com/p/moe-java
MOE_MIGRATED_REVID=81774211