Mojo JavaScript bindings should be binary compatible with C++

After this CL, the JavaScript bindings we autogenerate from
sample_service.mojom files are binary compatible with the C++ bindings. I've
also replaced the hand-written sample_service.js file with the auto-generated
version.

I've also moved the logic for assigning implicit ordinals to methods into
common code that's shared between the CPP and JS bindings generators.

BUG=317398

Review URL: https://codereview.chromium.org/81313003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236626 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 236626
Misc. cleanup:

* Don't inline not-normally-inlined methods in non-test code
* Shorten
* DISALLOW_COPY_AND_ASSIGN
* Overrides should be in same order as in base class
* No else after return
* Fix line wrapping, comments, indenting

I'm not bothering to clean up HorizontalPaddingButtonBorder,
NotifierGroupMenuButtonBorder, or TouchEditingMenuButtonBorder since I'll be
removing those entirely in my next patch.

BUG=none
TEST=none
R=msw@chromium.org, sky@chromium.org

Review URL: https://codereview.chromium.org/80103002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236627 0039d316-1c4b-4281-b951-d872f2087c98
Misc. cleanup:

* Scope constants/variables as narrowly as possible
* Don't override virtual methods unnecessarily
* Minor comment improvements
* Don't inline not-normally-inlined methods in non-test code
* Use {} where required
* Eliminate leftover forward-decl in MaximizeBubbleController (oops)
* Fix indenting & wrapping

BUG=none
TEST=none
R=sky@chromium.org

Review URL: https://codereview.chromium.org/80043002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236628 0039d316-1c4b-4281-b951-d872f2087c98
Adds handling of clicking on the omnibox search/go button.

BUG=304859, 311269

Review URL: https://codereview.chromium.org/78053009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236629 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 236629