Have TreeScopeStyleSheetCollection::document() return a reference.

Have TreeScopeStyleSheetCollection::document() return a reference.

Have TresScopeStyleSheetCollection::document() return a reference instead
of pointer as TreeScope::document() cannot return NULL.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=166297

[deepak.sa, 2014-01-30 10:54:57]

Please have a look.

[Inactive, 2014-01-30 14:33:35]

lgtm

[commit-bot: I haz the power, 2014-01-30 14:33:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/deepak.sa@samsung.com/148703004/20001

[commit-bot: I haz the power, 2014-01-30 14:33:47]

Failed to apply patch for Source/core/dom/StyleSheetCollection.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/dom/StyleSheetCollection.cpp
  Hunk #1 FAILED at 91.
  Hunk #2 FAILED at 217.
  2 out of 2 hunks FAILED -- saving rejects to file
Source/core/dom/StyleSheetCollection.cpp.rej

Patch:       Source/core/dom/StyleSheetCollection.cpp
Index: Source/core/dom/StyleSheetCollection.cpp
diff --git a/Source/core/dom/StyleSheetCollection.cpp
b/Source/core/dom/StyleSheetCollection.cpp
index
c655945b8618e2e59788fe7e67c34874da844c18..2c6ff7b81e264d4150cbfb7b790c2c2b07ed6ddd
100644
--- a/Source/core/dom/StyleSheetCollection.cpp
+++ b/Source/core/dom/StyleSheetCollection.cpp
@@ -91,7 +91,7 @@ void StyleSheetCollection::addStyleSheetCandidateNode(Node*
node, bool createdBy
     // since styles outside of the body and head continue to be shunted into
the head
     // (and thus can shift to end up before dynamically added DOM content that
is also
     // outside the body).
-    if (createdByParser && document()->body())
+    if (createdByParser && document().body())
         m_styleSheetCandidateNodes.parserAdd(node);
     else
         m_styleSheetCandidateNodes.add(node);
@@ -217,16 +217,16 @@ void
StyleSheetCollection::analyzeStyleSheetChange(StyleResolverUpdateMode updat
     // No point in doing the analysis work if we're just going to recalc the
whole document anyways.
     // This needs to be done after the compareStyleSheets calls above to ensure
we don't throw away
     // the StyleResolver if we don't need to.
-    if (document()->hasPendingForcedStyleRecalc())
+    if (document().hasPendingForcedStyleRecalc())
         return;
 
     // If we are already parsing the body and so may have significant amount of
elements, put some effort into trying to avoid style recalcs.
-    if (!document()->body() || document()->hasNodesWithPlaceholderStyle())
+    if (!document().body() || document().hasNodesWithPlaceholderStyle())
         return;
     StyleInvalidationAnalysis invalidationAnalysis(addedSheets);
     if (invalidationAnalysis.dirtiesAllStyle())
         return;
-    invalidationAnalysis.invalidateStyle(*document());
+    invalidationAnalysis.invalidateStyle(document());
     change.requiresFullStyleRecalc = false;
     return;
 }

[commit-bot: I haz the power, 2014-01-31 09:46:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/deepak.sa@samsung.com/148703004/60001

[commit-bot: I haz the power, 2014-01-31 12:42:12]

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

[commit-bot: I haz the power, 2014-01-31 12:42:12]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2014-01-31 12:42:18]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 12:42:22]

CQ bit was unchecked on CL. Ignoring.

[deepak.sa, 2014-02-03 07:40:45]

The CQ bit was checked by deepak.sa@samsung.com

[commit-bot: I haz the power, 2014-02-03 07:40:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/deepak.sa@samsung.com/148703004/60001

[commit-bot: I haz the power, 2014-02-03 09:03:44]

Change committed as 166297

[commit-bot: I haz the power, 2014-02-03 09:03:45]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-03 09:03:47]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-03 09:03:54]

CQ bit was unchecked on CL. Ignoring.