Last Comment Bug 770550 - rename FrameWorker.jsm exported symbol (FrameWorker -> getFrameWorkerHandle?)
: rename FrameWorker.jsm exported symbol (FrameWorker -> getFrameWorkerHandle?)
Product: Firefox
Classification: Client Software
Component: SocialAPI (show other bugs)
: Trunk
: All All
-- normal (vote)
: Firefox 16
Assigned To: :Gavin Sharp [email:]
: Shane Caraveo (:mixedpuppy)
Depends on:
  Show dependency treegraph
Reported: 2012-07-03 08:51 PDT by :Gavin Sharp [email:]
Modified: 2012-07-04 06:37 PDT (History)
1 user (show) in‑testsuite+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (12.42 KB, patch)
2012-07-03 08:57 PDT, :Gavin Sharp [email:]
no flags Details | Diff | Splinter Review
patch (12.52 KB, patch)
2012-07-03 10:07 PDT, :Gavin Sharp [email:]
markh: review+
Details | Diff | Splinter Review

Description User image :Gavin Sharp [email:] 2012-07-03 08:51:57 PDT
Since this function isn't a constructor, and the returned object isn't a FrameWorker object, I think it makes sense to change the exported symbol's name.

I picked "getFrameWorkerHandle" because I'd chosen "WorkerHandle" in the previous patch. I'm not particularly attached to that name, though, and it is kind of ugly. Feedback desired!
Comment 1 User image :Gavin Sharp [email:] 2012-07-03 08:57:39 PDT
Created attachment 638755 [details] [diff] [review]
Comment 2 User image :Gavin Sharp [email:] 2012-07-03 10:07:39 PDT
Created attachment 638782 [details] [diff] [review]

This has a test fix.
Comment 3 User image :Gavin Sharp [email:] 2012-07-03 17:19:29 PDT
Comment 4 User image Ryan VanderMeulen [:RyanVM] 2012-07-04 06:37:45 PDT

Note You need to log in before you can comment on or make changes to this bug.