history.pushState is not available in packaged apps.

[rpgobjects]

I get this error message when I run my chrome apps since m43 of chrome. Seems related to this change. https://codereview.chromium.org/988293002

It doesn't have an negative effect on my apps so far, but I only get it when I use angular (3.15) in my chrome apps.

I saw a related closed issue.
#1358

I'm using the angular-csp.css file in my app.

[jdiamond]

This prevents the error message from appearing for me:

angular
    .module('app', [], function($provide) {
        // Prevent Angular from sniffing for the history API
        // since it's not supported in packaged apps.
        $provide.decorator('$window', function($delegate) {
            $delegate.history = null;
            return $delegate;
        });
    })
;

[caitp]

well judging from the history of this, it seems like this feature has been disabled in chrome platform apps for a very long time (the bug is about getting rid of an extended IDL attribute and making the bindings layer simpler --- was this working for you during some window in packaged apps before M43?)

At any rate, @jdiamond's decorator trick will work for now, so maybe we can close this? I don't think we have a reliable way of detecting whether we're running in crippled API mode or not

[raghavcreddy]

The decorator is not working anymore starting from Chrome 48.0.2564.97 (64-bit).

Is there any workaround?

[gkalpak]

It is strange, because this workaround shouldn't be affected by browser behavior.
Do you have a (preferrably minimal) working demo of the issue ?

[raghavcreddy]

Here's a minimal working demo of the issue. It's a Chrome packaged app.
https://github.com/raghavcreddy/ChromeAppAngular

[gkalpak]

Thx @raghavcreddy. I'll take a look.

[gkalpak]

@raghavcreddy, indeed window.history isn't writable in Chrome 48.
I tried defining a new history getter on window and it seemed to work (see below). It achieves the same effect as the original workaround.

.config(function($provide) {
  $provide.decorator('$window', function($delegate) {
    Object.defineProperty($delegate, 'history', {get: () => null});
    return $delegate;
  });
});

// Or using the `.decorator()` helper:
.decorator('$window', function($delegate) {
  Object.defineProperty($delegate, 'history', {get: () => null});
  return $delegate;
});

[raghavcreddy]

Thank you @gkalpak

[gkalpak]

I have a fix for this is #13945. If that gets merged, then the workaround won't be necessary.
@raghavcreddy, btw, I think in Chrome Packaged Apps you should be running in CSP mode.

[schellmax]

thanks a lot for sharing the workaround.
for those not familiar with ES6 yet (like me), here's the 'back-ported' version ;)

    .decorator('$window', function($delegate) {
        Object.defineProperty($delegate, 'history', {get: function(){ return null; }});
        return $delegate;
    })