Apply :target styling even if fragment scrolling was skipped.

Source/core/frame/FrameView.h

 /*
    Copyright (C) 1997 Martin Jones (mjones@kde.org)
              (C) 1998 Waldo Bastian (bastian@kde.org)
              (C) 1998, 1999 Torben Weis (weis@kde.org)
              (C) 1999 Lars Knoll (knoll@kde.org)
              (C) 1999 Antti Koivisto (koivisto@kde.org)
    Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple 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 221 matching lines @@
     bool invalidateViewportConstrainedObjects();
 
     void incrementVisuallyNonEmptyCharacterCount(unsigned);
     void incrementVisuallyNonEmptyPixelCount(const IntSize&);
     void setIsVisuallyNonEmpty() { m_isVisuallyNonEmpty = true; }
     void enableAutoSizeMode(const IntSize& minSize, const IntSize& maxSize);
     void disableAutoSizeMode();
 
     void forceLayoutForPagination(const FloatSize& pageSize, const FloatSize& originalPageSize, float maximumShrinkFactor);
 
-    bool scrollToFragment(const KURL&);
-    bool scrollToAnchor(const String&);
-    void maintainScrollPositionAtAnchor(Node*);

[Nate Chapin, 2015/06/08 21:54:29]

Just to make sure I'm reading this correct: the only change to
maintainScrollPositionAtAnchor() is making it private?

[skobes, 2015/06/08 22:04:30]

Yes... I was making scrollToAnchor(const String&) private and noticed that this
one can be private as well.

+    enum UrlFragmentBehavior {

[Nate Chapin, 2015/06/08 21:54:29]

Would this be more readable as...

enum class UrlFragmentBehavior {
    Scroll,
    DontScroll
};

?

[skobes, 2015/06/08 22:04:30]

Do we want to start using enum class?  None of the other enums in FrameView are
done that way so I was trying to be consistent.

[Nate Chapin, 2015/06/08 22:17:31]

I've seen enum classes show up in blink lately and had them recommended to me a
couple times. I don't feel strongly; my comment was intended as an honest
question.

+        UrlFragmentScroll,
+        UrlFragmentDontScroll
+    };
+    bool processUrlFragment(const KURL&, UrlFragmentBehavior = UrlFragmentScroll);
 
     // Methods to convert points and rects between the coordinate space of the layoutObject, and this view.
     IntRect convertFromLayoutObject(const LayoutObject&, const IntRect&) const;
     IntRect convertToLayoutObject(const LayoutObject&, const IntRect&) const;
     IntPoint convertFromLayoutObject(const LayoutObject&, const IntPoint&) const;
     IntPoint convertToLayoutObject(const LayoutObject&, const IntPoint&) const;
 
     bool isFrameViewScrollCorner(LayoutScrollbarPart* scrollCorner) const { return m_scrollCorner == scrollCorner; }
 
     enum ScrollingReasons {
@@ skipping 404 matching lines @@
 
     bool wasViewportResized();
     void sendResizeEventIfNeeded();
 
     void updateScrollableAreaSet();
 
     void scheduleUpdateWidgetsIfNecessary();
     void updateWidgetsTimerFired(Timer<FrameView>*);
     bool updateWidgets();
 
+    bool processUrlFragmentHelper(const String&, UrlFragmentBehavior);
+    void maintainScrollPositionAtAnchor(Node*);
     void scrollToAnchor();
     void scrollPositionChanged();
     void didScrollTimerFired(Timer<FrameView>*);
 
     void updateLayersAndCompositingAfterScrollIfNeeded();
 
     static bool computeCompositedSelection(LocalFrame&, CompositedSelection&);
     void updateCompositedSelectionIfNeeded();
 
     // Returns true if the FrameView's own scrollbars overlay its content when visible.
@@ skipping 199 matching lines @@
     static const unsigned visualPixelThreshold = 32 * 32;
     if (m_visuallyNonEmptyPixelCount > visualPixelThreshold)
         setIsVisuallyNonEmpty();
 }
 
 DEFINE_TYPE_CASTS(FrameView, Widget, widget, widget->isFrameView(), widget.isFrameView());
 
 } // namespace blink
 
 #endif // FrameView_h