[ZEPPELIN-1763] Prevent duplicate $scope.$on('setNoteMenu') calls

[cuspymd]

What is this PR for?

It's to fix the issue 'ZEPPELIN-1763' (Zeppelin hangs if I repeatedly select and deselect note name).
When the note name was changed, there was a problem that the 'setNoteName' listener was registered in duplicate and occupied resources.
To fix it, I moved the function that registers the listener out of initNoteBook()

What type of PR is it?

[Bug Fix]

Todos

What is the Jira issue?

ZEPPELIN-1763

How should this be tested?

Repeat changing note name.

Screenshots (if appropriate)

Questions:

• Does the licenses files need update? No
• Is there breaking changes for older versions? No
• Does this needs documentation? No

The listners are added in duplicate on $scope.$on('setNoteMenu') calls.

[Leemoonsoo]

Thanks for the contribution.

I'm getting build error on zeppelin-web module

[INFO] Running "jscs:test" (jscs) task
[INFO] disallowMultipleLineBreaks: ; and it should have at most 2 line(s) between them at test/spec/controllers/notebook.js :
[INFO]    100 |    scope.killSaveTimer();
[INFO]    101 |    expect(scope.saveTimer).toEqual(null);
[INFO]    102 |  });
[INFO] -------------^
[INFO]    103 |
[INFO]    104 |
[INFO] >> 1 code style errors found!
[INFO] Warning: Task "jscs:test" failed. Use --force to continue.
[INFO] 
[INFO] Aborted due to warnings.

Could you check?

[cuspymd]

@Leemoonsoo Modified
Thanks.

[Leemoonsoo]

A test profile failure looks not related. (https://issues.apache.org/jira/browse/ZEPPELIN-1830)
LGTM and merge to master if there're no further comments.