Fix TODOs (firebase#1787)

Author: mmermerkaya

packages/messaging/src/controllers/base-controller.ts

@@ -337,10 +337,7 @@ export abstract class BaseController implements FirebaseMessaging {
    * Returns the current Notification Permission state.
    */
   getNotificationPermission_(): NotificationPermission {
-    // TODO: Remove the cast when this issue is fixed:
-    // https://github.com/Microsoft/TypeScript/issues/14701
-    // tslint:disable-next-line no-any
-    return (Notification as any).permission;
+    return Notification.permission;
   }
 
   getTokenDetailsModel(): TokenDetailsModel {

packages/messaging/src/controllers/sw-controller.ts

@@ -108,11 +108,8 @@ export class SwController extends BaseController {
       const notificationTitle = notificationDetails.title || '';
       const reg = await this.getSWRegistration_();
 
-      // TODO: Remove casts to any and redundant type declarations when this
-      // PR lands: https://github.com/Microsoft/TSJS-lib-generator/pull/438
-      // tslint:disable no-any
-      const actions: object[] = (notificationDetails as any).actions;
-      const maxActions: number | undefined = (Notification as any).maxActions;
+      const { actions } = notificationDetails;
+      const { maxActions } = Notification;
       // tslint:enable no-any
       if (actions && maxActions && actions.length > maxActions) {
         console.warn(