Account for null Document Frame when parsing inert attribute

Account for null Document Frame when parsing inert attribute

The SetIsInert method was added to Frame objects in r481761, which is
called when an inert attribute is parsed on an element within that
frame.

This CL removes the assumption that the current Document has a valid
Frame pointer, because that is sometimes false, such is in a
<template>.

BUG=736608
Review-Url: https://codereview.chromium.org/2961643002
Cr-Commit-Position: refs/heads/master@{#482999}
Committed: https://chromium.googlesource.com/chromium/src/+/c4570da4971280f5e012ced0a9d13167751035d1

[kenrb, 2017-06-26 16:44:55]

The CQ bit was checked by kenrb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-26 16:45:10]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kenrb, 2017-06-26 18:26:38]

kenrb@chromium.org changed reviewers:
+ tkent@chromium.org

[kenrb, 2017-06-26 18:26:39]

tkent@: PTAL?

[commit-bot: I haz the power, 2017-06-26 18:35:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-26 18:35:08]

Dry run: This issue passed the CQ dry run.

[tkent, 2017-06-26 22:29:56]

This CL needs a test.

[kenrb, 2017-06-27 21:27:22]

The CQ bit was checked by kenrb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-27 21:27:44]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kenrb, 2017-06-27 21:28:13]

Test added.

[tkent, 2017-06-27 22:47:14]

lgtm

[commit-bot: I haz the power, 2017-06-27 23:36:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-27 23:36:12]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[kenrb, 2017-06-28 13:32:30]

The CQ bit was checked by kenrb@chromium.org

[commit-bot: I haz the power, 2017-06-28 13:32:44]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-28 15:39:34]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498656750224200,
"parent_rev": "b6580ce6bda3382b3954c26776c206f08c7d46c8", "commit_rev":
"c4570da4971280f5e012ced0a9d13167751035d1"}

[commit-bot: I haz the power, 2017-06-28 15:39:50]

Description was changed from

==========
Account for null Document Frame when parsing inert attribute

The SetIsInert method was added to Frame objects in r481761, which is
called when an inert attribute is parsed on an element within that
frame.

This CL removes the assumption that the current Document has a valid
Frame pointer, because that is sometimes false, such is in a
<template>.

BUG=736608
==========

to

==========
Account for null Document Frame when parsing inert attribute

The SetIsInert method was added to Frame objects in r481761, which is
called when an inert attribute is parsed on an element within that
frame.

This CL removes the assumption that the current Document has a valid
Frame pointer, because that is sometimes false, such is in a
<template>.

BUG=736608

Review-Url: https://codereview.chromium.org/2961643002
Cr-Commit-Position: refs/heads/master@{#482999}
Committed:
https://chromium.googlesource.com/chromium/src/+/c4570da4971280f5e012ced0a9d1...
==========

[commit-bot: I haz the power, 2017-06-28 15:39:51]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/c4570da4971280f5e012ced0a9d1...