Navigation transitions (web to native app): Revert exit transition (Blink side)

Source/core/html/HTMLLinkElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Rob Buis (rwlbuis@gmail.com)
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 241 matching lines @@
 }
 
 void HTMLLinkElement::enableIfExitTransitionStyle()
 {
     if (m_relAttribute.isTransitionExitingStylesheet()) {
         if (LinkStyle* link = linkStyle())
             link->setDisabledState(false);
     }
 }
 
+void HTMLLinkElement::disableIfExitTransitionStyle()
+{
+    if (m_relAttribute.isTransitionExitingStylesheet()) {
+        if (LinkStyle* link = linkStyle())
+            link->setDisabledState(true);
+    }
+}
+
 Node::InsertionNotificationRequest HTMLLinkElement::insertedInto(ContainerNode* insertionPoint)
 {
     if (insertionPoint->inDocument()) {
         V8DOMActivityLogger* activityLogger = V8DOMActivityLogger::currentActivityLoggerIfIsolatedWorld();
         if (activityLogger) {
             Vector<String> argv;
             argv.append("link");
             argv.append(fastGetAttribute(relAttr));
             argv.append(fastGetAttribute(hrefAttr));
             activityLogger->logEvent("blinkAddElement", argv.size(), argv.data());
@@ skipping 380 matching lines @@
 
         if (m_sheet)
             m_sheet->setDisabled(disabled);
 
         // Load the sheet, since it's never been loaded before.
         if (!m_sheet && m_disabledState == EnabledViaScript) {
             if (m_owner->shouldProcessStyle())
                 process();
         } else {
             // FIXME: We don't have enough knowledge here to know if we should call addedStyleSheet() or removedStyleSheet().
-            m_owner->document().styleResolverChanged();
+            if (m_owner->relAttribute().isTransitionExitingStylesheet())
+                m_owner->document().styleResolverChanged(AnalyzedStyleUpdate);

[esprehn, 2014/10/15 18:01:26]

Why are transition sheets special? This doesn't seem right.

[Zhen Wang, 2014/10/15 19:50:58]

The default is FullStyleUpdate. We need AnalyzedStyleUpdate here. 

See the demo video in the CL description. I navigate from the page to a native
app and then press the back button to come back. If I don't treat transition
stylesheets specially here, the user interaction will not function properly
after coming back (cannot click to navigate any more.)

This is related to comments in CSSStyleSheet.cpp.

[esprehn, 2014/10/15 20:19:07]

ditto.

[Zhen Wang, 2014/10/15 23:04:41]

ditto

[Zhen Wang, 2014/12/03 17:54:29]

This is fixed by resetIsTransitionDocument now. 

+            else
+                m_owner->document().styleResolverChanged();
         }
     }
 }
 
 void LinkStyle::process()
 {
     ASSERT(m_owner->shouldProcessStyle());
     String type = m_owner->typeValue().lower();
     LinkRequestBuilder builder(m_owner);
 
@@ skipping 73 matching lines @@
         removePendingSheet();
 }
 
 void LinkStyle::trace(Visitor* visitor)
 {
     visitor->trace(m_sheet);
     LinkResource::trace(visitor);
 }
 
 } // namespace blink