Stop using controller as a variable name for FullscreenElementStack

Source/core/dom/FullscreenElementStack.h

Index: Source/core/dom/FullscreenElementStack.h
diff --git a/Source/core/dom/FullscreenElementStack.h b/Source/core/dom/FullscreenElementStack.h
index cf7c43487ca665f3b661df79834020b976e75695..8cff235ca0b1738a36bf5b2db3fc36a95c818d66 100644
--- a/Source/core/dom/FullscreenElementStack.h
+++ b/Source/core/dom/FullscreenElementStack.h
@@ -124,10 +124,10 @@ private:
 
 inline bool FullscreenElementStack::isActiveFullScreenElement(const Element& element)
 {
-    FullscreenElementStack* controller = fromIfExists(element.document());
-    if (!controller)
+    FullscreenElementStack* fullscreen = fromIfExists(element.document());
+    if (!fullscreen)
         return false;
-    return controller->webkitIsFullScreen() && controller->webkitCurrentFullScreenElement() == element;
+    return fullscreen->webkitCurrentFullScreenElement() == &element;

[falken, 2014/07/28 02:10:48]

I don't get how this worked before, webkitCurrentFullScreenElement() returns a
pointer yet we were comparing to a reference?

[philipj_slow, 2014/07/28 07:47:10]

That worked because of the
DEFINE_COMPARISON_OPERATORS_WITH_REFERENCES_REFCOUNTED macro, which was recently
discussed on blink-dev:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/6hUjMIIhesY/USLuqA1J...

I changed it for clarity, and in case that macro ends up being removed.

 }
 
 inline FullscreenElementStack* FullscreenElementStack::fromIfExists(Document& document)