Skip to content

Commit

Permalink
improve name for layoutCallbackHappened --> layoutCompleted_
Browse files Browse the repository at this point in the history
  • Loading branch information
samouri committed Jan 23, 2020
1 parent 3a2c0c8 commit 6175421
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions extensions/amp-script/0.1/amp-script.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ export class AmpScript extends AMP.BaseElement {
this.debugId_ = 'amp-script[unknown].js';

/** @private {boolean} */
this.layoutCallbackHappened_ = false;
this.layoutCompleted_ = false;

/**
* If true, most production constraints are disabled including script size,
Expand Down Expand Up @@ -141,7 +141,7 @@ export class AmpScript extends AMP.BaseElement {
*/
onLayoutMeasure() {
super.onLayoutMeasure();
if (this.layoutCallbackHappened_) {
if (this.layoutCompleted_) {
return;
}

Expand Down Expand Up @@ -173,7 +173,7 @@ export class AmpScript extends AMP.BaseElement {

/** @override */
layoutCallback() {
this.layoutCallbackHappened_ = true;
this.layoutCompleted_ = true;
this.userActivation_ = new UserActivationTracker(this.element);

// The displayed name of the combined script in dev tools.
Expand Down

0 comments on commit 6175421

Please sign in to comment.